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

Side by Side Diff: chrome/browser/ui/cocoa/location_bar/star_decoration.h

Issue 10874068: When using action box on Mac, hide star when page is not bookmarked. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Applied review comments. Created 8 years, 3 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_COCOA_LOCATION_BAR_STAR_DECORATION_H_ 5 #ifndef CHROME_BROWSER_UI_COCOA_LOCATION_BAR_STAR_DECORATION_H_
6 #define CHROME_BROWSER_UI_COCOA_LOCATION_BAR_STAR_DECORATION_H_ 6 #define CHROME_BROWSER_UI_COCOA_LOCATION_BAR_STAR_DECORATION_H_
7 7
8 #import <Cocoa/Cocoa.h> 8 #import <Cocoa/Cocoa.h>
9 9
10 #include "chrome/browser/ui/cocoa/location_bar/image_decoration.h" 10 #include "chrome/browser/ui/cocoa/location_bar/image_decoration.h"
11 11
12 class CommandUpdater; 12 class CommandUpdater;
13 13
14 // Star icon on the right side of the field. 14 // Star icon on the right side of the field.
15 15
16 class StarDecoration : public ImageDecoration { 16 class StarDecoration : public ImageDecoration {
17 public: 17 public:
18 explicit StarDecoration(CommandUpdater* command_updater); 18 explicit StarDecoration(CommandUpdater* command_updater);
19 virtual ~StarDecoration(); 19 virtual ~StarDecoration();
20 20
21 // Sets the image and tooltip based on |starred|. 21 // Sets the image and tooltip based on |starred|.
22 void SetStarred(bool starred); 22 void SetStarred(bool starred);
23 23
24 // Returns true if the star is lit.
25 bool starred() const { return starred_; }
26
24 // Get the point where the bookmark bubble should point within the 27 // Get the point where the bookmark bubble should point within the
25 // decoration's frame. 28 // decoration's frame.
26 NSPoint GetBubblePointInFrame(NSRect frame); 29 NSPoint GetBubblePointInFrame(NSRect frame);
27 30
28 // Implement |LocationBarDecoration|. 31 // Implement |LocationBarDecoration|.
29 virtual bool AcceptsMousePress() OVERRIDE; 32 virtual bool AcceptsMousePress() OVERRIDE;
30 virtual bool OnMousePressed(NSRect frame) OVERRIDE; 33 virtual bool OnMousePressed(NSRect frame) OVERRIDE;
31 virtual NSString* GetToolTip() OVERRIDE; 34 virtual NSString* GetToolTip() OVERRIDE;
32 35
33 private: 36 private:
34 // For bringing up bookmark bar. 37 // For bringing up bookmark bar.
35 CommandUpdater* command_updater_; // Weak, owned by Browser. 38 CommandUpdater* command_updater_; // Weak, owned by Browser.
36 39
37 // The string to show for a tooltip. 40 // The string to show for a tooltip.
38 scoped_nsobject<NSString> tooltip_; 41 scoped_nsobject<NSString> tooltip_;
39 42
43 // Whether the star icon is lit.
44 bool starred_;
45
40 DISALLOW_COPY_AND_ASSIGN(StarDecoration); 46 DISALLOW_COPY_AND_ASSIGN(StarDecoration);
41 }; 47 };
42 48
43 #endif // CHROME_BROWSER_UI_COCOA_LOCATION_BAR_STAR_DECORATION_H_ 49 #endif // CHROME_BROWSER_UI_COCOA_LOCATION_BAR_STAR_DECORATION_H_
OLDNEW
« no previous file with comments | « chrome/browser/ui/cocoa/location_bar/location_bar_view_mac.mm ('k') | chrome/browser/ui/cocoa/location_bar/star_decoration.mm » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698