Re-enable native UI for {frame:'chrome'} in app windows.

chrome/browser/ui/extensions/shell_window.cc

 // Copyright (c) 2012 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/extensions/shell_window.h"
 
 #include "base/utf_string_conversions.h"
 #include "chrome/browser/extensions/extension_process_manager.h"
 #include "chrome/browser/extensions/extension_system.h"
 #include "chrome/browser/extensions/shell_window_geometry_cache.h"
@@ skipping 47 matching lines @@
   DCHECK(rvh);
   BrowserThread::PostTask(BrowserThread::IO, FROM_HERE,
       base::Bind(&ResourceDispatcherHost::BlockRequestsForRoute,
           base::Unretained(ResourceDispatcherHost::Get()),
           rvh->GetProcess()->GetID(), rvh->GetRoutingID()));
 }
 
 }  // namespace
 
 ShellWindow::CreateParams::CreateParams()
-  : frame(ShellWindow::CreateParams::FRAME_NONE),
+  : frame(ShellWindow::CreateParams::FRAME_CHROME),
     bounds(-1, -1, kDefaultWidth, kDefaultHeight),
     restore_position(true), restore_size(true) {
 }
 
 ShellWindow::CreateParams::~CreateParams() {
 }
 
 ShellWindow* ShellWindow::Create(Profile* profile,
                                  const extensions::Extension* extension,
                                  const GURL& url,
@@ skipping 21 matching lines @@
       NULL);
   contents_.reset(TabContents::Factory::CreateTabContents(web_contents_));
   content::WebContentsObserver::Observe(web_contents_);
   web_contents_->SetDelegate(this);
   chrome::SetViewType(web_contents_, chrome::VIEW_TYPE_APP_SHELL);
   web_contents_->GetMutableRendererPrefs()->
       browser_handles_all_top_level_requests = true;
   web_contents_->GetRenderViewHost()->SyncRendererPrefs();
 
   native_window_.reset(NativeShellWindow::Create(this, params));
-  // Interpretation of the bounds passed to NativeShellWindow::Create varies
-  // between the different implementations, SetBounds behaves more consistent
-  // so call that one here too. A fix for http://crbug.com/130184 should make
-  // this no longer needed.
-  if (params.bounds.x() >= 0 && params.bounds.y() >= 0) {
-    native_window_->SetBounds(params.bounds);
-  }
 
   if (!params.window_key.empty()) {
     window_key_ = params.window_key;
 
     if (params.restore_position || params.restore_size) {
       extensions::ShellWindowGeometryCache* cache =
           extensions::ExtensionSystem::Get(profile())->
             shell_window_geometry_cache();
       gfx::Rect cached_bounds;
       if (cache->GetGeometry(extension()->id(), params.window_key,
@@ skipping 327 matching lines @@
   if (window_key_.empty())
     return;
 
   extensions::ShellWindowGeometryCache* cache =
       extensions::ExtensionSystem::Get(profile())->
           shell_window_geometry_cache();
 
   gfx::Rect bounds = native_window_->GetBounds();
   cache->SaveGeometry(extension()->id(), window_key_, bounds);
 }