Index: content/browser/renderer_host/ime_adapter_android.cc |
diff --git a/content/browser/renderer_host/ime_adapter_android.cc b/content/browser/renderer_host/ime_adapter_android.cc |
index 3a7d18b94da7ce41b8a4981e98e7d87b3eda70cc..08c42fd00c06081238559e5e6ae4b06f0d939a49 100644 |
--- a/content/browser/renderer_host/ime_adapter_android.cc |
+++ b/content/browser/renderer_host/ime_adapter_android.cc |
@@ -110,6 +110,7 @@ void AppendUnderlineSpan(JNIEnv*, |
ImeAdapterAndroid::ImeAdapterAndroid(RenderWidgetHostViewAndroid* rwhva) |
: rwhva_(rwhva) { |
+ DCHECK(rwhva_); |
} |
ImeAdapterAndroid::~ImeAdapterAndroid() { |
@@ -142,7 +143,7 @@ void ImeAdapterAndroid::SetComposingText(JNIEnv* env, |
const JavaParamRef<jobject>& text, |
const JavaParamRef<jstring>& text_str, |
int relative_cursor_pos) { |
- RenderWidgetHostImpl* rwhi = GetRenderWidgetHostImpl(); |
+ RenderWidgetHostImpl* rwhi = GetFocusedWidget(); |
if (!rwhi) |
return; |
@@ -172,7 +173,7 @@ void ImeAdapterAndroid::CommitText(JNIEnv* env, |
const JavaParamRef<jobject>& text, |
const JavaParamRef<jstring>& text_str, |
int relative_cursor_pos) { |
- RenderWidgetHostImpl* rwhi = GetRenderWidgetHostImpl(); |
+ RenderWidgetHostImpl* rwhi = GetFocusedWidget(); |
if (!rwhi) |
return; |
@@ -195,7 +196,7 @@ void ImeAdapterAndroid::CommitText(JNIEnv* env, |
void ImeAdapterAndroid::FinishComposingText(JNIEnv* env, |
const JavaParamRef<jobject>&) { |
- RenderWidgetHostImpl* rwhi = GetRenderWidgetHostImpl(); |
+ RenderWidgetHostImpl* rwhi = GetFocusedWidget(); |
if (!rwhi) |
return; |
@@ -288,7 +289,7 @@ void ImeAdapterAndroid::DeleteSurroundingText(JNIEnv*, |
bool ImeAdapterAndroid::RequestTextInputStateUpdate( |
JNIEnv* env, |
const JavaParamRef<jobject>&) { |
- RenderWidgetHostImpl* rwhi = GetRenderWidgetHostImpl(); |
+ RenderWidgetHostImpl* rwhi = GetFocusedWidget(); |
if (!rwhi) |
return false; |
rwhi->Send(new InputMsg_RequestTextInputStateUpdate(rwhi->GetRoutingID())); |
@@ -300,7 +301,7 @@ void ImeAdapterAndroid::RequestCursorUpdate( |
const base::android::JavaParamRef<jobject>& obj, |
bool immediate_request, |
bool monitor_request) { |
- RenderWidgetHostImpl* rwhi = GetRenderWidgetHostImpl(); |
+ RenderWidgetHostImpl* rwhi = GetFocusedWidget(); |
if (!rwhi) |
return; |
rwhi->Send(new InputMsg_RequestCompositionUpdate( |
@@ -312,23 +313,22 @@ void ImeAdapterAndroid::ResetImeAdapter(JNIEnv* env, |
java_ime_adapter_.reset(); |
} |
-RenderWidgetHostImpl* ImeAdapterAndroid::GetRenderWidgetHostImpl() { |
+RenderWidgetHostImpl* ImeAdapterAndroid::GetFocusedWidget() { |
DCHECK_CURRENTLY_ON(BrowserThread::UI); |
- DCHECK(rwhva_); |
- RenderWidgetHost* rwh = rwhva_->GetRenderWidgetHost(); |
- if (!rwh) |
- return nullptr; |
- |
- return RenderWidgetHostImpl::From(rwh); |
+ return rwhva_->GetFocusedWidget(); |
} |
RenderFrameHost* ImeAdapterAndroid::GetFocusedFrame() { |
- RenderWidgetHostImpl* rwh = GetRenderWidgetHostImpl(); |
+ DCHECK_CURRENTLY_ON(BrowserThread::UI); |
+ RenderWidgetHostImpl* rwh = |
+ RenderWidgetHostImpl::From(rwhva_->GetRenderWidgetHost()); |
if (!rwh) |
return nullptr; |
+ |
RenderViewHost* rvh = RenderViewHost::From(rwh); |
if (!rvh) |
return nullptr; |
+ |
EhsanK
2017/01/26 19:16:29
I deliberately added space here since I though it
|
FrameTreeNode* focused_frame = |
rvh->GetDelegate()->GetFrameTree()->GetFocusedFrame(); |
if (!focused_frame) |
@@ -338,10 +338,13 @@ RenderFrameHost* ImeAdapterAndroid::GetFocusedFrame() { |
} |
WebContents* ImeAdapterAndroid::GetWebContents() { |
- RenderWidgetHostImpl* rwh = GetRenderWidgetHostImpl(); |
- if (!rwh) |
+ DCHECK_CURRENTLY_ON(BrowserThread::UI); |
+ RenderWidgetHostImpl* rwh = |
+ RenderWidgetHostImpl::From(rwhva_->GetRenderWidgetHost()); |
+ if (!rwh || !rwh->delegate()) |
return nullptr; |
- return WebContents::FromRenderViewHost(RenderViewHost::From(rwh)); |
+ |
+ return rwh->GetAsWebContents(); |
} |
std::vector<blink::WebCompositionUnderline> |