[MacViews] Frameless app windows: make content view cover title bar.

chrome/browser/ui/views/apps/chrome_native_app_window_views_mac.mm

 // Copyright 2015 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.
 
 #import "chrome/browser/ui/views/apps/chrome_native_app_window_views_mac.h"
 
 #include "chrome/browser/apps/app_shim/extension_app_shim_handler_mac.h"
+#import "chrome/browser/ui/views/apps/app_window_native_widget_mac.h"

[tapted, 2015/03/18 00:06:39]

nit: import->include

[jackhou1, 2015/03/18 03:44:39]

Done.

 
 ChromeNativeAppWindowViewsMac::ChromeNativeAppWindowViewsMac()
     : is_hidden_with_app_(false) {
 }
 
 ChromeNativeAppWindowViewsMac::~ChromeNativeAppWindowViewsMac() {
 }
 
+void ChromeNativeAppWindowViewsMac::OnBeforeWidgetInit(
+    const extensions::AppWindow::CreateParams& create_params,
+    views::Widget::InitParams* init_params,
+    views::Widget* widget) {
+  init_params->native_widget = new AppWindowNativeWidgetMac(widget);

[tapted, 2015/03/18 00:06:39]

DCHECK(!init_params->native_widget);

i.e. ensure ChromeViewsDelegate hasn't already put an object on this that will
leak.

[jackhou1, 2015/03/18 03:44:39]

Done.

+  ChromeNativeAppWindowViews::OnBeforeWidgetInit(create_params, init_params,
+                                                 widget);
+}
+
 void ChromeNativeAppWindowViewsMac::Show() {
   if (is_hidden_with_app_) {
     // If there is a shim to gently request attention, return here. Otherwise
     // show the window as usual.
     if (apps::ExtensionAppShimHandler::ActivateAndRequestUserAttentionForWindow(
             app_window())) {
       return;
     }
   }
 
@@ skipping 10 matching lines @@
 void ChromeNativeAppWindowViewsMac::ShowWithApp() {
   is_hidden_with_app_ = false;
   if (!app_window()->is_hidden())
     ShowInactive();
 }
 
 void ChromeNativeAppWindowViewsMac::HideWithApp() {
   is_hidden_with_app_ = true;
   ChromeNativeAppWindowViews::Hide();
 }