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

Side by Side Diff: chrome/browser/ui/views/location_bar/location_bar_bubble_delegate_view.h

Issue 2905243003: Cleanup BookmarkBubbleView, remove LocationBarBubbleDelegateView::GetDialogButtons() (Closed)
Patch Set: Rebase for r475249 Created 3 years, 6 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 unified diff | Download patch
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 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 #ifndef CHROME_BROWSER_UI_VIEWS_LOCATION_BAR_LOCATION_BAR_BUBBLE_DELEGATE_VIEW_H _ 5 #ifndef CHROME_BROWSER_UI_VIEWS_LOCATION_BAR_LOCATION_BAR_BUBBLE_DELEGATE_VIEW_H _
6 #define CHROME_BROWSER_UI_VIEWS_LOCATION_BAR_LOCATION_BAR_BUBBLE_DELEGATE_VIEW_H _ 6 #define CHROME_BROWSER_UI_VIEWS_LOCATION_BAR_LOCATION_BAR_BUBBLE_DELEGATE_VIEW_H _
7 7
8 #include "base/macros.h" 8 #include "base/macros.h"
9 #include "content/public/browser/notification_observer.h" 9 #include "content/public/browser/notification_observer.h"
10 #include "content/public/browser/notification_registrar.h" 10 #include "content/public/browser/notification_registrar.h"
(...skipping 32 matching lines...) Expand 10 before | Expand all | Expand 10 after
43 content::WebContents* web_contents); 43 content::WebContents* web_contents);
44 44
45 // TODO(varkha): Delete this override and use the constructor above. 45 // TODO(varkha): Delete this override and use the constructor above.
46 LocationBarBubbleDelegateView(views::View* anchor_view, 46 LocationBarBubbleDelegateView(views::View* anchor_view,
47 content::WebContents* web_contents); 47 content::WebContents* web_contents);
48 ~LocationBarBubbleDelegateView() override; 48 ~LocationBarBubbleDelegateView() override;
49 49
50 // Displays the bubble with appearance and behavior tailored for |reason|. 50 // Displays the bubble with appearance and behavior tailored for |reason|.
51 void ShowForReason(DisplayReason reason); 51 void ShowForReason(DisplayReason reason);
52 52
53 // views::BubbleDialogDelegateView:
54 int GetDialogButtons() const override;
55
56 // content::NotificationObserver: 53 // content::NotificationObserver:
57 void Observe(int type, 54 void Observe(int type,
58 const content::NotificationSource& source, 55 const content::NotificationSource& source,
59 const content::NotificationDetails& details) override; 56 const content::NotificationDetails& details) override;
60 57
61 protected: 58 protected:
62 // The class listens for WebContentsView events and closes the bubble. Useful 59 // The class listens for WebContentsView events and closes the bubble. Useful
63 // for bubbles that do not start out focused but need to close when the user 60 // for bubbles that do not start out focused but need to close when the user
64 // interacts with the web view. 61 // interacts with the web view.
65 class WebContentMouseHandler : public ui::EventHandler { 62 class WebContentMouseHandler : public ui::EventHandler {
(...skipping 24 matching lines...) Expand all
90 void AdjustForFullscreen(const gfx::Rect& screen_bounds); 87 void AdjustForFullscreen(const gfx::Rect& screen_bounds);
91 88
92 private: 89 private:
93 // Used to register for fullscreen change notifications. 90 // Used to register for fullscreen change notifications.
94 content::NotificationRegistrar registrar_; 91 content::NotificationRegistrar registrar_;
95 92
96 DISALLOW_COPY_AND_ASSIGN(LocationBarBubbleDelegateView); 93 DISALLOW_COPY_AND_ASSIGN(LocationBarBubbleDelegateView);
97 }; 94 };
98 95
99 #endif // CHROME_BROWSER_UI_VIEWS_LOCATION_BAR_LOCATION_BAR_BUBBLE_DELEGATE_VIE W_H_ 96 #endif // CHROME_BROWSER_UI_VIEWS_LOCATION_BAR_LOCATION_BAR_BUBBLE_DELEGATE_VIE W_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698