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

Side by Side Diff: chrome/browser/ui/ash/launcher/browser_shortcut_launcher_item_controller.h

Issue 23530020: [cleanup] Remove LauncherItemChanged() from LauncherItemController (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 7 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
OLDNEW
1 // Copyright (c) 2013 The Chromium Authors. All rights reserved. 1 // Copyright (c) 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_ASH_LAUNCHER_BROWSER_SHORTCUT_LAUNCHER_ITEM_CONTROLLER _H_ 5 #ifndef CHROME_BROWSER_UI_ASH_LAUNCHER_BROWSER_SHORTCUT_LAUNCHER_ITEM_CONTROLLER _H_
6 #define CHROME_BROWSER_UI_ASH_LAUNCHER_BROWSER_SHORTCUT_LAUNCHER_ITEM_CONTROLLER _H_ 6 #define CHROME_BROWSER_UI_ASH_LAUNCHER_BROWSER_SHORTCUT_LAUNCHER_ITEM_CONTROLLER _H_
7 7
8 #include "chrome/browser/ui/ash/launcher/launcher_item_controller.h" 8 #include "chrome/browser/ui/ash/launcher/launcher_item_controller.h"
9 9
10 namespace content { 10 namespace content {
(...skipping 17 matching lines...) Expand all
28 virtual ~BrowserShortcutLauncherItemController(); 28 virtual ~BrowserShortcutLauncherItemController();
29 29
30 // LauncherItemController overrides: 30 // LauncherItemController overrides:
31 virtual string16 GetTitle() OVERRIDE; 31 virtual string16 GetTitle() OVERRIDE;
32 virtual bool IsCurrentlyShownInWindow(aura::Window* window) const OVERRIDE; 32 virtual bool IsCurrentlyShownInWindow(aura::Window* window) const OVERRIDE;
33 virtual bool IsOpen() const OVERRIDE; 33 virtual bool IsOpen() const OVERRIDE;
34 virtual bool IsVisible() const OVERRIDE; 34 virtual bool IsVisible() const OVERRIDE;
35 virtual void Launch(int event_flags) OVERRIDE; 35 virtual void Launch(int event_flags) OVERRIDE;
36 virtual void Activate() OVERRIDE; 36 virtual void Activate() OVERRIDE;
37 virtual void Close() OVERRIDE; 37 virtual void Close() OVERRIDE;
38 virtual void LauncherItemChanged(
39 int model_index,
40 const ash::LauncherItem& old_item) OVERRIDE;
41 virtual void Clicked(const ui::Event& event) OVERRIDE; 38 virtual void Clicked(const ui::Event& event) OVERRIDE;
42 virtual void OnRemoved() OVERRIDE; 39 virtual void OnRemoved() OVERRIDE;
43 virtual ChromeLauncherAppMenuItems GetApplicationList( 40 virtual ChromeLauncherAppMenuItems GetApplicationList(
44 int event_flags) OVERRIDE; 41 int event_flags) OVERRIDE;
45 42
46 private: 43 private:
47 // Get the favicon for the browser list entry for |web_contents|. 44 // Get the favicon for the browser list entry for |web_contents|.
48 // Note that for incognito windows the incognito icon will be returned. 45 // Note that for incognito windows the incognito icon will be returned.
49 gfx::Image GetBrowserListIcon(content::WebContents* web_contents) const; 46 gfx::Image GetBrowserListIcon(content::WebContents* web_contents) const;
50 47
51 // Get the title for the browser list entry for |web_contents|. 48 // Get the title for the browser list entry for |web_contents|.
52 // If |web_contents| has not loaded, returns "Net Tab". 49 // If |web_contents| has not loaded, returns "Net Tab".
53 string16 GetBrowserListTitle(content::WebContents* web_contents) const; 50 string16 GetBrowserListTitle(content::WebContents* web_contents) const;
54 51
55 // Check if the given |web_contents| is in incognito mode. 52 // Check if the given |web_contents| is in incognito mode.
56 bool IsIncognito(content::WebContents* web_contents) const; 53 bool IsIncognito(content::WebContents* web_contents) const;
57 54
58 // Activate a browser - or advance to the next one on the list. 55 // Activate a browser - or advance to the next one on the list.
59 void ActivateOrAdvanceToNextBrowser(); 56 void ActivateOrAdvanceToNextBrowser();
60 57
61 Profile* profile_; 58 Profile* profile_;
62 59
63 DISALLOW_COPY_AND_ASSIGN(BrowserShortcutLauncherItemController); 60 DISALLOW_COPY_AND_ASSIGN(BrowserShortcutLauncherItemController);
64 }; 61 };
65 62
66 #endif // CHROME_BROWSER_UI_ASH_LAUNCHER_BROWSER_SHORTCUT_LAUNCHER_ITEM_CONTROL LER_H_ 63 #endif // CHROME_BROWSER_UI_ASH_LAUNCHER_BROWSER_SHORTCUT_LAUNCHER_ITEM_CONTROL LER_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698