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

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

Issue 1947533003: Second try at fixing location bar icon keyboard activation. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 7 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 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 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_BUBBLE_ICON_VIEW_H_ 5 #ifndef CHROME_BROWSER_UI_VIEWS_LOCATION_BAR_BUBBLE_ICON_VIEW_H_
6 #define CHROME_BROWSER_UI_VIEWS_LOCATION_BAR_BUBBLE_ICON_VIEW_H_ 6 #define CHROME_BROWSER_UI_VIEWS_LOCATION_BAR_BUBBLE_ICON_VIEW_H_
7 7
8 #include <memory> 8 #include <memory>
9 9
10 #include "base/macros.h" 10 #include "base/macros.h"
(...skipping 44 matching lines...) Expand 10 before | Expand all | Expand 10 after
55 virtual void OnExecuting(ExecuteSource execute_source) = 0; 55 virtual void OnExecuting(ExecuteSource execute_source) = 0;
56 56
57 // views::View: 57 // views::View:
58 void GetAccessibleState(ui::AXViewState* state) override; 58 void GetAccessibleState(ui::AXViewState* state) override;
59 bool GetTooltipText(const gfx::Point& p, base::string16* tooltip) const 59 bool GetTooltipText(const gfx::Point& p, base::string16* tooltip) const
60 override; 60 override;
61 gfx::Size GetPreferredSize() const override; 61 gfx::Size GetPreferredSize() const override;
62 void Layout() override; 62 void Layout() override;
63 bool OnMousePressed(const ui::MouseEvent& event) override; 63 bool OnMousePressed(const ui::MouseEvent& event) override;
64 void OnMouseReleased(const ui::MouseEvent& event) override; 64 void OnMouseReleased(const ui::MouseEvent& event) override;
65 bool OnKeyPressed(const ui::KeyEvent& event) override;
65 bool OnKeyReleased(const ui::KeyEvent& event) override; 66 bool OnKeyReleased(const ui::KeyEvent& event) override;
66 void ViewHierarchyChanged( 67 void ViewHierarchyChanged(
67 const ViewHierarchyChangedDetails& details) override; 68 const ViewHierarchyChangedDetails& details) override;
68 void OnNativeThemeChanged(const ui::NativeTheme* theme) override; 69 void OnNativeThemeChanged(const ui::NativeTheme* theme) override;
69 void AddInkDropLayer(ui::Layer* ink_drop_layer) override; 70 void AddInkDropLayer(ui::Layer* ink_drop_layer) override;
70 void RemoveInkDropLayer(ui::Layer* ink_drop_layer) override; 71 void RemoveInkDropLayer(ui::Layer* ink_drop_layer) override;
71 std::unique_ptr<views::InkDropHover> CreateInkDropHover() const override; 72 std::unique_ptr<views::InkDropHover> CreateInkDropHover() const override;
72 SkColor GetInkDropBaseColor() const override; 73 SkColor GetInkDropBaseColor() const override;
73 74
74 // ui::EventHandler: 75 // ui::EventHandler:
(...skipping 52 matching lines...) Expand 10 before | Expand all | Expand 10 after
127 // prevent the bubble from reshowing. 128 // prevent the bubble from reshowing.
128 bool suppress_mouse_released_action_; 129 bool suppress_mouse_released_action_;
129 130
130 // Animation delegate for the ink drop ripple effect. 131 // Animation delegate for the ink drop ripple effect.
131 std::unique_ptr<views::InkDropDelegate> ink_drop_delegate_; 132 std::unique_ptr<views::InkDropDelegate> ink_drop_delegate_;
132 133
133 DISALLOW_COPY_AND_ASSIGN(BubbleIconView); 134 DISALLOW_COPY_AND_ASSIGN(BubbleIconView);
134 }; 135 };
135 136
136 #endif // CHROME_BROWSER_UI_VIEWS_LOCATION_BAR_BUBBLE_ICON_VIEW_H_ 137 #endif // CHROME_BROWSER_UI_VIEWS_LOCATION_BAR_BUBBLE_ICON_VIEW_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698