Implement Mac Pop-up Dictionary for OOPIF.

content/renderer/text_input_client_observer.cc

 // Copyright (c) 2012 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/renderer/text_input_client_observer.h"
 
 #include <stddef.h>
 
 #include <memory>
 
 #include "build/build_config.h"
 #include "content/common/text_input_client_messages.h"
 #include "content/renderer/pepper/pepper_plugin_instance_impl.h"
+#include "content/renderer/render_frame_impl.h"
 #include "content/renderer/render_view_impl.h"
+#include "content/renderer/render_widget.h"
+#include "ipc/ipc_message.h"
 #include "third_party/WebKit/public/platform/WebPoint.h"
 #include "third_party/WebKit/public/platform/WebRect.h"
 #include "third_party/WebKit/public/platform/WebString.h"
 #include "third_party/WebKit/public/web/WebLocalFrame.h"
 #include "third_party/WebKit/public/web/WebView.h"
 #include "third_party/WebKit/public/web/mac/WebSubstringUtil.h"
 #include "ui/gfx/geometry/rect.h"
 
 namespace content {
 
-TextInputClientObserver::TextInputClientObserver(RenderViewImpl* render_view)
-#if defined(ENABLE_PLUGINS)
-    : RenderViewObserver(render_view), render_view_impl_(render_view) {
-#else
-    : RenderViewObserver(render_view) {
-#endif
-}
+TextInputClientObserver::TextInputClientObserver(RenderWidget* render_widget)
+    : render_widget_(render_widget) {}
 
 TextInputClientObserver::~TextInputClientObserver() {
 }
 
 bool TextInputClientObserver::OnMessageReceived(const IPC::Message& message) {
   bool handled = true;
   IPC_BEGIN_MESSAGE_MAP(TextInputClientObserver, message)
     IPC_MESSAGE_HANDLER(TextInputClientMsg_StringAtPoint,
                         OnStringAtPoint)
     IPC_MESSAGE_HANDLER(TextInputClientMsg_CharacterIndexForPoint,
                         OnCharacterIndexForPoint)
     IPC_MESSAGE_HANDLER(TextInputClientMsg_FirstRectForCharacterRange,
                         OnFirstRectForCharacterRange)
     IPC_MESSAGE_HANDLER(TextInputClientMsg_StringForRange, OnStringForRange)
     IPC_MESSAGE_UNHANDLED(handled = false)
   IPC_END_MESSAGE_MAP()
   return handled;
 }
 
-void TextInputClientObserver::OnDestruct() {
-  delete this;
+bool TextInputClientObserver::Send(IPC::Message* message) {
+  return render_widget_->Send(message);
 }
 
-blink::WebView* TextInputClientObserver::webview() {
-  return render_view()->GetWebView();
+blink::WebFrameWidget* TextInputClientObserver::GetWebFrameWidget() const {
+  return static_cast<blink::WebFrameWidget*>(render_widget_->GetWebWidget());
 }
 
+blink::WebLocalFrame* TextInputClientObserver::GetFocusedFrame() const {
+  return RenderFrameImpl::FromWebFrame(GetWebFrameWidget()->localRoot())
+      ->render_view()
+      ->webview()
+      ->focusedFrame();
+}
+
+#if defined(ENABLE_PLUGINS)
+PepperPluginInstanceImpl* TextInputClientObserver::GetFocusedPepperPlugin()

[EhsanK, 2016/09/14 22:10:32]

lfg@: I think it will be more correct if we returned the focused pepper plugin
which belongs to the widget instead.  I assume we can have such plugings in
OOPIFs, right? In that case we are finding it from the page. What if the plugin
changes focus before the IPC arrives for the previously focused plugin in a
different widget (same process)? Maybe I should compare
plugin->render_frame()->GetWebFrame()->localRoot() agains the local root of the
widget? WDYT?

[lfg, 2016/09/15 15:51:51]

Is it possible that a frame is not focused, but a pepper plugin inside the frame
is focused? Why can't you just return the focused plugin of the focused frame?

[EhsanK, 2016/09/19 22:34:05]

I don't think it is (could not repro locally). I also think getting it from the
focused frame (in widget) is a better idea.

+    const {
+  return RenderFrameImpl::FromWebFrame(GetWebFrameWidget()->localRoot())
+            ->render_view()
+            ->GetFocusedPepperPlugin();
+}
+#endif
+
 void TextInputClientObserver::OnStringAtPoint(gfx::Point point) {
 #if defined(OS_MACOSX)
   blink::WebPoint baselinePoint;
   NSAttributedString* string = blink::WebSubstringUtil::attributedWordAtPoint(
-      webview(), point, baselinePoint);
+      GetWebFrameWidget(), point, baselinePoint);
 
   std::unique_ptr<const mac::AttributedStringCoder::EncodedString> encoded(
       mac::AttributedStringCoder::Encode(string));
   Send(new TextInputClientReplyMsg_GotStringAtPoint(
-      routing_id(), *encoded.get(), baselinePoint));
+      render_widget_->routing_id(), *encoded.get(), baselinePoint));
 #else
   NOTIMPLEMENTED();
 #endif
 }
 
 void TextInputClientObserver::OnCharacterIndexForPoint(gfx::Point point) {
   blink::WebPoint web_point(point);
   uint32_t index = static_cast<uint32_t>(
-      webview()->focusedFrame()->characterIndexForPoint(web_point));
-  Send(new TextInputClientReplyMsg_GotCharacterIndexForPoint(routing_id(),
-      index));
+      GetFocusedFrame()->characterIndexForPoint(web_point));
+  Send(new TextInputClientReplyMsg_GotCharacterIndexForPoint(
+      render_widget_->routing_id(), index));
 }
 
 void TextInputClientObserver::OnFirstRectForCharacterRange(gfx::Range range) {
   gfx::Rect rect;
 #if defined(ENABLE_PLUGINS)
-  if (render_view_impl_->GetFocusedPepperPlugin()) {
-    rect = render_view_impl_->GetFocusedPepperPlugin()->GetCaretBounds();
+  PepperPluginInstanceImpl* focused_plugin = GetFocusedPepperPlugin();
+  if (focused_plugin) {
+    rect = focused_plugin->GetCaretBounds();
   } else
 #endif
   {
-    blink::WebLocalFrame* frame = webview()->focusedFrame();
+    blink::WebLocalFrame* frame = GetFocusedFrame();
     // TODO(yabinh): Null check should not be necessary.
     // See crbug.com/304341
     if (frame) {
       blink::WebRect web_rect;
       frame->firstRectForCharacterRange(range.start(), range.length(),
                                         web_rect);
       rect = web_rect;
     }
   }
-  Send(new TextInputClientReplyMsg_GotFirstRectForRange(routing_id(), rect));
+  Send(new TextInputClientReplyMsg_GotFirstRectForRange(
+      render_widget_->routing_id(), rect));
 }
 
 void TextInputClientObserver::OnStringForRange(gfx::Range range) {
 #if defined(OS_MACOSX)
   blink::WebPoint baselinePoint;
   NSAttributedString* string = nil;
-  blink::WebLocalFrame* frame = webview()->focusedFrame();
+  blink::WebLocalFrame* frame = GetFocusedFrame();
   // TODO(yabinh): Null check should not be necessary.
   // See crbug.com/304341
   if (frame) {
     string = blink::WebSubstringUtil::attributedSubstringInRange(
         frame, range.start(), range.length(), &baselinePoint);
   }
   std::unique_ptr<const mac::AttributedStringCoder::EncodedString> encoded(
       mac::AttributedStringCoder::Encode(string));
-  Send(new TextInputClientReplyMsg_GotStringForRange(routing_id(),
-      *encoded.get(), baselinePoint));
+  Send(new TextInputClientReplyMsg_GotStringForRange(
+      render_widget_->routing_id(), *encoded.get(), baselinePoint));
 #else
   NOTIMPLEMENTED();
 #endif
 }
 
 }  // namespace content