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

Unified Diff: content/common/navigation_params.cc

Issue 1294243004: PlzNavigate: Make ServiceWorker work with PlzNavigate. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Review comments Created 5 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: content/common/navigation_params.cc
diff --git a/content/common/navigation_params.cc b/content/common/navigation_params.cc
index 5c5dfc778fe5c9be0097c21c204f815da8960176..aaed0d0ec56197af8b8b4a260461df4afb2f9820 100644
--- a/content/common/navigation_params.cc
+++ b/content/common/navigation_params.cc
@@ -6,6 +6,7 @@
#include "base/command_line.h"
#include "base/memory/ref_counted_memory.h"
+#include "content/common/service_worker/service_worker_types.h"
#include "content/public/common/content_switches.h"
namespace content {
@@ -58,18 +59,24 @@ CommonNavigationParams::~CommonNavigationParams() {
}
BeginNavigationParams::BeginNavigationParams()
- : load_flags(0), has_user_gesture(false) {
-}
+ : load_flags(0),
+ has_user_gesture(false),
+ skip_service_worker(false),
+ request_context_type(REQUEST_CONTEXT_TYPE_HYPERLINK) {}
nasko 2015/10/02 22:09:45 Did we confirm the current code uses hyperlink or
Fabrice (no longer in Chrome) 2015/10/06 17:21:38 Done in the other patch.
-BeginNavigationParams::BeginNavigationParams(std::string method,
- std::string headers,
- int load_flags,
- bool has_user_gesture)
+BeginNavigationParams::BeginNavigationParams(
+ std::string method,
+ std::string headers,
+ int load_flags,
+ bool has_user_gesture,
+ bool skip_service_worker,
+ RequestContextType request_context_type)
: method(method),
headers(headers),
load_flags(load_flags),
- has_user_gesture(has_user_gesture) {
-}
+ has_user_gesture(has_user_gesture),
+ skip_service_worker(skip_service_worker),
+ request_context_type(request_context_type) {}
StartNavigationParams::StartNavigationParams()
: is_post(false),
@@ -115,8 +122,8 @@ RequestNavigationParams::RequestNavigationParams()
pending_history_list_offset(-1),
current_history_list_offset(-1),
current_history_list_length(0),
- should_clear_history_list(false) {
-}
+ should_clear_history_list(false),
+ service_worker_provider_id(kInvalidServiceWorkerProviderId) {}
RequestNavigationParams::RequestNavigationParams(
bool is_overriding_user_agent,
@@ -148,8 +155,8 @@ RequestNavigationParams::RequestNavigationParams(
pending_history_list_offset(pending_history_list_offset),
current_history_list_offset(current_history_list_offset),
current_history_list_length(current_history_list_length),
- should_clear_history_list(should_clear_history_list) {
-}
+ should_clear_history_list(should_clear_history_list),
+ service_worker_provider_id(kInvalidServiceWorkerProviderId) {}
RequestNavigationParams::~RequestNavigationParams() {
}

Powered by Google App Engine
This is Rietveld 408576698