Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(166)

Side by Side Diff: chrome/browser/views/location_bar_view.cc

Issue 459005: Make sure we don't dismiss extension popups when the focus... (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: Created 11 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2009 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2009 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/views/location_bar_view.h" 5 #include "chrome/browser/views/location_bar_view.h"
6 6
7 #if defined(OS_LINUX) 7 #if defined(OS_LINUX)
8 #include <gtk/gtk.h> 8 #include <gtk/gtk.h>
9 #endif 9 #endif
10 10
(...skipping 1445 matching lines...) Expand 10 before | Expand all | Expand 10 after
1456 } 1456 }
1457 SetVisible(visible); 1457 SetVisible(visible);
1458 } 1458 }
1459 1459
1460 void LocationBarView::PageActionImageView::BubbleBrowserWindowClosing( 1460 void LocationBarView::PageActionImageView::BubbleBrowserWindowClosing(
1461 BrowserBubble* bubble) { 1461 BrowserBubble* bubble) {
1462 HidePopup(); 1462 HidePopup();
1463 } 1463 }
1464 1464
1465 void LocationBarView::PageActionImageView::BubbleLostFocus( 1465 void LocationBarView::PageActionImageView::BubbleLostFocus(
1466 BrowserBubble* bubble) { 1466 BrowserBubble* bubble,
1467 gfx::NativeView focused_view) {
1467 if (!popup_) 1468 if (!popup_)
1468 return; 1469 return;
1469 1470
1470 MessageLoop::current()->PostTask(FROM_HERE, 1471 MessageLoop::current()->PostTask(FROM_HERE,
1471 method_factory_.NewRunnableMethod( 1472 method_factory_.NewRunnableMethod(
1472 &LocationBarView::PageActionImageView::HidePopup)); 1473 &LocationBarView::PageActionImageView::HidePopup));
1473 } 1474 }
1474 1475
1475 void LocationBarView::PageActionImageView::HidePopup() { 1476 void LocationBarView::PageActionImageView::HidePopup() {
1476 if (!popup_) 1477 if (!popup_)
(...skipping 121 matching lines...) Expand 10 before | Expand all | Expand 10 after
1598 const int button = 1; // Left mouse button. 1599 const int button = 1; // Left mouse button.
1599 page_action_views_[i]->image_view()->ExecuteAction(button); 1600 page_action_views_[i]->image_view()->ExecuteAction(button);
1600 return; 1601 return;
1601 } 1602 }
1602 ++current; 1603 ++current;
1603 } 1604 }
1604 } 1605 }
1605 } 1606 }
1606 NOTREACHED(); 1607 NOTREACHED();
1607 } 1608 }
OLDNEW
« chrome/browser/views/browser_actions_container.cc ('K') | « chrome/browser/views/location_bar_view.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698