Set an initial background for RenderFrameHosts during commit.

content/browser/web_contents/web_contents_impl.cc

Index: content/browser/web_contents/web_contents_impl.cc
diff --git a/content/browser/web_contents/web_contents_impl.cc b/content/browser/web_contents/web_contents_impl.cc
index 6749aafe832138b29352a637eb1107f508b56817..b4046682eace1da751df6efae1ba3daa9f9c14cc 100644
--- a/content/browser/web_contents/web_contents_impl.cc
+++ b/content/browser/web_contents/web_contents_impl.cc
@@ -4082,20 +4082,23 @@ void WebContentsImpl::NotifyViewSwapped(RenderViewHost* old_host,
 
 void WebContentsImpl::NotifyFrameSwapped(RenderFrameHost* old_host,
                                          RenderFrameHost* new_host) {
-  // Copies the background color from an old WebContents to a new one that
-  // replaces it on the screen. This allows the new WebContents to use the
+  for (auto& observer : observers_)
+    observer.RenderFrameHostChanged(old_host, new_host);
+}
+
+void WebContentsImpl::NotifyInitialBackground(SkColor old_background,
+                                              RenderFrameHost* new_host) {
+  // This code copies the background color from an old WebContents to a new one
+  // that replaces it on the screen. This allows the new WebContents to use the
   // old one's background color as the starting background color, before having
   // loaded any contents. As a result, we avoid flashing white when navigating
   // from a site whith a dark background to another site with a dark background.
-  if (old_host && new_host) {
-    RenderWidgetHostView* old_view = old_host->GetView();
+  if (new_host) {
     RenderWidgetHostView* new_view = new_host->GetView();
-    if (old_view && new_view)
-      new_view->SetBackgroundColor(old_view->background_color());
+    if (new_view) {

[nasko, 2017/03/08 22:54:20]

nit: No need for {} on one line if statements.

+      new_view->SetBackgroundColor(old_background);
+    }
   }
-
-  for (auto& observer : observers_)
-    observer.RenderFrameHostChanged(old_host, new_host);
 }
 
 // TODO(avi): Remove this entire function because this notification is already