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

Side by Side Diff: chrome/browser/ui/cocoa/tabs/tab_strip_model_observer_bridge.h

Issue 671653002: Standardize usage of virtual/override/final in chrome/browser/ui/ (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 2 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) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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_TABS_TAB_STRIP_MODEL_OBSERVER_BRIDGE_H_ 5 #ifndef CHROME_BROWSER_UI_COCOA_TABS_TAB_STRIP_MODEL_OBSERVER_BRIDGE_H_
6 #define CHROME_BROWSER_UI_COCOA_TABS_TAB_STRIP_MODEL_OBSERVER_BRIDGE_H_ 6 #define CHROME_BROWSER_UI_COCOA_TABS_TAB_STRIP_MODEL_OBSERVER_BRIDGE_H_
7 7
8 #import <Foundation/Foundation.h> 8 #import <Foundation/Foundation.h>
9 9
10 #include "base/compiler_specific.h" 10 #include "base/compiler_specific.h"
11 #include "chrome/browser/ui/tabs/tab_strip_model_observer.h" 11 #include "chrome/browser/ui/tabs/tab_strip_model_observer.h"
12 12
13 class TabStripModel; 13 class TabStripModel;
14 14
15 namespace content { 15 namespace content {
16 class WebContents; 16 class WebContents;
17 } 17 }
18 18
19 // A C++ bridge class to handle receiving notifications from the C++ tab strip 19 // A C++ bridge class to handle receiving notifications from the C++ tab strip
20 // model. When the caller allocates a bridge, it automatically registers for 20 // model. When the caller allocates a bridge, it automatically registers for
21 // notifications from |model| and passes messages to |controller| via the 21 // notifications from |model| and passes messages to |controller| via the
22 // informal protocol below. The owner of this object is responsible for deleting 22 // informal protocol below. The owner of this object is responsible for deleting
23 // it (and thus unhooking notifications) before |controller| is destroyed. 23 // it (and thus unhooking notifications) before |controller| is destroyed.
24 class TabStripModelObserverBridge : public TabStripModelObserver { 24 class TabStripModelObserverBridge : public TabStripModelObserver {
25 public: 25 public:
26 TabStripModelObserverBridge(TabStripModel* model, id controller); 26 TabStripModelObserverBridge(TabStripModel* model, id controller);
27 virtual ~TabStripModelObserverBridge(); 27 ~TabStripModelObserverBridge() override;
28 28
29 // Overridden from TabStripModelObserver 29 // Overridden from TabStripModelObserver
30 virtual void TabInsertedAt(content::WebContents* contents, 30 void TabInsertedAt(content::WebContents* contents,
31 int index, 31 int index,
32 bool foreground) override; 32 bool foreground) override;
33 virtual void TabClosingAt(TabStripModel* tab_strip_model, 33 void TabClosingAt(TabStripModel* tab_strip_model,
34 content::WebContents* contents, 34 content::WebContents* contents,
35 int index) override; 35 int index) override;
36 virtual void TabDetachedAt(content::WebContents* contents, 36 void TabDetachedAt(content::WebContents* contents, int index) override;
37 int index) override; 37 void TabDeactivated(content::WebContents* contents) override;
38 virtual void TabDeactivated(content::WebContents* contents) override; 38 void ActiveTabChanged(content::WebContents* old_contents,
39 virtual void ActiveTabChanged(content::WebContents* old_contents, 39 content::WebContents* new_contents,
40 content::WebContents* new_contents, 40 int index,
41 int index, 41 int reason) override;
42 int reason) override; 42 void TabSelectionChanged(TabStripModel* tab_strip_model,
43 virtual void TabSelectionChanged( 43 const ui::ListSelectionModel& old_model) override;
44 TabStripModel* tab_strip_model, 44 void TabMoved(content::WebContents* contents,
45 const ui::ListSelectionModel& old_model) override; 45 int from_index,
46 virtual void TabMoved(content::WebContents* contents, 46 int to_index) override;
47 int from_index, 47 void TabChangedAt(content::WebContents* contents,
48 int to_index) override; 48 int index,
49 virtual void TabChangedAt(content::WebContents* contents, 49 TabChangeType change_type) override;
50 int index, 50 void TabReplacedAt(TabStripModel* tab_strip_model,
51 TabChangeType change_type) override; 51 content::WebContents* old_contents,
52 virtual void TabReplacedAt(TabStripModel* tab_strip_model, 52 content::WebContents* new_contents,
53 content::WebContents* old_contents, 53 int index) override;
54 content::WebContents* new_contents, 54 void TabMiniStateChanged(content::WebContents* contents, int index) override;
55 int index) override; 55 void TabStripEmpty() override;
56 virtual void TabMiniStateChanged(content::WebContents* contents, 56 void TabStripModelDeleted() override;
57 int index) override;
58 virtual void TabStripEmpty() override;
59 virtual void TabStripModelDeleted() override;
60 57
61 private: 58 private:
62 id controller_; // weak, owns me 59 id controller_; // weak, owns me
63 TabStripModel* model_; // weak, owned by Browser 60 TabStripModel* model_; // weak, owned by Browser
64 }; 61 };
65 62
66 // A collection of methods which can be selectively implemented by any 63 // A collection of methods which can be selectively implemented by any
67 // Cocoa object to receive updates about changes to a tab strip model. It is 64 // Cocoa object to receive updates about changes to a tab strip model. It is
68 // ok to not implement them, the calling code checks before calling. 65 // ok to not implement them, the calling code checks before calling.
69 @interface NSObject(TabStripModelBridge) 66 @interface NSObject(TabStripModelBridge)
(...skipping 19 matching lines...) Expand all
89 previousContents:(content::WebContents*)oldContents 86 previousContents:(content::WebContents*)oldContents
90 atIndex:(NSInteger)index; 87 atIndex:(NSInteger)index;
91 - (void)tabMiniStateChangedWithContents:(content::WebContents*)contents 88 - (void)tabMiniStateChangedWithContents:(content::WebContents*)contents
92 atIndex:(NSInteger)index; 89 atIndex:(NSInteger)index;
93 - (void)tabStripEmpty; 90 - (void)tabStripEmpty;
94 - (void)tabStripModelDeleted; 91 - (void)tabStripModelDeleted;
95 - (void)tabSelectionChanged; 92 - (void)tabSelectionChanged;
96 @end 93 @end
97 94
98 #endif // CHROME_BROWSER_UI_COCOA_TABS_TAB_STRIP_MODEL_OBSERVER_BRIDGE_H_ 95 #endif // CHROME_BROWSER_UI_COCOA_TABS_TAB_STRIP_MODEL_OBSERVER_BRIDGE_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698