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

Unified Diff: chrome/browser/net/resource_prefetch_predictor_observer.h

Issue 2887133003: predictors: Refactor resource_prefetch_predictor triggering. (Closed)
Patch Set: . Created 3 years, 7 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/net/resource_prefetch_predictor_observer.h
diff --git a/chrome/browser/net/resource_prefetch_predictor_observer.h b/chrome/browser/net/resource_prefetch_predictor_observer.h
deleted file mode 100644
index 4801540ce9be4e4b8c414fd8be20e694a61fae84..0000000000000000000000000000000000000000
--- a/chrome/browser/net/resource_prefetch_predictor_observer.h
+++ /dev/null
@@ -1,82 +0,0 @@
-// Copyright 2014 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.
-
-#ifndef CHROME_BROWSER_NET_RESOURCE_PREFETCH_PREDICTOR_OBSERVER_H_
-#define CHROME_BROWSER_NET_RESOURCE_PREFETCH_PREDICTOR_OBSERVER_H_
-
-#include <memory>
-
-#include "base/macros.h"
-#include "base/memory/weak_ptr.h"
-#include "chrome/browser/predictors/resource_prefetch_predictor.h"
-#include "content/public/browser/resource_request_info.h"
-#include "content/public/common/resource_type.h"
-
-namespace net {
-class URLRequest;
-}
-
-class GURL;
-
-namespace chrome_browser_net {
-
-// Observes resource requests in the ResourceDispatcherHostDelegate and notifies
-// the ResourcePrefetchPredictor about the ones it is interested in.
-// - Has an instance per profile, and is owned by the corresponding
-// ProfileIOData.
-// - Needs to be constructed on UI thread. Can be destroyed on UI or IO thread.
-// As for member functions, public members are meant to be called on the IO
-// thread and private members from the UI thread.
-class ResourcePrefetchPredictorObserver {
- public:
- explicit ResourcePrefetchPredictorObserver(
- predictors::ResourcePrefetchPredictor* predictor);
- ~ResourcePrefetchPredictorObserver();
-
- // Parts of the ResourceDispatcherHostDelegate that we want to observe.
- void OnRequestStarted(net::URLRequest* request,
- content::ResourceType resource_type,
- const content::ResourceRequestInfo::WebContentsGetter&
- web_contents_getter);
- void OnRequestRedirected(
- net::URLRequest* request,
- const GURL& redirect_url,
- const content::ResourceRequestInfo::WebContentsGetter&
- web_contents_getter);
- void OnResponseStarted(net::URLRequest* request,
- const content::ResourceRequestInfo::WebContentsGetter&
- web_contents_getter);
-
- private:
- void OnRequestStartedOnUIThread(
- std::unique_ptr<predictors::ResourcePrefetchPredictor::URLRequestSummary>
- summary,
- const content::ResourceRequestInfo::WebContentsGetter&
- web_contents_getter,
- const GURL& main_frame_url,
- const base::TimeTicks& creation_time) const;
- void OnRequestRedirectedOnUIThread(
- std::unique_ptr<predictors::ResourcePrefetchPredictor::URLRequestSummary>
- summary,
- const content::ResourceRequestInfo::WebContentsGetter&
- web_contents_getter,
- const GURL& main_frame_url,
- const base::TimeTicks& creation_time) const;
- void OnResponseStartedOnUIThread(
- std::unique_ptr<predictors::ResourcePrefetchPredictor::URLRequestSummary>
- summary,
- const content::ResourceRequestInfo::WebContentsGetter&
- web_contents_getter,
- const GURL& main_frame_url,
- const base::TimeTicks& creation_time) const;
-
- // Owned by profile.
- base::WeakPtr<predictors::ResourcePrefetchPredictor> predictor_;
-
- DISALLOW_COPY_AND_ASSIGN(ResourcePrefetchPredictorObserver);
-};
-
-} // namespace chrome_browser_net
-
-#endif // CHROME_BROWSER_NET_RESOURCE_PREFETCH_PREDICTOR_OBSERVER_H_
« no previous file with comments | « chrome/browser/net/loading_predictor_observer.cc ('k') | chrome/browser/net/resource_prefetch_predictor_observer.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698