Support for Adaptive Handle Orientation

content/browser/android/popup_touch_handle_drawable.cc

 // Copyright 2014 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 "content/browser/android/popup_touch_handle_drawable.h"
 
 #include "jni/PopupTouchHandleDrawable_jni.h"
 
 namespace content {
 
 PopupTouchHandleDrawable::PopupTouchHandleDrawable(
     base::android::ScopedJavaLocalRef<jobject> drawable,
     float dpi_scale)
     : dpi_scale_(dpi_scale), drawable_(drawable) {
   DCHECK(drawable.obj());
+  drawable_horizontal_padding_ = GetPaddingFromResources();
 }
 
 PopupTouchHandleDrawable::~PopupTouchHandleDrawable() {
   // Explicitly disabling ensures that any external references to the Java
   // object are cleared, allowing it to be GC'ed in a timely fashion.
   SetEnabled(false);
 }
 
 void PopupTouchHandleDrawable::SetEnabled(bool enabled) {
   JNIEnv* env = base::android::AttachCurrentThread();
   if (enabled)
     Java_PopupTouchHandleDrawable_show(env, drawable_.obj());
   else
     Java_PopupTouchHandleDrawable_hide(env, drawable_.obj());
 }
 
 void PopupTouchHandleDrawable::SetOrientation(
-    ui::TouchHandleOrientation orientation) {
+    ui::TouchHandleOrientation orientation,
+    bool mirror_vertical,
+    bool mirror_horizontal) {
   JNIEnv* env = base::android::AttachCurrentThread();
   jobject obj = drawable_.obj();
-  Java_PopupTouchHandleDrawable_setOrientation(env, obj,
-                                               static_cast<int>(orientation));
+
+  Java_PopupTouchHandleDrawable_setOrientation(
+      env, obj, static_cast<int>(orientation), mirror_vertical,
+      mirror_horizontal);
+}
+
+void PopupTouchHandleDrawable::SetOrigin(const gfx::PointF& origin) {
+  JNIEnv* env = base::android::AttachCurrentThread();
+  jobject obj = drawable_.obj();
+
+  const gfx::PointF origin_pix = gfx::ScalePoint(origin, dpi_scale_);
+  Java_PopupTouchHandleDrawable_setOrigin(env, obj, origin_pix.x(),
+                                          origin_pix.y());
 }
 
 void PopupTouchHandleDrawable::SetAlpha(float alpha) {
   JNIEnv* env = base::android::AttachCurrentThread();
   bool visible = alpha > 0;
   Java_PopupTouchHandleDrawable_setVisible(env, drawable_.obj(), visible);
 }
 
-void PopupTouchHandleDrawable::SetFocus(const gfx::PointF& position) {
-  const gfx::PointF position_pix = gfx::ScalePoint(position, dpi_scale_);
-  JNIEnv* env = base::android::AttachCurrentThread();
-  Java_PopupTouchHandleDrawable_setFocus(
-      env, drawable_.obj(), position_pix.x(), position_pix.y());
-}
-
 gfx::RectF PopupTouchHandleDrawable::GetVisibleBounds() const {
   JNIEnv* env = base::android::AttachCurrentThread();
   gfx::RectF unscaled_rect(
       Java_PopupTouchHandleDrawable_getPositionX(env, drawable_.obj()),
       Java_PopupTouchHandleDrawable_getPositionY(env, drawable_.obj()),
       Java_PopupTouchHandleDrawable_getVisibleWidth(env, drawable_.obj()),
       Java_PopupTouchHandleDrawable_getVisibleHeight(env, drawable_.obj()));
   return gfx::ScaleRect(unscaled_rect, 1.f / dpi_scale_);
 }
 
+const float PopupTouchHandleDrawable::GetPaddingFromResources() {

[jdduke (slow), 2015/08/17 17:20:50]

Nit: This is only use once, let's just inline it in the constructor.

[AviD, 2015/08/18 13:56:14]

Done.

+  JNIEnv* env = base::android::AttachCurrentThread();
+  const float horizontal_padding =
+      Java_PopupTouchHandleDrawable_getHandleHorizontalPadding(env,
+                                                               drawable_.obj());
+  return horizontal_padding;
+}
+
+const float PopupTouchHandleDrawable::GetDrawableHorizontalPadding() const {
+  return drawable_horizontal_padding_;
+}
+
 // static
 bool PopupTouchHandleDrawable::RegisterPopupTouchHandleDrawable(JNIEnv* env) {
   return RegisterNativesImpl(env);
 }
 
 }  // namespace content