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

Side by Side Diff: chrome/browser/android/offline_pages/prerendering_offliner_unittest.cc

Issue 2624833002: Revert of [Offline Pages] Remove need for OfflinePolicy when constructing an Offliner. (Closed)
Patch Set: Created 3 years, 11 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 #include "chrome/browser/android/offline_pages/prerendering_offliner.h" 5 #include "chrome/browser/android/offline_pages/prerendering_offliner.h"
6 6
7 #include <memory> 7 #include <memory>
8 8
9 #include "base/bind.h" 9 #include "base/bind.h"
10 #include "base/run_loop.h" 10 #include "base/run_loop.h"
(...skipping 170 matching lines...) Expand 10 before | Expand all | Expand 10 after
181 181
182 PrerenderingOfflinerTest::PrerenderingOfflinerTest() 182 PrerenderingOfflinerTest::PrerenderingOfflinerTest()
183 : thread_bundle_(content::TestBrowserThreadBundle::IO_MAINLOOP), 183 : thread_bundle_(content::TestBrowserThreadBundle::IO_MAINLOOP),
184 completion_callback_called_(false), 184 completion_callback_called_(false),
185 request_status_(Offliner::RequestStatus::UNKNOWN) {} 185 request_status_(Offliner::RequestStatus::UNKNOWN) {}
186 186
187 PrerenderingOfflinerTest::~PrerenderingOfflinerTest() {} 187 PrerenderingOfflinerTest::~PrerenderingOfflinerTest() {}
188 188
189 void PrerenderingOfflinerTest::SetUp() { 189 void PrerenderingOfflinerTest::SetUp() {
190 model_ = new MockOfflinePageModel(); 190 model_ = new MockOfflinePageModel();
191 offliner_.reset(new PrerenderingOffliner(profile(), model_)); 191 offliner_.reset(new PrerenderingOffliner(profile(), nullptr, model_));
192 std::unique_ptr<MockPrerenderingLoader> mock_loader( 192 std::unique_ptr<MockPrerenderingLoader> mock_loader(
193 new MockPrerenderingLoader(nullptr)); 193 new MockPrerenderingLoader(nullptr));
194 loader_ = mock_loader.get(); 194 loader_ = mock_loader.get();
195 offliner_->SetLoaderForTesting(std::move(mock_loader)); 195 offliner_->SetLoaderForTesting(std::move(mock_loader));
196 } 196 }
197 197
198 void PrerenderingOfflinerTest::OnCompletion(const SavePageRequest& request, 198 void PrerenderingOfflinerTest::OnCompletion(const SavePageRequest& request,
199 Offliner::RequestStatus status) { 199 Offliner::RequestStatus status) {
200 DCHECK(!completion_callback_called_); // Expect single callback per request. 200 DCHECK(!completion_callback_called_); // Expect single callback per request.
201 completion_callback_called_ = true; 201 completion_callback_called_ = true;
(...skipping 191 matching lines...) Expand 10 before | Expand all | Expand 10 after
393 EXPECT_FALSE(loader()->IsIdle()); 393 EXPECT_FALSE(loader()->IsIdle());
394 394
395 offliner()->SetApplicationStateForTesting( 395 offliner()->SetApplicationStateForTesting(
396 base::android::APPLICATION_STATE_HAS_RUNNING_ACTIVITIES); 396 base::android::APPLICATION_STATE_HAS_RUNNING_ACTIVITIES);
397 397
398 // Loader still loading since not low-end device. 398 // Loader still loading since not low-end device.
399 EXPECT_FALSE(loader()->IsIdle()); 399 EXPECT_FALSE(loader()->IsIdle());
400 } 400 }
401 401
402 } // namespace offline_pages 402 } // namespace offline_pages
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698