Add GetScrollOffset function to PPB_View

This adds a function to PPB_View which allows plugins to know the scroll offset
of the page when they are in view. This is useful for OOP PDF which uses the
scroll offset of the window it is contained in to determine the document's
scroll location. A web page can send scroll location via postMessage but
this is slow. Sending the offset directly via view messages is much faster
and seems reasonable.

We don't send the scroll offset in the cases where the plugin is off screen
to avoid any more additional IPC traffic than what currently exists.

BUG=303491

Review URL: https://codereview.chromium.org/329033003

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@278236 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 27 insertions, 0 deletions

diff --git a/ppapi/tests/test_view.cc b/ppapi/tests/test_view.cc
index d141c05..5ce903a 100644
--- a/ppapi/tests/test_view.cc
+++ b/ppapi/tests/test_view.cc
@@ -46,6 +46,7 @@ void TestView::RunTests(const std::string& filter) {
   RUN_TEST(PageHideShow, filter);
   RUN_TEST(SizeChange, filter);
   RUN_TEST(ClipChange, filter);
+  RUN_TEST(ScrollOffsetChange, filter);
 }
 
 bool TestView::WaitUntilViewChanged() {
@@ -199,3 +200,29 @@ std::string TestView::TestClipChange() {
   ASSERT_TRUE(last_view_.GetClipRect() == desired_clip);
   PASS();
 }
+
+std::string TestView::TestScrollOffsetChange() {
+  instance_->EvalScript("document.body.style.width = '5000px';"
+                        "document.body.style.height = '5000px';");
+  instance_->EvalScript("window.scrollTo(5, 1);");
+
+  PP_Time begin_time = pp::Module::Get()->core()->GetTime();
+  while (WaitUntilViewChanged() &&
+         last_view_.GetScrollOffset() != pp::Point(5, 1) &&
+         pp::Module::Get()->core()->GetTime() - begin_time <
+             kViewChangeTimeoutSec) {
+  }
+  ASSERT_EQ(pp::Point(5, 1), last_view_.GetScrollOffset());
+
+  instance_->EvalScript("window.scrollTo(0, 0);");
+
+  begin_time = pp::Module::Get()->core()->GetTime();
+  while (WaitUntilViewChanged() &&
+         last_view_.GetScrollOffset() != pp::Point(0, 0) &&
+         pp::Module::Get()->core()->GetTime() - begin_time <
+             kViewChangeTimeoutSec) {
+  }
+  ASSERT_EQ(pp::Point(0, 0), last_view_.GetScrollOffset());
+
+  PASS();
+}