Chromium Code Reviews| Index: content/shell/android/java/src/org/chromium/content_shell/ShellManager.java |
| =================================================================== |
| --- content/shell/android/java/src/org/chromium/content_shell/ShellManager.java (revision 251875) |
| +++ content/shell/android/java/src/org/chromium/content_shell/ShellManager.java (working copy) |
| @@ -13,6 +13,7 @@ |
| import org.chromium.base.JNINamespace; |
| import org.chromium.base.ThreadUtils; |
| import org.chromium.content.browser.ContentView; |
| +import org.chromium.content.browser.ContentViewClient; |
| import org.chromium.content.browser.ContentViewRenderView; |
| import org.chromium.ui.base.WindowAndroid; |
| @@ -31,6 +32,7 @@ |
| // The target for all content rendering. |
| private ContentViewRenderView mContentViewRenderView; |
| + private ContentViewClient mContentViewClient; |
| /** |
| * Constructor for inflating via XML. |
| @@ -98,6 +100,10 @@ |
| mContentViewRenderView.setOverlayVideoMode(enabled); |
| } |
| + public void setContentViewClient(ContentViewClient client) { |
|
Ted C
2014/02/20 00:54:08
I would just have ShellManager build the ContentVi
|
| + mContentViewClient = client; |
| + } |
| + |
| @SuppressWarnings("unused") |
| @CalledByNative |
| private Object createShell(long nativeShellPtr) { |
| @@ -106,6 +112,7 @@ |
| (LayoutInflater) getContext().getSystemService(Context.LAYOUT_INFLATER_SERVICE); |
| Shell shellView = (Shell) inflater.inflate(R.layout.shell_view, null); |
| shellView.initialize(nativeShellPtr, mWindow); |
| + shellView.setContentViewClient(mContentViewClient); |
|
Ted C
2014/02/20 00:54:08
just pass this param with initialize above.
|
| // TODO(tedchoc): Allow switching back to these inactive shells. |
| if (mActiveShell != null) removeShell(mActiveShell); |