Introduce ViewRoot to forward input/view events to native

ui/android/view_android.cc

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "ui/android/view_android.h"
 
 #include <algorithm>
 
 #include "base/android/jni_android.h"
 #include "cc/layers/layer.h"
 #include "jni/ViewAndroidDelegate_jni.h"
+#include "ui/android/view_client.h"
 #include "ui/android/window_android.h"
 #include "ui/display/display.h"
 #include "ui/display/screen.h"
 
 namespace ui {
 
+using base::android::JavaParamRef;
 using base::android::JavaRef;
 using base::android::ScopedJavaLocalRef;
 
 ViewAndroid::ScopedAnchorView::ScopedAnchorView(
     JNIEnv* env,
     const JavaRef<jobject>& jview,
     const JavaRef<jobject>& jdelegate)
     : view_(env, jview.obj()), delegate_(env, jdelegate.obj()) {
   // If there's a view, then we need a delegate to remove it.
   DCHECK(!jdelegate.is_null() || jview.is_null());
@@ skipping 33 matching lines @@
   view_.reset();
   delegate_.reset();
 }
 
 const base::android::ScopedJavaLocalRef<jobject>
 ViewAndroid::ScopedAnchorView::view() const {
   JNIEnv* env = base::android::AttachCurrentThread();
   return view_.get(env);
 }
 
-ViewAndroid::ViewAndroid(const JavaRef<jobject>& delegate)
-    : parent_(nullptr)
-    , delegate_(base::android::AttachCurrentThread(),
-                delegate.obj()) {}
-
-ViewAndroid::ViewAndroid() : parent_(nullptr) {}
+ViewAndroid::ViewAndroid(ViewClient* client) : parent_(nullptr),
+                                               client_(client) {}
+ViewAndroid::ViewAndroid() : ViewAndroid(nullptr) {}
 
 ViewAndroid::~ViewAndroid() {
   RemoveFromParent();
 
   for (std::list<ViewAndroid*>::iterator it = children_.begin();
        it != children_.end(); it++) {
     DCHECK_EQ((*it)->parent_, this);
     (*it)->parent_ = nullptr;
   }
 }
 
 void ViewAndroid::SetDelegate(const JavaRef<jobject>& delegate) {
+  // A ViewAndroid may have its own delegate or otherwise will
+  // use the next available parent's delegate.
   JNIEnv* env = base::android::AttachCurrentThread();
   delegate_ = JavaObjectWeakGlobalRef(env, delegate);
 }
 
 void ViewAndroid::AddChild(ViewAndroid* child) {
   DCHECK(child);
   DCHECK(std::find(children_.begin(), children_.end(), child) ==
          children_.end());
 
   children_.push_back(child);
@@ skipping 73 matching lines @@
 bool ViewAndroid::StartDragAndDrop(const JavaRef<jstring>& jtext,
                                    const JavaRef<jobject>& jimage) {
   ScopedJavaLocalRef<jobject> delegate(GetViewAndroidDelegate());
   if (delegate.is_null())
     return false;
   JNIEnv* env = base::android::AttachCurrentThread();
   return Java_ViewAndroidDelegate_startDragAndDrop(env, delegate, jtext,
                                                    jimage);
 }
 
+int ViewAndroid::GetPhysicalBackingSizeWidthPix() {
+  if (!parent_ || parent_ == GetWindowAndroid())

[boliu, 2016/12/06 00:34:14]

should just return own size, not recurse

ie we should pay the cost to update the whole tree in SetPhysicalBackingSize, so
that the getter is free

[Jinsuk Kim, 2016/12/06 07:35:07]

Done.

+    return physical_width_pix_;
+  else
+    return parent_->GetPhysicalBackingSizeWidthPix();
+}
+
+int ViewAndroid::GetPhysicalBackingSizeHeightPix() {
+  if (!parent_ || parent_ == GetWindowAndroid())
+    return physical_height_pix_;
+  else
+    return parent_->GetPhysicalBackingSizeHeightPix();
+}
+
+gfx::Size ViewAndroid::GetPhysicalBackingSize() {
+  return gfx::Size(GetPhysicalBackingSizeWidthPix(),
+                   GetPhysicalBackingSizeHeightPix());
+}
+
+void ViewAndroid::SetPhysicalBackingSize(int width, int height) {
+  if (!parent_ || parent_ == GetWindowAndroid()) {
+    physical_width_pix_ = width;
+    physical_height_pix_ = height;
+  } else {
+    return parent_->SetPhysicalBackingSize(width, height);

[boliu, 2016/12/06 00:34:14]

recursion in the wrong direction.. this value should be propagated *down*, not
up?

oh actually, looking at OnPhysicalBackingSizeChanged below, this should not
recurse at all?

[Jinsuk Kim, 2016/12/06 07:35:07]

Done.

+  }
+}
+
+void ViewAndroid::UpdateLayerBounds() {
+  layer_->SetBounds(GetPhysicalBackingSize());

[boliu, 2016/12/06 00:34:14]

just do this inside SetPhyiscalBackingSize, then client don't ever need to worry
about UpdateLayerBounds

[Jinsuk Kim, 2016/12/06 07:35:07]

Done.

+}
+
+bool ViewAndroid::OnPhysicalBackingSizeChanged(int width, int height) {
+  if (width == GetPhysicalBackingSizeWidthPix()
+       && height == GetPhysicalBackingSizeHeightPix())
+    return true;
+
+  if (client_ && client_->OnPhysicalBackingSizeChanged(width, height)) {
+    SetPhysicalBackingSize(width, height);
+    UpdateLayerBounds();
+    return true;
+  }
+  for (auto it = children_.begin(); it != children_.end(); ++it) {
+    if ((*it)->OnPhysicalBackingSizeChanged(width, height))
+      return true;
+  }
+  return false;
+}
+
 }  // namespace ui