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

Unified Diff: chrome/browser/ui/views/page_info_bubble_view.cc

Issue 6622002: Do all OOLing in the views code. linux_views now builds clean with the clang plugin. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 9 years, 10 months 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/ui/views/page_info_bubble_view.cc
diff --git a/chrome/browser/ui/views/page_info_bubble_view.cc b/chrome/browser/ui/views/page_info_bubble_view.cc
index cd3fcc3688e72549ecacd1434f704d3573e3ba18..b112a000bb6e9e8b2ce6426926f24082c381afca 100644
--- a/chrome/browser/ui/views/page_info_bubble_view.cc
+++ b/chrome/browser/ui/views/page_info_bubble_view.cc
@@ -207,6 +207,18 @@ void PageInfoBubbleView::ModelChanged() {
resize_animation_.Show();
}
+bool PageInfoBubbleView::CloseOnEscape() {
+ return true;
+}
+
+bool PageInfoBubbleView::FadeInOnShow() {
+ return false;
+}
+
+std::wstring PageInfoBubbleView::accessible_name() {
+ return L"PageInfoBubble";
+}
+
void PageInfoBubbleView::LinkActivated(views::Link* source, int event_flags) {
// We want to make sure the info bubble closes once the link is activated. So
// we close it explicitly rather than relying on a side-effect of opening a

Powered by Google App Engine
This is Rietveld 408576698