Fix for white screen when casting on Android.

chromecast/browser/android/cast_window_android.cc

 // Copyright 2014 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 "chromecast/browser/android/cast_window_android.h"
 
 #include "base/message_loop/message_loop_proxy.h"
 #include "chromecast/browser/android/cast_window_manager.h"
 #include "chromecast/browser/cast_content_window.h"
 #include "content/public/browser/devtools_agent_host.h"
 #include "content/public/browser/navigation_controller.h"
 #include "content/public/browser/navigation_entry.h"
 #include "content/public/browser/render_process_host.h"
 #include "content/public/browser/render_view_host.h"
+#include "content/public/browser/render_widget_host_view.h"
 #include "content/public/common/renderer_preferences.h"
 #include "jni/CastWindowAndroid_jni.h"
+#include "ui/gfx/skia_util.h"
 
 namespace chromecast {
 namespace shell {
 
 namespace {
 
 // The time (in milliseconds) we wait for after a page is closed (i.e.
 // after an app is stopped) before we delete the corresponding WebContents.
 const int kWebContentsDestructionDelayInMs = 50;
 
 }  // namespace
 
 // static
 bool CastWindowAndroid::RegisterJni(JNIEnv* env) {
   return RegisterNativesImpl(env);
 }
 
 // static
 CastWindowAndroid* CastWindowAndroid::CreateNewWindow(
     content::BrowserContext* browser_context,
     const GURL& url) {
   CastWindowAndroid* window_android = new CastWindowAndroid(browser_context);
   window_android->Initialize();
 
   if (!url.is_empty())
     window_android->LoadURL(url);
+
+  content::RenderWidgetHostView* rwhv =
+      window_android->web_contents_->GetRenderWidgetHostView();

[gunsch, 2015/02/24 05:55:09]

optional: instead of accessing web_contents_ (private member from a static
method), you could call web_contents() API (provided by WebContentsDelegate*).

[halliwell, 2015/02/24 06:23:43]

I don't believe there is such an API?  WebContentsDelegate actually maintains an
entire set of WebContents*'s.

Is private member from a static method a problem in the style guide?  If so, we
could add an accessor into CastWindowAndroid.  If not ... I don't personally
have a problem with it, at least not for simple cases like this.

+  if (rwhv) {
+    rwhv->SetBackgroundColor(SK_ColorBLACK);
+  }
+
   return window_android;
 }
 
 CastWindowAndroid::CastWindowAndroid(content::BrowserContext* browser_context)
     : browser_context_(browser_context),
       content_window_(new CastContentWindow),
       weak_factory_(this) {
 }
 
 void CastWindowAndroid::Initialize() {
@@ skipping 89 matching lines @@
   contents->GetRenderViewHost()->Blur();
 }
 
 void CastWindowAndroid::RenderProcessGone(base::TerminationStatus status) {
   LOG(ERROR) << "Render process gone: status=" << status;
   Destroy();
 }
 
 }  // namespace shell
 }  // namespace chromecast