| Index: content/browser/android/content_view_core_impl.cc
|
| diff --git a/content/browser/android/content_view_core_impl.cc b/content/browser/android/content_view_core_impl.cc
|
| index 392db7b22efd445ef65ae90fb3b2a9caad6f73e2..542594337964a7749cd7e12016b83cbbaf40ba39 100644
|
| --- a/content/browser/android/content_view_core_impl.cc
|
| +++ b/content/browser/android/content_view_core_impl.cc
|
| @@ -55,6 +55,7 @@
|
| #include "jni/DragEvent_jni.h"
|
| #include "third_party/WebKit/public/platform/WebInputEvent.h"
|
| #include "ui/android/view_android.h"
|
| +#include "ui/android/view_root.h"
|
| #include "ui/android/window_android.h"
|
| #include "ui/base/clipboard/clipboard.h"
|
| #include "ui/base/ui_base_switches_util.h"
|
| @@ -230,12 +231,6 @@ ContentViewCoreImpl::ContentViewCoreImpl(
|
| dpi_scale_(dpi_scale),
|
| device_orientation_(0),
|
| accessibility_enabled_(false) {
|
| - GetViewAndroid()->SetLayer(cc::Layer::Create());
|
| - gfx::Size physical_size(
|
| - Java_ContentViewCore_getPhysicalBackingWidthPix(env, obj),
|
| - Java_ContentViewCore_getPhysicalBackingHeightPix(env, obj));
|
| - GetViewAndroid()->GetLayer()->SetBounds(physical_size);
|
| -
|
| // Currently, the only use case we have for overriding a user agent involves
|
| // spoofing a desktop Linux user agent for "Request desktop site".
|
| // Automatically set it for all WebContents so that it is available when a
|
| @@ -277,22 +272,23 @@ ContentViewCoreImpl::~ContentViewCoreImpl() {
|
| }
|
| }
|
|
|
| -void ContentViewCoreImpl::UpdateWindowAndroid(
|
| +void ContentViewCoreImpl::UpdateViewRoot(
|
| JNIEnv* env,
|
| const base::android::JavaParamRef<jobject>& obj,
|
| - jlong window_android) {
|
| + jlong view_root_ptr) {
|
| + ui::ViewRoot* view_root = reinterpret_cast<ui::ViewRoot*>(view_root_ptr);
|
| ui::ViewAndroid* view = GetViewAndroid();
|
| - ui::WindowAndroid* window =
|
| - reinterpret_cast<ui::WindowAndroid*>(window_android);
|
| - if (window == GetWindowAndroid())
|
| + if (view_root == view->GetViewRoot())
|
| return;
|
| - if (GetWindowAndroid()) {
|
| +
|
| + if (view->GetViewRoot()) {
|
| for (auto& observer : observer_list_)
|
| observer.OnDetachedFromWindow();
|
| view->RemoveFromParent();
|
| }
|
| - if (window) {
|
| - window->AddChild(view);
|
| +
|
| + if (view_root) {
|
| + view_root->AddChild(view);
|
| for (auto& observer : observer_list_)
|
| observer.OnAttachedToWindow();
|
| }
|
| @@ -1273,6 +1269,10 @@ void ContentViewCoreImpl::WasResized(JNIEnv* env,
|
| Java_ContentViewCore_getPhysicalBackingHeightPix(env, obj));
|
| GetViewAndroid()->GetLayer()->SetBounds(physical_size);
|
|
|
| + gfx::Size view_size(GetViewSize());
|
| + GetViewAndroid()->SetBounds(gfx::Point(),
|
| + view_size.width(),
|
| + view_size.height());
|
| SendScreenRectsAndResizeWidget();
|
| }
|
|
|
| @@ -1444,6 +1444,26 @@ bool ContentViewCoreImpl::IsTouchDragDropEnabled(
|
| return switches::IsTouchDragDropEnabled();
|
| }
|
|
|
| +void ContentViewCoreImpl::MoveToFrontInViewHierarchy(
|
| + JNIEnv* env,
|
| + const base::android::JavaParamRef<jobject>& jobj) {
|
| + ui::ViewAndroid* view_android = GetViewAndroid();
|
| + ui::ViewRoot* view_root =
|
| + static_cast<ui::ViewRoot*>(view_android->GetViewRoot());
|
| + view_root->MoveToFront(view_android);
|
| +}
|
| +
|
| +void ContentViewCoreImpl::UpdateViewBounds(
|
| + JNIEnv* env,
|
| + const base::android::JavaParamRef<jobject>& jobj,
|
| + int x,
|
| + int y,
|
| + int width,
|
| + int height) {
|
| + ui::ViewAndroid* view_android = GetViewAndroid();
|
| + view_android->SetBounds(gfx::Point(x, y), width, height);
|
| +}
|
| +
|
| void ContentViewCoreImpl::OnDragEvent(
|
| JNIEnv* env,
|
| const base::android::JavaParamRef<jobject>& jobj,
|
| @@ -1552,7 +1572,7 @@ jlong Init(JNIEnv* env,
|
| const JavaParamRef<jobject>& obj,
|
| const JavaParamRef<jobject>& jweb_contents,
|
| const JavaParamRef<jobject>& jview_android_delegate,
|
| - jlong jwindow_android,
|
| + jlong view_root_ptr,
|
| jfloat dipScale,
|
| const JavaParamRef<jobject>& retained_objects_set) {
|
| WebContentsImpl* web_contents = static_cast<WebContentsImpl*>(
|
| @@ -1561,11 +1581,14 @@ jlong Init(JNIEnv* env,
|
| "A ContentViewCoreImpl should be created with a valid WebContents.";
|
| ui::ViewAndroid* view_android = web_contents->GetView()->GetNativeView();
|
| view_android->SetDelegate(jview_android_delegate);
|
| + view_android->SetLayer(cc::Layer::Create());
|
| + gfx::Size physical_size(
|
| + Java_ContentViewCore_getPhysicalBackingWidthPix(env, obj),
|
| + Java_ContentViewCore_getPhysicalBackingHeightPix(env, obj));
|
| + view_android->GetLayer()->SetBounds(physical_size);
|
|
|
| - ui::WindowAndroid* window_android =
|
| - reinterpret_cast<ui::WindowAndroid*>(jwindow_android);
|
| - DCHECK(window_android);
|
| - window_android->AddChild(view_android);
|
| + ui::ViewRoot* view_root = reinterpret_cast<ui::ViewRoot*>(view_root_ptr);
|
| + view_root->AddChild(view_android);
|
|
|
| // TODO: pass dipScale.
|
| ContentViewCoreImpl* view = new ContentViewCoreImpl(
|
|
|