Implement Mac Pop-up Dictionary for OOPIF.

content/browser/site_per_process_mac_browsertest.mm

Index: content/browser/site_per_process_mac_browsertest.mm
diff --git a/content/browser/site_per_process_mac_browsertest.mm b/content/browser/site_per_process_mac_browsertest.mm
new file mode 100644
index 0000000000000000000000000000000000000000..01b797314d429054293bccad6eb9e1d971a4537d
--- /dev/null
+++ b/content/browser/site_per_process_mac_browsertest.mm
@@ -0,0 +1,126 @@
+// Copyright (c) 2016 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/site_per_process_browsertest.h"
+
+#include "content/public/test/content_browser_test_utils.h"
+#include "content/public/test/test_utils.h"
+
+namespace content {
+
+namespace {
+
+// Helper class for TextInputClientMac.
+class TextInputClientMacHelper {
+ public:
+  TextInputClientMacHelper() {}
+  ~TextInputClientMacHelper() {}
+
+  void WaitForStringFromRange(RenderWidgetHost* rwh, const gfx::Range& range) {
+    GetStringFromRangeForRenderWidget(
+        rwh, range, base::Bind(&TextInputClientMacHelper::OnResult,
+                               base::Unretained(this)));
+    loop_runner_ = new MessageLoopRunner();
+    loop_runner_->Run();
+  }
+
+  void WaitForStringAtPoint(RenderWidgetHost* rwh, const gfx::Point& point) {
+    GetStringAtPointForRenderWidget(
+        rwh, point, base::Bind(&TextInputClientMacHelper::OnResult,
+                               base::Unretained(this)));
+    loop_runner_ = new MessageLoopRunner();
+    loop_runner_->Run();
+  }
+  const std::string& word() const { return word_; }
+  const gfx::Point& point() const { return point_; }
+
+ private:
+  void OnResult(const std::string& string, const gfx::Point& point) {
+    if (!BrowserThread::CurrentlyOn(BrowserThread::UI)) {
+      BrowserThread::PostTask(
+          BrowserThread::UI, FROM_HERE,
+          base::Bind(&TextInputClientMacHelper::OnResult,
+                     base::Unretained(this), string, point));
+      return;
+    }
+    word_ = string;
+    point_ = point;
+
+    if (loop_runner_ && loop_runner_->loop_running())
+      loop_runner_->Quit();
+  }
+
+  std::string word_;
+  gfx::Point point_;
+  scoped_refptr<MessageLoopRunner> loop_runner_;
+
+  DISALLOW_COPY_AND_ASSIGN(TextInputClientMacHelper);
+};
+
+}  // namespace
+
+// Site per process browser tests inside content which are specific to Mac OSX
+// platform.
+class SitePerProcessMacBrowserTest : public SitePerProcessBrowserTest {};

[Charlie Reis, 2016/09/23 06:24:10]

I'm probably just missing something from here or the build file, but what's
limiting these tests to running on Mac?  I don't see any ifdefs or a
platform-specific section in content/test/BUILD.gn, but it does look like the
tests run on the Mac bot and not the Linux bot.

[EhsanK, 2016/09/23 14:36:49]

I would, maybe say the .mm extension does the job? Probably this test file is
automatically ignored on other platforms? This seems to be the case with some
other files as well, e.g., "render_view_browsertest_mac.mm".

+
+// This test will load a text only page inside a child frame and then queries
+// the string range which includes the first word. Then it uses the returned
+// point to query the text again and verifies that correct result is returned.
+// Finally, the returned words are compared against the first word in the html
+// file which is "This".
+IN_PROC_BROWSER_TEST_F(SitePerProcessMacBrowserTest,
+                       GetStringFromRangeAndPointChildFrame) {
+  GURL main_url(embedded_test_server()->GetURL(
+      "a.com", "/cross_site_iframe_factory.html?a(b)"));
+  EXPECT_TRUE(NavigateToURL(shell(), main_url));
+  FrameTreeNode* root = web_contents()->GetFrameTree()->root();
+  FrameTreeNode* child = root->child_at(0);
+  NavigateFrameToURL(child,
+                     embedded_test_server()->GetURL("b.com", "/title1.html"));
+
+  RenderWidgetHost* child_widget_host =
+      child->current_frame_host()->GetRenderWidgetHost();
+  TextInputClientMacHelper helper;
+
+  // Get string from range.
+  helper.WaitForStringFromRange(child_widget_host, gfx::Range(0, 4));
+  gfx::Point point = helper.point();
+  std::string word = helper.word();
+
+  // Now get it at a given point.
+  point.SetPoint(point.x(),
+                 child_widget_host->GetView()->GetViewBounds().size().height() -
+                     point.y());
+  helper.WaitForStringAtPoint(child_widget_host, point);
+  EXPECT_EQ(word, helper.word());
+  EXPECT_EQ("This", word);
+}
+
+// This test will load a text only page and then queries the string range which
+// includes the first word. Then it uses the returned point to query the text
+// again and verifies that correct result is returned. Finally, the returned
+// words are compared against the first word in the html file which is "This".
+IN_PROC_BROWSER_TEST_F(SitePerProcessMacBrowserTest,
+                       GetStringFromRangeAndPointMainFrame) {
+  GURL main_url(embedded_test_server()->GetURL("a.com", "/title1.html"));
+  EXPECT_TRUE(NavigateToURL(shell(), main_url));
+  FrameTreeNode* root = web_contents()->GetFrameTree()->root();
+  RenderWidgetHost* widget_host =
+      root->current_frame_host()->GetRenderWidgetHost();
+  TextInputClientMacHelper helper;
+
+  // Get string from range.
+  helper.WaitForStringFromRange(widget_host, gfx::Range(0, 4));
+  gfx::Point point = helper.point();
+  std::string word = helper.word();
+
+  // Now get it at a given point.
+  point.SetPoint(
+      point.x(),
+      widget_host->GetView()->GetViewBounds().size().height() - point.y());
+  helper.WaitForStringAtPoint(widget_host, point);
+  EXPECT_EQ(word, helper.word());
+  EXPECT_EQ("This", word);
+}
+}  // namespace content