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

Unified Diff: content/common/navigation_params.h

Issue 2099243002: PlzNavigate: properly set the initiator of the navigation (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase 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/common/navigation_params.h
diff --git a/content/common/navigation_params.h b/content/common/navigation_params.h
index ec5ec419118524efed1c9ebb93fdd39c62d59a8a..4c3ac3078f68f88f35d23d79f28fb03b7614bd48 100644
--- a/content/common/navigation_params.h
+++ b/content/common/navigation_params.h
@@ -11,6 +11,7 @@
#include <string>
#include "base/memory/ref_counted.h"
+#include "base/optional.h"
#include "base/time/time.h"
#include "build/build_config.h"
#include "content/common/content_export.h"
@@ -22,6 +23,7 @@
#include "content/public/common/resource_response.h"
#include "ui/base/page_transition_types.h"
#include "url/gurl.h"
+#include "url/origin.h"
namespace content {
@@ -146,8 +148,10 @@ struct CONTENT_EXPORT BeginNavigationParams {
int load_flags,
bool has_user_gesture,
bool skip_service_worker,
- RequestContextType request_context_type);
+ RequestContextType request_context_type,
+ const base::Optional<url::Origin>& initiator);
BeginNavigationParams(const BeginNavigationParams& other);
+ ~BeginNavigationParams();
// Additional HTTP request headers.
std::string headers;
@@ -163,6 +167,11 @@ struct CONTENT_EXPORT BeginNavigationParams {
// Indicates the request context type.
RequestContextType request_context_type;
+
+ // Indicates the initiator of the request. In auxilliary navigations, this is
+ // the origin of the document that triggered the navigation. This parameter
+ // can be null during browser-initiated navigations.
nasko 2016/11/18 19:49:15 nit: s/during/for/
+ base::Optional<url::Origin> initiator;
};
// Provided by the browser -----------------------------------------------------

Powered by Google App Engine
This is Rietveld 408576698