Fixing the focus issue in open/save dialogs.

chrome/browser/ui/views/extensions/extension_dialog.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/views/extensions/extension_dialog.h"
 
 #include "chrome/browser/extensions/extension_host.h"
 #include "chrome/browser/extensions/extension_process_manager.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/ui/browser.h"
@@ skipping 173 matching lines @@
 
 void ExtensionDialog::MaybeFocusRenderView() {
   views::FocusManager* focus_manager = GetWidget()->GetFocusManager();
   // Already there's a focused view, so no need to switch the focus.
   if (focus_manager && focus_manager->GetFocusedView())
     return;
 
   content::RenderWidgetHostView* view = host()->render_view_host()->GetView();
   if (!view)
     return;
 

[oshima, 2012/06/22 14:34:41]

can you please add
// TODO(oshima): Views + aura doesn't seem to update the views focus
// manager when an aura window gets focus. This is a workaround for
// this issue. Fix it this and remove this workaround.
// See bug.com/127222.

[SeRya, 2012/06/22 14:45:27]

Done.

+  focus_manager->SetFocusedView(GetContentsView());
   view->Focus();
 }
 
 /////////////////////////////////////////////////////////////////////////////
 // views::WidgetDelegate overrides.
 
 bool ExtensionDialog::CanResize() const {
   return false;
 }
 
@@ skipping 57 matching lines @@
       if (content::Details<ExtensionHost>(host()) != details)
         return;
       if (observer_)
         observer_->ExtensionTerminated(this);
       break;
     default:
       NOTREACHED() << L"Received unexpected notification";
       break;
   }
 }