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

Side by Side Diff: chrome/browser/ui/views/browser_actions_container.h

Issue 6685069: Disambiguate OnMouseCaptureLost from OnMouseReleased, etc. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Address most TODOs and sync. Created 9 years, 9 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 | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2011 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2011 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_BROWSER_ACTIONS_CONTAINER_H_ 5 #ifndef CHROME_BROWSER_UI_VIEWS_BROWSER_ACTIONS_CONTAINER_H_
6 #define CHROME_BROWSER_UI_VIEWS_BROWSER_ACTIONS_CONTAINER_H_ 6 #define CHROME_BROWSER_UI_VIEWS_BROWSER_ACTIONS_CONTAINER_H_
7 #pragma once 7 #pragma once
8 8
9 #include <set> 9 #include <set>
10 #include <string> 10 #include <string>
(...skipping 81 matching lines...) Expand 10 before | Expand all | Expand 10 after
92 const NotificationSource& source, 92 const NotificationSource& source,
93 const NotificationDetails& details) OVERRIDE; 93 const NotificationDetails& details) OVERRIDE;
94 94
95 // MenuButton behavior overrides. These methods all default to TextButton 95 // MenuButton behavior overrides. These methods all default to TextButton
96 // behavior unless this button is a popup. In that case, it uses MenuButton 96 // behavior unless this button is a popup. In that case, it uses MenuButton
97 // behavior. MenuButton has the notion of a child popup being shown where the 97 // behavior. MenuButton has the notion of a child popup being shown where the
98 // button will stay in the pushed state until the "menu" (a popup in this 98 // button will stay in the pushed state until the "menu" (a popup in this
99 // case) is dismissed. 99 // case) is dismissed.
100 virtual bool Activate() OVERRIDE; 100 virtual bool Activate() OVERRIDE;
101 virtual bool OnMousePressed(const views::MouseEvent& event) OVERRIDE; 101 virtual bool OnMousePressed(const views::MouseEvent& event) OVERRIDE;
102 virtual void OnMouseReleased(const views::MouseEvent& event, 102 virtual void OnMouseReleased(const views::MouseEvent& event) OVERRIDE;
103 bool canceled) OVERRIDE;
104 virtual void OnMouseExited(const views::MouseEvent& event) OVERRIDE; 103 virtual void OnMouseExited(const views::MouseEvent& event) OVERRIDE;
105 virtual bool OnKeyReleased(const views::KeyEvent& event) OVERRIDE; 104 virtual bool OnKeyReleased(const views::KeyEvent& event) OVERRIDE;
106 virtual void ShowContextMenu(const gfx::Point& p, 105 virtual void ShowContextMenu(const gfx::Point& p,
107 bool is_mouse_gesture) OVERRIDE; 106 bool is_mouse_gesture) OVERRIDE;
108 107
109 // Notifications when to set button state to pushed/not pushed (for when the 108 // Notifications when to set button state to pushed/not pushed (for when the
110 // popup/context menu is hidden or shown by the container). 109 // popup/context menu is hidden or shown by the container).
111 void SetButtonPushed(); 110 void SetButtonPushed();
112 void SetButtonNotPushed(); 111 void SetButtonNotPushed();
113 112
(...skipping 403 matching lines...) Expand 10 before | Expand all | Expand 10 after
517 516
518 ScopedRunnableMethodFactory<BrowserActionsContainer> task_factory_; 517 ScopedRunnableMethodFactory<BrowserActionsContainer> task_factory_;
519 518
520 // Handles delayed showing of the overflow menu when hovering. 519 // Handles delayed showing of the overflow menu when hovering.
521 ScopedRunnableMethodFactory<BrowserActionsContainer> show_menu_task_factory_; 520 ScopedRunnableMethodFactory<BrowserActionsContainer> show_menu_task_factory_;
522 521
523 DISALLOW_COPY_AND_ASSIGN(BrowserActionsContainer); 522 DISALLOW_COPY_AND_ASSIGN(BrowserActionsContainer);
524 }; 523 };
525 524
526 #endif // CHROME_BROWSER_UI_VIEWS_BROWSER_ACTIONS_CONTAINER_H_ 525 #endif // CHROME_BROWSER_UI_VIEWS_BROWSER_ACTIONS_CONTAINER_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698