Hooking up setBackgroundColor from AwContents to render process

android_webview/browser/renderer_host/aw_render_view_host_ext.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 "android_webview/browser/renderer_host/aw_render_view_host_ext.h"
 
 #include "android_webview/browser/aw_browser_context.h"
 #include "android_webview/browser/scoped_allow_wait_for_legacy_web_view_api.h"
 #include "android_webview/common/aw_switches.h"
 #include "android_webview/common/render_view_messages.h"
 #include "base/android/scoped_java_ref.h"
 #include "base/callback.h"
 #include "base/command_line.h"
 #include "base/logging.h"
 #include "content/public/browser/android/content_view_core.h"
 #include "content/public/browser/render_process_host.h"
 #include "content/public/browser/render_view_host.h"
 #include "content/public/browser/user_metrics.h"
 #include "content/public/browser/web_contents.h"
 #include "content/public/common/frame_navigate_params.h"
 
 namespace android_webview {
 
 AwRenderViewHostExt::AwRenderViewHostExt(
     AwRenderViewHostExtClient* client, content::WebContents* contents)
     : content::WebContentsObserver(contents),
       client_(client),
+      background_color_(SK_ColorWHITE),
       has_new_hit_test_data_(false) {
   DCHECK(client_);
 }
 
 AwRenderViewHostExt::~AwRenderViewHostExt() {}
 
 void AwRenderViewHostExt::DocumentHasImages(DocumentHasImagesResult result) {
   DCHECK(CalledOnValidThread());
   if (!web_contents()->GetRenderViewHost()) {
     result.Run(false);
@@ skipping 40 matching lines @@
   DCHECK(CalledOnValidThread());
   Send(new AwViewMsg_ResetScrollAndScaleState(web_contents()->GetRoutingID()));
 }
 
 void AwRenderViewHostExt::SetInitialPageScale(double page_scale_factor) {
   DCHECK(CalledOnValidThread());
   Send(new AwViewMsg_SetInitialPageScale(web_contents()->GetRoutingID(),
                                          page_scale_factor));
 }
 
+void AwRenderViewHostExt::SetBackgroundColor(SkColor c) {

[mkosiba (inactive), 2013/07/23 11:14:40]

why not put it in AwSettings? There's already logic in there to sync the setting
on RenderViewCreated and this code seems to be replicating that.

[joth, 2013/07/23 17:38:59]

I feel AwSettings is the upstream (native) implementation of WebSettings. It's a
peer of AwContents - both handle the JNI boundary.
Internally AwSettings delegates IPC to AwRenderViewHostExt anyway, so delegating
here to AwSettings is just putting another player in the path for no benefit
AFAICT.
Also, the 'sync the settings' code in AwSettings is heavily focused on 'pulling'
values from java to native in a thread safe way. background color doesn't need
any of that (it's pushed from java, and single threaded), so I think it would be
confusing matters to route it via that path.

It an interesting point that AwSettings uses RenderViewCreated rather than
RenderViewReady. RVR seems to work, but the docs say "This method is invoked
when the RenderView of the current RenderViewHost is ready, e.g. because we
recreated it after a crash." which is kind of ambiguous. I think I'll switch to
use RVC to be consistent with AwSettings.

+  if (background_color_ == c)
+    return;
+  background_color_ = c;
+  Send(new AwViewMsg_SetBackgroundColor(web_contents()->GetRoutingID(),
+                                        background_color_));
+}
+
+void AwRenderViewHostExt::RenderViewReady() {
+  Send(new AwViewMsg_SetBackgroundColor(web_contents()->GetRoutingID(),
+                                        background_color_));
+}
+
 void AwRenderViewHostExt::RenderProcessGone(base::TerminationStatus status) {
   DCHECK(CalledOnValidThread());
   for (std::map<int, DocumentHasImagesResult>::iterator pending_req =
            pending_document_has_images_requests_.begin();
        pending_req != pending_document_has_images_requests_.end();
       ++pending_req) {
     pending_req->second.Run(false);
   }
 }
 
@@ skipping 39 matching lines @@
   DCHECK(CalledOnValidThread());
   last_hit_test_data_ = hit_test_data;
   has_new_hit_test_data_ = true;
 }
 
 void AwRenderViewHostExt::OnPageScaleFactorChanged(float page_scale_factor) {
   client_->OnWebLayoutPageScaleFactorChanged(page_scale_factor);
 }
 
 }  // namespace android_webview