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

Unified Diff: content/public/browser/navigation_handle.h

Issue 2413663003: Expose GlobalRequestID in NavigationHandle and ResourceRequestInfo. (Closed)
Patch Set: Use OnURLRequestDestroyed Created 4 years, 1 month 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: content/public/browser/navigation_handle.h
diff --git a/content/public/browser/navigation_handle.h b/content/public/browser/navigation_handle.h
index f2506c82dce30a5df29d0599180cd8268f09ae2a..2ed412b6167491c6bf6cbc49194088ea52c37645 100644
--- a/content/public/browser/navigation_handle.h
+++ b/content/public/browser/navigation_handle.h
@@ -6,6 +6,7 @@
#define CONTENT_PUBLIC_BROWSER_NAVIGATION_HANDLE_H_
#include <memory>
+#include <string>
#include "content/common/content_export.h"
#include "content/public/browser/navigation_throttle.h"
@@ -21,6 +22,7 @@ class HttpResponseHeaders;
} // namespace net
namespace content {
+struct GlobalRequestID;
class NavigationData;
class NavigationThrottle;
class RenderFrameHost;
@@ -188,6 +190,14 @@ class CONTENT_EXPORT NavigationHandle {
virtual void CancelDeferredNavigation(
NavigationThrottle::ThrottleCheckResult result) = 0;
+ // Returns the ID of the URLRequest associated with this navigation. Can only
+ // be called from NavigationThrottle::WillProcessResponse and
+ // WebContentsObserver::ReadyToCommitNavigation.
+ // In the case of transfer navigations, this is the ID of the first request
+ // made. The transferred request's ID will not be tracked by the
+ // NavigationHandle.
+ virtual const GlobalRequestID& GetGlobalRequestID() = 0;
+
// Testing methods ----------------------------------------------------------
//
// The following methods should be used exclusively for writing unit tests.
« no previous file with comments | « content/browser/loader/resource_request_info_impl.cc ('k') | content/public/browser/resource_request_info.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698