[MacViews] Implement colored window frames.

chrome/browser/ui/views/apps/app_window_native_widget_mac.h

Index: chrome/browser/ui/views/apps/app_window_native_widget_mac.h
diff --git a/chrome/browser/ui/views/apps/app_window_native_widget_mac.h b/chrome/browser/ui/views/apps/app_window_native_widget_mac.h
index ae7cfc2c4eb5d8833abddace515c30a976dac671..26a796b87428af3d92fae80dc48711b094a46f20 100644
--- a/chrome/browser/ui/views/apps/app_window_native_widget_mac.h
+++ b/chrome/browser/ui/views/apps/app_window_native_widget_mac.h
@@ -7,11 +7,16 @@
 
 #include "ui/views/widget/native_widget_mac.h"
 
+namespace extensions {
+class NativeAppWindow;
+}
+
 // This implements features specific to app windows, e.g. frameless windows that
 // behave like normal windows.
 class AppWindowNativeWidgetMac : public views::NativeWidgetMac {
  public:
-  explicit AppWindowNativeWidgetMac(views::Widget* widget);
+  AppWindowNativeWidgetMac(views::Widget* widget,
+                           extensions::NativeAppWindow* native_app_window);
   ~AppWindowNativeWidgetMac() override;
 
  protected:
@@ -20,6 +25,9 @@ class AppWindowNativeWidgetMac : public views::NativeWidgetMac {
       const views::Widget::InitParams& params) override;
 
  private:
+  // Weak. NativeAppWindowViews owns a Widget.

[tapted, 2015/05/14 04:23:50]

The Widget in NativeAppWindowViews is also weak.

I think technically the Widget owner is either "itself" or the WebContents in
AppWindowContentsImpl, since extensions::AppWindow is its WebContentsDelegate,
and it guarantees to call native_app_window_->Close() along with the
WebContents.

So I'd say "Weak. Owned by extensions::AppWindow (which manages our Widget via
its WebContents)."

[jackhou1, 2015/05/14 06:20:40]

Done.

+  extensions::NativeAppWindow* native_app_window_;
+
   DISALLOW_COPY_AND_ASSIGN(AppWindowNativeWidgetMac);
 };