Make ContentViewRenderView's SurfaceView injectable

Make ContentViewRenderView's SurfaceView injectable

For apps using ContentViewRenderView, make it possible to inject
SurfaceView object that ContentViewRenderView will use. Provide
default operation which just creates a new SurfaceView object.

BUG=229369
R=yfriedman@chromium.org
TBR=yfriedman@chromium.org

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=193347

[wonsik, 2013-04-09 12:33:26]

Please take a look. Thanks!

[wonsik, 2013-04-09 12:34:03]

+ycheo

[Yaron, 2013-04-09 16:20:41]

+dtrainor

[David Trainor- moved to gerrit, 2013-04-09 17:21:21]

lgtm

https://codereview.chromium.org/13812009/diff/1/content/public/android/java/s...
File
content/public/android/java/src/org/chromium/content/browser/ContentViewRenderView.java
(right):

https://codereview.chromium.org/13812009/diff/1/content/public/android/java/s...
content/public/android/java/src/org/chromium/content/browser/ContentViewRenderView.java:93:
protected SurfaceView createSurfaceView(Context context) {
javadoc :)

[wonsik, 2013-04-10 01:39:05]

Thanks for the quick review!

https://codereview.chromium.org/13812009/diff/1/content/public/android/java/s...
File
content/public/android/java/src/org/chromium/content/browser/ContentViewRenderView.java
(right):

https://codereview.chromium.org/13812009/diff/1/content/public/android/java/s...
content/public/android/java/src/org/chromium/content/browser/ContentViewRenderView.java:93:
protected SurfaceView createSurfaceView(Context context) {
On 2013/04/09 17:21:21, David Trainor wrote:
> javadoc :)

Done.

[commit-bot: I haz the power, 2013-04-10 01:39:48]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/wonsik@chromium.org/13812009/7001

[commit-bot: I haz the power, 2013-04-10 01:39:50]

Presubmit check for 13812009-7001 failed and returned exit status 1.

INFO:root:Found 1 file(s).

Running presubmit commit checks ...
Running /b/commit-queue/workdir/chromium/PRESUBMIT.py

** Presubmit ERRORS **
Missing LGTM from an OWNER for these files:
   
content/public/android/java/src/org/chromium/content/browser/ContentViewRenderView.java

[commit-bot: I haz the power, 2013-04-10 04:57:46]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/wonsik@chromium.org/13812009/7001

[commit-bot: I haz the power, 2013-04-10 08:51:14]

Change committed as 193347