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

Unified Diff: chrome/browser/android/offline_pages/offline_page_request_interceptor.cc

Issue 2388253002: Use the previews black list for offline previews (Closed)
Patch Set: mmenke comments Created 4 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/android/offline_pages/offline_page_request_interceptor.cc
diff --git a/chrome/browser/android/offline_pages/offline_page_request_interceptor.cc b/chrome/browser/android/offline_pages/offline_page_request_interceptor.cc
index 604714ca2fc6b3f80b25fbad30ec795c467481a4..f08ece361530b40e9f33bf881fa62d55f6c14ac0 100644
--- a/chrome/browser/android/offline_pages/offline_page_request_interceptor.cc
+++ b/chrome/browser/android/offline_pages/offline_page_request_interceptor.cc
@@ -1,25 +1,29 @@
// Copyright 2016 The Chromium Authors. All rights reserved.
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
#include "chrome/browser/android/offline_pages/offline_page_request_interceptor.h"
#include "base/supports_user_data.h"
#include "chrome/browser/android/offline_pages/offline_page_request_job.h"
+#include "components/previews/core/previews_decider.h"
namespace offline_pages {
-OfflinePageRequestInterceptor::OfflinePageRequestInterceptor() {}
+OfflinePageRequestInterceptor::OfflinePageRequestInterceptor(
+ previews::PreviewsDecider* previews_decider)
+ : previews_decider_(previews_decider) {}
OfflinePageRequestInterceptor::~OfflinePageRequestInterceptor() {}
net::URLRequestJob* OfflinePageRequestInterceptor::MaybeInterceptRequest(
net::URLRequest* request,
net::NetworkDelegate* network_delegate) const {
// OfflinePageRequestJob::Create may return a nullptr if the interception
// is not needed for some sort of requests, like non-main resource request,
// non-http request and more.
- return OfflinePageRequestJob::Create(request, network_delegate);
+ return OfflinePageRequestJob::Create(request, network_delegate,
+ previews_decider_);
}
} // namespace offline_pages

Powered by Google App Engine
This is Rietveld 408576698