Sets the default background color of inline signin and user manager to grey

content/public/browser/render_widget_host_view.h

Index: content/public/browser/render_widget_host_view.h
diff --git a/content/public/browser/render_widget_host_view.h b/content/public/browser/render_widget_host_view.h
index 9fe797c755f1d0cc2002b0fdd87f99c3e636efc3..6dca69f839f1f334cd7460d08e5927b08e5c7bd3 100644
--- a/content/public/browser/render_widget_host_view.h
+++ b/content/public/browser/render_widget_host_view.h
@@ -10,6 +10,7 @@
 #include "base/strings/string16.h"
 #include "content/common/content_export.h"
 #include "third_party/skia/include/core/SkBitmap.h"
+#include "third_party/skia/include/core/SkColor.h"
 #include "third_party/skia/include/core/SkRegion.h"
 #include "third_party/WebKit/public/web/WebInputEvent.h"
 #include "ui/gfx/native_widget_types.h"
@@ -109,6 +110,9 @@ class CONTENT_EXPORT RenderWidgetHostView {
   virtual void SetBackgroundOpaque(bool opaque) = 0;
   virtual bool GetBackgroundOpaque() = 0;
 
+  // Subclasses should override this method to set the initial background color.
+  virtual void SetBackgroundColor(SkColor color) = 0;

[sky, 2014/10/07 22:03:59]

Can SetBackgroundOpaque be removed if you add this?

[guohui, 2014/10/08 18:11:51]

Theoretically yes, you could replace SetBackgroundOpaque(false) with
SetBackgroundColor(0x00). However i prefer to keep both since all existing
callers of SetBakcgroundOpaque do not care about the color, also maybe in the
future we will support colored transparent background.

[guohui, 2014/10/08 18:22:33]

eh i forgot SkColor has an alpha component, so it could represent a transparent
color, then i do agree that it is cleaner to have only one method to set the
transparency. I ll fix it in a new patch.

Thanks!

+
   // Return value indicates whether the mouse is locked successfully or not.
   virtual bool LockMouse() = 0;
   virtual void UnlockMouse() = 0;