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

Side by Side Diff: chrome/browser/web_resource/resource_request_allowed_notifier_test_util.h

Issue 648653003: Standardize usage of virtual/override/final in chrome/browser/ (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 2013 The Chromium Authors. All rights reserved. 1 // Copyright 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_WEB_RESOURCE_RESOURCE_REQUEST_ALLOWED_NOTIFIER_TEST_UTIL_ H_ 5 #ifndef CHROME_BROWSER_WEB_RESOURCE_RESOURCE_REQUEST_ALLOWED_NOTIFIER_TEST_UTIL_ H_
6 #define CHROME_BROWSER_WEB_RESOURCE_RESOURCE_REQUEST_ALLOWED_NOTIFIER_TEST_UTIL_ H_ 6 #define CHROME_BROWSER_WEB_RESOURCE_RESOURCE_REQUEST_ALLOWED_NOTIFIER_TEST_UTIL_ H_
7 7
8 #include "chrome/browser/web_resource/resource_request_allowed_notifier.h" 8 #include "chrome/browser/web_resource/resource_request_allowed_notifier.h"
9 9
10 // A subclass of ResourceRequestAllowedNotifier used to expose some 10 // A subclass of ResourceRequestAllowedNotifier used to expose some
11 // functionality for testing. 11 // functionality for testing.
12 // 12 //
13 // By default, the constructor sets this class to override 13 // By default, the constructor sets this class to override
14 // ResourceRequestsAllowed, so its state can be set with SetRequestsAllowed. 14 // ResourceRequestsAllowed, so its state can be set with SetRequestsAllowed.
15 // This is meant for higher level tests of services to ensure they adhere to the 15 // This is meant for higher level tests of services to ensure they adhere to the
16 // notifications of the ResourceRequestAllowedNotifier. Lower level tests can 16 // notifications of the ResourceRequestAllowedNotifier. Lower level tests can
17 // disable this by calling SetRequestsAllowedOverride with the value they want 17 // disable this by calling SetRequestsAllowedOverride with the value they want
18 // it to return. 18 // it to return.
19 class TestRequestAllowedNotifier : public ResourceRequestAllowedNotifier { 19 class TestRequestAllowedNotifier : public ResourceRequestAllowedNotifier {
20 public: 20 public:
21 TestRequestAllowedNotifier(); 21 TestRequestAllowedNotifier();
22 virtual ~TestRequestAllowedNotifier(); 22 ~TestRequestAllowedNotifier() override;
23 23
24 // A version of |Init()| that accepts a custom EulaAcceptedNotifier. 24 // A version of |Init()| that accepts a custom EulaAcceptedNotifier.
25 void InitWithEulaAcceptNotifier( 25 void InitWithEulaAcceptNotifier(
26 Observer* observer, 26 Observer* observer,
27 scoped_ptr<EulaAcceptedNotifier> eula_notifier); 27 scoped_ptr<EulaAcceptedNotifier> eula_notifier);
28 28
29 // Makes ResourceRequestsAllowed return |allowed| when it is called. 29 // Makes ResourceRequestsAllowed return |allowed| when it is called.
30 void SetRequestsAllowedOverride(bool allowed); 30 void SetRequestsAllowedOverride(bool allowed);
31 31
32 // Notify observers that requests are allowed. This will only work if 32 // Notify observers that requests are allowed. This will only work if
33 // the observer is expecting a notification. 33 // the observer is expecting a notification.
34 void NotifyObserver(); 34 void NotifyObserver();
35 35
36 // ResourceRequestAllowedNotifier overrides: 36 // ResourceRequestAllowedNotifier overrides:
37 virtual State GetResourceRequestsAllowedState() override; 37 State GetResourceRequestsAllowedState() override;
38 virtual EulaAcceptedNotifier* CreateEulaNotifier() override; 38 EulaAcceptedNotifier* CreateEulaNotifier() override;
39 39
40 private: 40 private:
41 scoped_ptr<EulaAcceptedNotifier> test_eula_notifier_; 41 scoped_ptr<EulaAcceptedNotifier> test_eula_notifier_;
42 bool override_requests_allowed_; 42 bool override_requests_allowed_;
43 bool requests_allowed_; 43 bool requests_allowed_;
44 44
45 DISALLOW_COPY_AND_ASSIGN(TestRequestAllowedNotifier); 45 DISALLOW_COPY_AND_ASSIGN(TestRequestAllowedNotifier);
46 }; 46 };
47 47
48 #endif // CHROME_BROWSER_WEB_RESOURCE_RESOURCE_REQUEST_ALLOWED_NOTIFIER_TEST_UT IL_H_ 48 #endif // CHROME_BROWSER_WEB_RESOURCE_RESOURCE_REQUEST_ALLOWED_NOTIFIER_TEST_UT IL_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698