Delegate root layer scroll offset to android_webview.

content/renderer/android/synchronous_compositor_impl.cc

Index: content/renderer/android/synchronous_compositor_impl.cc
diff --git a/content/renderer/android/synchronous_compositor_impl.cc b/content/renderer/android/synchronous_compositor_impl.cc
index d871a0cd4bdd31d0d63bcce956397fec545922a8..27e91f9eddce1e3b1fb8fad59fff2007512a14ae 100644
--- a/content/renderer/android/synchronous_compositor_impl.cc
+++ b/content/renderer/android/synchronous_compositor_impl.cc
@@ -72,6 +72,16 @@ void SynchronousCompositorImpl::DidDestroySynchronousOutputSurface() {
   compositor_client_->DidDestroyCompositor(this);
 }
 
+void SynchronousCompositorImpl::SetTotalScrollOffset(gfx::Vector2dF new_value) {
+  DCHECK(CalledOnValidThread());
+  compositor_client_->SetTotalRootLayerScrollOffset(new_value);

[mkosiba (inactive), 2013/05/28 17:29:50]

actually it looks like this ends up causing a use-after-free problem if the
WebContents is being deleted (in which case the android_webview layer un-sets
the compositor_client_ *before* the SynchronousCompositorImpl gets destroyed).

Ideally we'd tie the existence of a RootLayerScrollOffsetDelegate with the
existence of the compositor_client_ (set the RLSCOD_ to NULL if the
compositor_client_ is NULL) but for that I'd need to change the code Jared is
currently adding as part of https://codereview.chromium.org/15920002/ to use the
SynchronousCompositorImpl.
Thoughts?

[joth, 2013/05/28 17:36:25]

Hmm.. I think the bug is in your prior refactor patch. The current ToT code
always does a null-check on compositor_client_ before calling through it. Can we
just use that same policy in the new synchronous_compositor_impl.cc arrangement?

https://code.google.com/p/chromium/codesearch#chromium/src/content/renderer/a...

[mkosiba (inactive), 2013/05/29 13:18:24]

ah, I see. Yes, I can do that. Do you know if the compositor_client_ will be
null in cases other than setup/teardown?

+}
+
+gfx::Vector2dF SynchronousCompositorImpl::GetTotalScrollOffset() {
+  DCHECK(CalledOnValidThread());
+  return compositor_client_->GetTotalRootLayerScrollOffset();
+}
+
 // Not using base::NonThreadSafe as we want to enforce a more exacting threading
 // requirement: SynchronousCompositorImpl() must only be used by
 // embedders that supply their own compositor loop via