Remove title and icon from chrome apps native style title bars.

chrome/browser/ui/views/apps/app_window_desktop_window_tree_host_win.cc

Index: chrome/browser/ui/views/apps/app_window_desktop_window_tree_host_win.cc
diff --git a/chrome/browser/ui/views/apps/app_window_desktop_window_tree_host_win.cc b/chrome/browser/ui/views/apps/app_window_desktop_window_tree_host_win.cc
new file mode 100644
index 0000000000000000000000000000000000000000..7f31b2145930f81f6990c87107f3c7ee236b3b3d
--- /dev/null
+++ b/chrome/browser/ui/views/apps/app_window_desktop_window_tree_host_win.cc
@@ -0,0 +1,78 @@
+// 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 "chrome/browser/ui/views/apps/app_window_desktop_window_tree_host_win.h"
+
+#include <dwmapi.h>
+
+#include "base/debug/stack_trace.h"

[tapted, 2014/04/03 19:40:24]

stray debugging header

[benwells, 2014/04/04 06:50:43]

So there is .. gone

+
+#include "chrome/browser/ui/views/apps/chrome_native_app_window_views_win.h"
+#include "chrome/browser/ui/views/apps/glass_app_window_frame_view.h"
+#include "grit/theme_resources.h"
+#include "ui/base/theme_provider.h"
+#include "ui/gfx/win/dpi.h"
+#include "ui/views/controls/menu/native_menu_win.h"
+
+#pragma comment(lib, "dwmapi.lib")
+
+AppWindowDesktopWindowTreeHostWin::AppWindowDesktopWindowTreeHostWin(
+    ChromeNativeAppWindowViewsWin* app_window,
+    views::DesktopNativeWidgetAura* desktop_native_widget_aura)
+    : DesktopWindowTreeHostWin(app_window->widget(),
+                               desktop_native_widget_aura),
+      app_window_(app_window) {}
+
+AppWindowDesktopWindowTreeHostWin::~AppWindowDesktopWindowTreeHostWin() {}
+
+bool AppWindowDesktopWindowTreeHostWin::GetClientAreaInsets(
+    gfx::Insets* insets) const {
+  // Use the default client insets for an opaque frame or a glass popup/app
+  // frame.
+  if (!app_window_->glass_frame_view())
+    return false;
+
+  // This tells Windows that the whole of the window is a client area, meaning
+  // Chrome will draw it. Windows still fills in the glass bits because of the
+  // DwmExtendFrameIntoClientArea call in |UpdateDWMFrame|.
+  insets->Set(0, 0, 0, 0);
+  return true;
+}
+
+void AppWindowDesktopWindowTreeHostWin::HandleFrameChanged() {
+  // We need to update the glass region on or off before the base class adjusts
+  // the window region.
+  UpdateDWMFrame();
+  DesktopWindowTreeHostWin::HandleFrameChanged();
+}
+
+void AppWindowDesktopWindowTreeHostWin::PostHandleMSG(UINT message,
+                                                      WPARAM w_param,
+                                                      LPARAM l_param) {
+  switch (message) {
+    case WM_WINDOWPOSCHANGED: {
+      UpdateDWMFrame();
+      break;
+    }
+  }

[sky, 2014/04/03 19:43:56]

I'm surprised you don't need a default block here.

[benwells, 2014/04/04 06:50:43]

I think it is good. The function in DesktopWindowTreeHostWin this overrides is
empty so normally this class does nothing. Also see
BrowserDesktopWindowTreeHostWin's implementation which doesn't have a default.

BTW I considered making this an if but a switch seemed slightly better for this
type of function.

+}
+
+void AppWindowDesktopWindowTreeHostWin::UpdateDWMFrame() {
+  if (!GetWidget()->client_view() || !app_window_->glass_frame_view())
+    return;
+
+  MARGINS margins = {0};
+
+  // If the opaque frame is visible, we use the default (zero) margins.
+  // Otherwise, we need to figure out how to extend the glass in.
+  if (app_window_->glass_frame_view()) {
+    gfx::Insets insets = app_window_->glass_frame_view()->GetGlassInsets();
+    margins.cxLeftWidth = insets.left();
+    margins.cxRightWidth = insets.right();
+    margins.cyBottomHeight = insets.bottom();
+    margins.cyTopHeight = insets.top();
+  }
+
+  DwmExtendFrameIntoClientArea(GetHWND(), &margins);
+}