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

Unified Diff: ui/android/event_forwarder.cc

Issue 2896993002: Route OnDragEvent through ViewAndroid tree (Closed)
Patch Set: +ui/base Created 3 years, 7 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
« no previous file with comments | « ui/android/event_forwarder.h ('k') | ui/android/java/src/org/chromium/ui/base/EventForwarder.java » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ui/android/event_forwarder.cc
diff --git a/ui/android/event_forwarder.cc b/ui/android/event_forwarder.cc
index 911fa406931ffbfe9e7528202a2a03584125c80c..0a19b41732c3c40cab49bcedf5f894090b56507a 100644
--- a/ui/android/event_forwarder.cc
+++ b/ui/android/event_forwarder.cc
@@ -4,15 +4,18 @@
#include "ui/android/event_forwarder.h"
+#include "base/android/jni_array.h"
#include "base/metrics/histogram_macros.h"
#include "jni/EventForwarder_jni.h"
#include "ui/android/view_android.h"
-#include "ui/android/view_client.h"
+#include "ui/base/ui_base_switches_util.h"
+#include "ui/events/android/drag_event_android.h"
#include "ui/events/android/motion_event_android.h"
#include "ui/events/base_event_utils.h"
namespace ui {
+using base::android::AppendJavaStringArrayToStringVector;
using base::android::JavaParamRef;
using base::android::ScopedJavaLocalRef;
@@ -28,9 +31,10 @@ EventForwarder::~EventForwarder() {
ScopedJavaLocalRef<jobject> EventForwarder::GetJavaObject() {
if (java_obj_.is_null()) {
+ JNIEnv* env = base::android::AttachCurrentThread();
java_obj_.Reset(
- Java_EventForwarder_create(base::android::AttachCurrentThread(),
- reinterpret_cast<intptr_t>(this)));
+ Java_EventForwarder_create(env, reinterpret_cast<intptr_t>(this),
+ switches::IsTouchDragDropEnabled()));
}
return ScopedJavaLocalRef<jobject>(java_obj_);
}
@@ -137,6 +141,26 @@ void EventForwarder::OnMouseWheelEvent(JNIEnv* env,
view_->OnMouseWheelEvent(event);
}
+void EventForwarder::OnDragEvent(JNIEnv* env,
+ const JavaParamRef<jobject>& jobj,
+ jint action,
+ jint x,
+ jint y,
+ jint screen_x,
+ jint screen_y,
+ const JavaParamRef<jobjectArray>& j_mimeTypes,
+ const JavaParamRef<jstring>& j_content) {
+ float dip_scale = view_->GetDipScale();
+ gfx::PointF location(x / dip_scale, y / dip_scale);
+ gfx::PointF root_location(screen_x / dip_scale, screen_y / dip_scale);
+ std::vector<base::string16> mime_types;
+ AppendJavaStringArrayToStringVector(env, j_mimeTypes, &mime_types);
+
+ DragEventAndroid event(env, action, location, root_location, mime_types,
+ j_content.obj());
+ view_->OnDragEvent(event);
+}
+
bool RegisterEventForwarder(JNIEnv* env) {
return RegisterNativesImpl(env);
}
« no previous file with comments | « ui/android/event_forwarder.h ('k') | ui/android/java/src/org/chromium/ui/base/EventForwarder.java » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698