Always use opaque frame for extension app windows

chrome/browser/views/frame/browser_frame_win.cc

 // Copyright (c) 2010 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/views/frame/browser_frame_win.h"
 
 #include <dwmapi.h>
 #include <shellapi.h>
 
 #include <set>
@@ skipping 93 matching lines @@
   // This is implemented for a different interface than GetThemeProvider is,
   // but they mean the same things.
   return GetThemeProvider();
 }
 
 bool BrowserFrameWin::AlwaysUseNativeFrame() const {
   // App panel windows draw their own frame.
   if (browser_view_->IsBrowserTypePanel())
     return false;
 
+  if (browser_view_->browser()->type() == Browser::TYPE_EXTENSION_APP)

[Peter Kasting, 2010/04/28 04:45:17]

Nit: Add a comment about this (and, if applicable, when/what bug is about
removing this).

+    return false;
+
   // We don't theme popup or app windows, so regardless of whether or not a
   // theme is active for normal browser windows, we don't want to use the custom
   // frame for popups/apps.
   if (!browser_view_->IsBrowserTypeNormal() && win_util::ShouldUseVistaFrame())
     return true;
 
   // Otherwise, we use the native frame when we're told we should by the theme
   // provider (e.g. no custom theme is active).
   return GetThemeProvider()->ShouldUseNativeFrame();
 }
@@ skipping 193 matching lines @@
 
   DWORD window_style = GetWindowLong(GWL_STYLE);
   if (browser_view_->UsingSideTabs()) {
     if (window_style & WS_CAPTION)
       SetWindowLong(GWL_STYLE, window_style & ~WS_CAPTION);
   } else {
     if (!(window_style & WS_CAPTION))
       SetWindowLong(GWL_STYLE, window_style | WS_CAPTION);
   }
 }