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

Unified Diff: content/browser/renderer_host/render_widget_host_view_android.h

Issue 2752113005: Let ImeAdapterAndroid have the same lifecycle as its Java peer (Closed)
Patch Set: fix tests Created 3 years, 8 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/renderer_host/render_widget_host_view_android.h
diff --git a/content/browser/renderer_host/render_widget_host_view_android.h b/content/browser/renderer_host/render_widget_host_view_android.h
index 1cd67ac225d2ba77cd6d7a7359a6569dd551cf71..a1492ec2b37ec8fcb379fc592c750eafc748c138 100644
--- a/content/browser/renderer_host/render_widget_host_view_android.h
+++ b/content/browser/renderer_host/render_widget_host_view_android.h
@@ -24,7 +24,6 @@
#include "content/browser/accessibility/browser_accessibility_manager.h"
#include "content/browser/android/content_view_core_impl_observer.h"
#include "content/browser/renderer_host/delegated_frame_evictor.h"
-#include "content/browser/renderer_host/ime_adapter_android.h"
#include "content/browser/renderer_host/input/stylus_text_selector.h"
#include "content/browser/renderer_host/render_widget_host_view_base.h"
#include "content/browser/renderer_host/text_input_manager.h"
@@ -51,6 +50,7 @@ struct DidOverscrollParams;
namespace content {
class ContentViewCoreImpl;
+class ImeAdapterAndroid;
class OverscrollControllerAndroid;
class RenderWidgetHost;
class RenderWidgetHostImpl;
@@ -227,6 +227,11 @@ class CONTENT_EXPORT RenderWidgetHostViewAndroid
void ResolveTapDisambiguation(double timestamp_seconds,
gfx::Point tap_viewport_offset,
bool is_long_press);
+ void set_ime_adapter(ImeAdapterAndroid* ime_adapter) {
+ ime_adapter_android_ = ime_adapter;
+ }
+
+ base::WeakPtr<RenderWidgetHostViewAndroid> GetWeakPtrAndroid();
void OnStartContentIntent(const GURL& content_url, bool is_main_frame);
@@ -236,8 +241,6 @@ class CONTENT_EXPORT RenderWidgetHostViewAndroid
void SetDoubleTapSupportEnabled(bool enabled);
void SetMultiTouchZoomSupportEnabled(bool enabled);
- long GetNativeImeAdapter();
-
void WasResized();
bool HasValidFrame() const;
@@ -272,7 +275,7 @@ class CONTENT_EXPORT RenderWidgetHostViewAndroid
void OnTextSelectionChanged(TextInputManager* text_input_manager,
RenderWidgetHostViewBase* updated_view) override;
- ImeAdapterAndroid* ime_adapter_for_testing() { return &ime_adapter_android_; }
+ ImeAdapterAndroid* ime_adapter_for_testing() { return ime_adapter_android_; }
// Exposed for tests.
cc::SurfaceId SurfaceIdForTesting() const override;
@@ -347,7 +350,7 @@ class CONTENT_EXPORT RenderWidgetHostViewAndroid
// ContentViewCoreImpl is our interface to the view system.
ContentViewCoreImpl* content_view_core_;
- ImeAdapterAndroid ime_adapter_android_;
+ ImeAdapterAndroid* ime_adapter_android_;
// Body background color of the underlying document.
SkColor cached_background_color_;
« no previous file with comments | « content/browser/renderer_host/ime_adapter_android.cc ('k') | content/browser/renderer_host/render_widget_host_view_android.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698