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

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

Issue 2637563002: [Offline Pages] Snapshotting on timeout of last retry. (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 "base/bind.h" 7 #include "base/bind.h"
8 #include "base/metrics/histogram_macros.h" 8 #include "base/metrics/histogram_macros.h"
9 #include "base/sys_info.h" 9 #include "base/sys_info.h"
10 #include "chrome/browser/android/offline_pages/offline_page_mhtml_archiver.h" 10 #include "chrome/browser/android/offline_pages/offline_page_mhtml_archiver.h"
11 #include "chrome/browser/net/prediction_options.h" 11 #include "chrome/browser/net/prediction_options.h"
12 #include "chrome/browser/profiles/profile.h" 12 #include "chrome/browser/profiles/profile.h"
13 #include "chrome/common/pref_names.h" 13 #include "chrome/common/pref_names.h"
14 #include "components/content_settings/core/common/pref_names.h" 14 #include "components/content_settings/core/common/pref_names.h"
15 #include "components/offline_pages/core/background/offliner_policy.h"
15 #include "components/offline_pages/core/background/save_page_request.h" 16 #include "components/offline_pages/core/background/save_page_request.h"
16 #include "components/offline_pages/core/client_namespace_constants.h" 17 #include "components/offline_pages/core/client_namespace_constants.h"
17 #include "components/offline_pages/core/offline_page_model.h" 18 #include "components/offline_pages/core/offline_page_model.h"
18 #include "components/prefs/pref_service.h" 19 #include "components/prefs/pref_service.h"
19 #include "content/public/browser/browser_context.h" 20 #include "content/public/browser/browser_context.h"
20 #include "content/public/browser/web_contents.h" 21 #include "content/public/browser/web_contents.h"
21 22
22 namespace { 23 namespace {
23 24
24 bool AreThirdPartyCookiesBlocked(content::BrowserContext* browser_context) { 25 bool AreThirdPartyCookiesBlocked(content::BrowserContext* browser_context) {
(...skipping 17 matching lines...) Expand all
42 43
43 } // namespace 44 } // namespace
44 45
45 namespace offline_pages { 46 namespace offline_pages {
46 47
47 PrerenderingOffliner::PrerenderingOffliner( 48 PrerenderingOffliner::PrerenderingOffliner(
48 content::BrowserContext* browser_context, 49 content::BrowserContext* browser_context,
49 const OfflinerPolicy* policy, 50 const OfflinerPolicy* policy,
50 OfflinePageModel* offline_page_model) 51 OfflinePageModel* offline_page_model)
51 : browser_context_(browser_context), 52 : browser_context_(browser_context),
53 policy_(policy),
52 offline_page_model_(offline_page_model), 54 offline_page_model_(offline_page_model),
53 pending_request_(nullptr), 55 pending_request_(nullptr),
54 is_low_end_device_(base::SysInfo::IsLowEndDevice()), 56 is_low_end_device_(base::SysInfo::IsLowEndDevice()),
55 app_listener_(nullptr), 57 app_listener_(nullptr),
56 weak_ptr_factory_(this) {} 58 weak_ptr_factory_(this) {}
57 59
58 PrerenderingOffliner::~PrerenderingOffliner() {} 60 PrerenderingOffliner::~PrerenderingOffliner() {}
59 61
60 void PrerenderingOffliner::OnLoadPageDone( 62 void PrerenderingOffliner::OnLoadPageDone(
61 const SavePageRequest& request, 63 const SavePageRequest& request,
(...skipping 156 matching lines...) Expand 10 before | Expand all | Expand 10 after
218 220
219 void PrerenderingOffliner::Cancel() { 221 void PrerenderingOffliner::Cancel() {
220 if (pending_request_) { 222 if (pending_request_) {
221 pending_request_.reset(nullptr); 223 pending_request_.reset(nullptr);
222 app_listener_.reset(nullptr); 224 app_listener_.reset(nullptr);
223 GetOrCreateLoader()->StopLoading(); 225 GetOrCreateLoader()->StopLoading();
224 // TODO(dougarnett): Consider ability to cancel SavePage request. 226 // TODO(dougarnett): Consider ability to cancel SavePage request.
225 } 227 }
226 } 228 }
227 229
230 bool PrerenderingOffliner::HandleTimeout(const SavePageRequest& request) {
231 if (pending_request_) {
232 DCHECK(request.request_id() == pending_request_->request_id());
233 if (GetOrCreateLoader()->IsLowbarMet() &&
234 (request.started_attempt_count() + 1 >= policy_->GetMaxStartedTries() ||
235 request.completed_attempt_count() + 1 >=
236 policy_->GetMaxCompletedTries())) {
237 GetOrCreateLoader()->StartSnapshot();
238 return true;
239 }
240 }
241 return false;
242 }
243
228 void PrerenderingOffliner::SetLoaderForTesting( 244 void PrerenderingOffliner::SetLoaderForTesting(
229 std::unique_ptr<PrerenderingLoader> loader) { 245 std::unique_ptr<PrerenderingLoader> loader) {
230 DCHECK(!loader_); 246 DCHECK(!loader_);
231 loader_ = std::move(loader); 247 loader_ = std::move(loader);
232 } 248 }
233 249
234 void PrerenderingOffliner::SetLowEndDeviceForTesting(bool is_low_end_device) { 250 void PrerenderingOffliner::SetLowEndDeviceForTesting(bool is_low_end_device) {
235 is_low_end_device_ = is_low_end_device; 251 is_low_end_device_ = is_low_end_device;
236 } 252 }
237 253
(...skipping 25 matching lines...) Expand all
263 base::android::APPLICATION_STATE_HAS_RUNNING_ACTIVITIES) { 279 base::android::APPLICATION_STATE_HAS_RUNNING_ACTIVITIES) {
264 DVLOG(1) << "App became active, canceling current offlining request"; 280 DVLOG(1) << "App became active, canceling current offlining request";
265 SavePageRequest* request = pending_request_.get(); 281 SavePageRequest* request = pending_request_.get();
266 Cancel(); 282 Cancel();
267 completion_callback_.Run(*request, 283 completion_callback_.Run(*request,
268 Offliner::RequestStatus::FOREGROUND_CANCELED); 284 Offliner::RequestStatus::FOREGROUND_CANCELED);
269 } 285 }
270 } 286 }
271 287
272 } // namespace offline_pages 288 } // namespace offline_pages
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698