Add GetScrollOffset function to PPB_View

content/renderer/pepper/pepper_plugin_instance_impl.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/pepper/pepper_plugin_instance_impl.h"
 
 #include "base/bind.h"
 #include "base/callback_helpers.h"
 #include "base/command_line.h"
 #include "base/debug/trace_event.h"
@@ skipping 1186 matching lines @@
     new_clip = clip;
 
   cut_outs_rects_ = cut_outs_rects;
 
   view_data_.rect = PP_FromGfxRect(position);
   view_data_.clip_rect = PP_FromGfxRect(clip);
   view_data_.device_scale = container_->deviceScaleFactor();
   view_data_.css_scale =
       container_->pageZoomFactor() * container_->pageScaleFactor();
 
+  // Only set the scroll offset if the plugin is visible.

[dmichael (off chromium), 2014/06/16 22:31:56]

Why have this restriction?

[raymes, 2014/06/17 00:43:00]

Originally I had this because I thought we might end up spamming the plugin with
more messages than usual.

But now that I think about it, it will be getting view change messages as the
value of GetRect changes when the window is scrolled anyway, so there won't be
much difference.

I've removed the check, thanks!

+  if (!view_data_.is_page_visible ||
+      view_data_.clip_rect.size.width == 0 ||
+      view_data_.clip_rect.size.height == 0) {
+    view_data_.scroll_offset = PP_MakePoint(0, 0);
+  } else {
+    gfx::Size scroll_offset =
+        container_->element().document().frame()->scrollOffset();
+    view_data_.scroll_offset = PP_MakePoint(scroll_offset.width(),
+                                            scroll_offset.height());
+  }
+
   if (desired_fullscreen_state_ || view_data_.is_fullscreen) {
     WebElement element = container_->element();
     WebDocument document = element.document();
     bool is_fullscreen_element = (element == document.fullScreenElement());
     if (!view_data_.is_fullscreen && desired_fullscreen_state_ &&
         render_frame()->GetRenderWidget()->is_fullscreen() &&
         is_fullscreen_element) {
       // Entered fullscreen. Only possible via SetFullscreen().
       view_data_.is_fullscreen = true;
     } else if (view_data_.is_fullscreen && !is_fullscreen_element) {
@@ skipping 2016 matching lines @@
     // Running out-of-process. Initiate an IPC call to notify the plugin
     // process.
     ppapi::proxy::HostDispatcher* dispatcher =
         ppapi::proxy::HostDispatcher::GetForInstance(pp_instance());
     dispatcher->Send(new PpapiMsg_PPPInstance_HandleDocumentLoad(
         ppapi::API_ID_PPP_INSTANCE, pp_instance(), pending_host_id, data));
   }
 }
 
 }  // namespace content