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

Side by Side Diff: chrome/browser/ui/panels/test_panel_active_state_observer.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_PANELS_TEST_PANEL_ACTIVE_STATE_OBSERVER_H_ 5 #ifndef CHROME_BROWSER_UI_PANELS_TEST_PANEL_ACTIVE_STATE_OBSERVER_H_
6 #define CHROME_BROWSER_UI_PANELS_TEST_PANEL_ACTIVE_STATE_OBSERVER_H_ 6 #define CHROME_BROWSER_UI_PANELS_TEST_PANEL_ACTIVE_STATE_OBSERVER_H_
7 7
8 #include "chrome/browser/ui/panels/test_panel_notification_observer.h" 8 #include "chrome/browser/ui/panels/test_panel_notification_observer.h"
9 9
10 class Panel; 10 class Panel;
11 11
12 // Custom notification observer for waiting on panel active state. 12 // Custom notification observer for waiting on panel active state.
13 // Modeled after ui_test_utils notification observers. 13 // Modeled after ui_test_utils notification observers.
14 class PanelActiveStateObserver : public TestPanelNotificationObserver { 14 class PanelActiveStateObserver : public TestPanelNotificationObserver {
15 public: 15 public:
16 // Register to listen for panel active state change notifications 16 // Register to listen for panel active state change notifications
17 // for the specified panel to detect a change to the expected active 17 // for the specified panel to detect a change to the expected active
18 // state. 18 // state.
19 PanelActiveStateObserver(Panel* panel, bool expect_active); 19 PanelActiveStateObserver(Panel* panel, bool expect_active);
20 virtual ~PanelActiveStateObserver(); 20 ~PanelActiveStateObserver() override;
21 21
22 private: 22 private:
23 // TestNotificationObserver override: 23 // TestNotificationObserver override:
24 virtual bool AtExpectedState() override; 24 bool AtExpectedState() override;
25 25
26 Panel* panel_; 26 Panel* panel_;
27 bool expect_active_; 27 bool expect_active_;
28 28
29 DISALLOW_COPY_AND_ASSIGN(PanelActiveStateObserver); 29 DISALLOW_COPY_AND_ASSIGN(PanelActiveStateObserver);
30 }; 30 };
31 31
32 #endif // CHROME_BROWSER_UI_PANELS_TEST_PANEL_ACTIVE_STATE_OBSERVER_H_ 32 #endif // CHROME_BROWSER_UI_PANELS_TEST_PANEL_ACTIVE_STATE_OBSERVER_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698