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

Side by Side Diff: chrome/browser/android/offline_pages/prerender_adapter.h

Issue 2044613003: Adds unit tests for offline_pages::PrerenderAdapter calling PrerenderManager. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 6 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 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 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_ANDROID_OFFLINE_PAGES_PRERENDER_ADAPTER_H_ 5 #ifndef CHROME_BROWSER_ANDROID_OFFLINE_PAGES_PRERENDER_ADAPTER_H_
6 #define CHROME_BROWSER_ANDROID_OFFLINE_PAGES_PRERENDER_ADAPTER_H_ 6 #define CHROME_BROWSER_ANDROID_OFFLINE_PAGES_PRERENDER_ADAPTER_H_
7 7
8 #include <memory> 8 #include <memory>
9 9
10 #include "chrome/browser/prerender/prerender_handle.h" 10 #include "chrome/browser/prerender/prerender_handle.h"
(...skipping 72 matching lines...) Expand 10 before | Expand all | Expand 10 after
83 83
84 // Cancels any current prerendering operation and destroys its local handle. 84 // Cancels any current prerendering operation and destroys its local handle.
85 virtual void DestroyActive(); 85 virtual void DestroyActive();
86 86
87 // PrerenderHandle::Observer interface: 87 // PrerenderHandle::Observer interface:
88 void OnPrerenderStart(prerender::PrerenderHandle* handle) override; 88 void OnPrerenderStart(prerender::PrerenderHandle* handle) override;
89 void OnPrerenderStopLoading(prerender::PrerenderHandle* handle) override; 89 void OnPrerenderStopLoading(prerender::PrerenderHandle* handle) override;
90 void OnPrerenderDomContentLoaded(prerender::PrerenderHandle* handle) override; 90 void OnPrerenderDomContentLoaded(prerender::PrerenderHandle* handle) override;
91 void OnPrerenderStop(prerender::PrerenderHandle* handle) override; 91 void OnPrerenderStop(prerender::PrerenderHandle* handle) override;
92 92
93 // Overrides the PrerenderManager for unit testing.
94 void SetPrerenderManagerForTest(
95 prerender::PrerenderManager* test_prerender_manager);
96
93 private: 97 private:
94 // At most one prerender request may be active for this adapter and this 98 // At most one prerender request may be active for this adapter and this
95 // holds its handle if one is active. 99 // holds its handle if one is active.
96 std::unique_ptr<prerender::PrerenderHandle> active_handle_; 100 std::unique_ptr<prerender::PrerenderHandle> active_handle_;
97 101
98 // Observer of active handle events. Not owned. 102 // Observer of active handle events. Not owned.
99 PrerenderAdapter::Observer* observer_; 103 PrerenderAdapter::Observer* observer_;
100 104
105 // Overridden PrerenderManager for unit tests. Not owned.
106 prerender::PrerenderManager* test_prerender_manager_;
107
101 DISALLOW_COPY_AND_ASSIGN(PrerenderAdapter); 108 DISALLOW_COPY_AND_ASSIGN(PrerenderAdapter);
102 }; 109 };
103 110
104 } // namespace offline_pages 111 } // namespace offline_pages
105 112
106 #endif // CHROME_BROWSER_ANDROID_OFFLINE_PAGES_PRERENDER_ADAPTER_H_ 113 #endif // CHROME_BROWSER_ANDROID_OFFLINE_PAGES_PRERENDER_ADAPTER_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698