Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(717)

Unified Diff: content/browser/android/content_view_core_impl.cc

Issue 2595263002: Introduce ViewRoot forwarding input/view events to native (Closed)
Patch Set: unittest Created 3 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
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(
« no previous file with comments | « content/browser/android/content_view_core_impl.h ('k') | content/browser/android/content_view_render_view.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698