Chromecast Android shell improvements

chromecast/shell/browser/android/cast_window_android.h

Index: chromecast/shell/browser/android/cast_window_android.h
diff --git a/chromecast/shell/browser/android/cast_window_android.h b/chromecast/shell/browser/android/cast_window_android.h
index 979aea962820b0149e2adbef8ebeb343cfc460de..cafb068baf3a4f5152af826f251f8087cefadba6 100644
--- a/chromecast/shell/browser/android/cast_window_android.h
+++ b/chromecast/shell/browser/android/cast_window_android.h
@@ -50,7 +50,7 @@ class CastWindowAndroid : public content::WebContentsDelegate,
   // Registers the JNI methods for CastWindowAndroid.
   static bool RegisterJni(JNIEnv* env);
 
-  // WebContentsDelegate implementation.
+  // content::WebContentsDelegate

[lcwu1, 2014/09/16 22:46:50]

I thought our convention is to use "content::WebContentsDelegate:". (Sorry that
I missed this when reviewing the internal CL.)

[gunsch, 2014/09/16 23:02:36]

Done.

[lcwu1, 2014/09/16 23:26:06]

Sorry again. :-) I meant to say "content::WebContentsDelegate implementation:".
I missed the all important 'implementation'.

   virtual void AddNewContents(content::WebContents* source,
                               content::WebContents* new_contents,
                               WindowOpenDisposition disposition,
@@ -67,6 +67,9 @@ class CastWindowAndroid : public content::WebContentsDelegate,
   virtual void ActivateContents(content::WebContents* contents) OVERRIDE;
   virtual void DeactivateContents(content::WebContents* contents) OVERRIDE;
 
+  // content::WebContentsObserver

[lcwu1, 2014/09/16 22:46:50]

Ditto.

[gunsch, 2014/09/16 23:02:36]

Done.

+  virtual void RenderProcessGone(base::TerminationStatus status) OVERRIDE;
+
  private:
   explicit CastWindowAndroid(content::WebContents* web_contents);