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

Unified Diff: chrome/browser/renderer_host/chrome_resource_dispatcher_host_delegate.h

Issue 1721813002: Adding DRP specfic UMA for FirstContentfulPaint (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: NavigationData final draft before adding tests Created 4 years, 8 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/renderer_host/chrome_resource_dispatcher_host_delegate.h
diff --git a/chrome/browser/renderer_host/chrome_resource_dispatcher_host_delegate.h b/chrome/browser/renderer_host/chrome_resource_dispatcher_host_delegate.h
index 25b26f125180e5dbcf41b400f5d4162b9148b0fd..95d590f6a385fcf078f24bb96252237aa48630c0 100644
--- a/chrome/browser/renderer_host/chrome_resource_dispatcher_host_delegate.h
+++ b/chrome/browser/renderer_host/chrome_resource_dispatcher_host_delegate.h
@@ -1,37 +1,42 @@
// Copyright (c) 2012 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_RENDERER_HOST_CHROME_RESOURCE_DISPATCHER_HOST_DELEGATE_H_
#define CHROME_BROWSER_RENDERER_HOST_CHROME_RESOURCE_DISPATCHER_HOST_DELEGATE_H_
#include <map>
+#include <memory>
#include <set>
#include "base/compiler_specific.h"
#include "base/macros.h"
#include "base/memory/ref_counted.h"
#include "chrome/browser/external_protocol/external_protocol_handler.h"
#include "content/public/browser/resource_dispatcher_host_delegate.h"
class DelayedResourceQueue;
class DownloadRequestLimiter;
namespace extensions {
class UserScriptListener;
}
namespace safe_browsing {
class SafeBrowsingService;
}
+namespace content {
+class NavigationData;
+}
+
// Implements ResourceDispatcherHostDelegate. Currently used by the Prerender
// system to abort requests and add to the load flags when a request begins.
class ChromeResourceDispatcherHostDelegate
: public content::ResourceDispatcherHostDelegate {
public:
ChromeResourceDispatcherHostDelegate();
~ChromeResourceDispatcherHostDelegate() override;
// ResourceDispatcherHostDelegate implementation.
bool ShouldBeginRequest(const std::string& method,
@@ -84,20 +89,23 @@ class ChromeResourceDispatcherHostDelegate
void RequestComplete(net::URLRequest* url_request) override;
bool ShouldEnableLoFiMode(
const net::URLRequest& url_request,
content::ResourceContext* resource_context) override;
// Called on the UI thread. Allows switching out the
// ExternalProtocolHandler::Delegate for testing code.
static void SetExternalProtocolHandlerDelegateForTesting(
ExternalProtocolHandler::Delegate* delegate);
+ std::unique_ptr<content::NavigationData> GetNavigationData(
nasko 2016/04/26 20:18:39 Keep the ordering the same as the inherited interf
RyanSturm 2016/04/27 23:27:46 Done.
+ net::URLRequest* request) const override;
+
private:
#if defined(ENABLE_EXTENSIONS)
struct StreamTargetInfo {
std::string extension_id;
std::string view_id;
};
#endif
void AppendStandardResourceThrottles(
net::URLRequest* request,

Powered by Google App Engine
This is Rietveld 408576698