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

Side by Side Diff: content/common/navigation_params.h

Issue 2735183003: PlzNavigate: stop navigations when opening a document for write (Closed)
Patch Set: Rebase Created 3 years, 9 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 unified diff | Download patch
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef CONTENT_COMMON_NAVIGATION_PARAMS_H_ 5 #ifndef CONTENT_COMMON_NAVIGATION_PARAMS_H_
6 #define CONTENT_COMMON_NAVIGATION_PARAMS_H_ 6 #define CONTENT_COMMON_NAVIGATION_PARAMS_H_
7 7
8 #include <stdint.h> 8 #include <stdint.h>
9 9
10 #include <map> 10 #include <map>
(...skipping 124 matching lines...) Expand 10 before | Expand all | Expand 10 after
135 // TODO(clamy): See if it is possible to reuse this in 135 // TODO(clamy): See if it is possible to reuse this in
136 // ResourceMsg_Request_Params. 136 // ResourceMsg_Request_Params.
137 BeginNavigationParams(); 137 BeginNavigationParams();
138 BeginNavigationParams( 138 BeginNavigationParams(
139 std::string headers, 139 std::string headers,
140 int load_flags, 140 int load_flags,
141 bool has_user_gesture, 141 bool has_user_gesture,
142 bool skip_service_worker, 142 bool skip_service_worker,
143 RequestContextType request_context_type, 143 RequestContextType request_context_type,
144 blink::WebMixedContentContextType mixed_content_context_type, 144 blink::WebMixedContentContextType mixed_content_context_type,
145 const base::Optional<url::Origin>& initiator_origin); 145 const base::Optional<url::Origin>& initiator_origin,
146 int renderer_navigation_id);
146 BeginNavigationParams(const BeginNavigationParams& other); 147 BeginNavigationParams(const BeginNavigationParams& other);
147 ~BeginNavigationParams(); 148 ~BeginNavigationParams();
148 149
149 // Additional HTTP request headers. 150 // Additional HTTP request headers.
150 std::string headers; 151 std::string headers;
151 152
152 // net::URLRequest load flags (net::LOAD_NORMAL) by default). 153 // net::URLRequest load flags (net::LOAD_NORMAL) by default).
153 int load_flags; 154 int load_flags;
154 155
155 // True if the request was user initiated. 156 // True if the request was user initiated.
(...skipping 13 matching lines...) Expand all
169 std::string searchable_form_encoding; 170 std::string searchable_form_encoding;
170 171
171 // Indicates the initiator of the request. In auxilliary navigations, this is 172 // Indicates the initiator of the request. In auxilliary navigations, this is
172 // the origin of the document that triggered the navigation. This parameter 173 // the origin of the document that triggered the navigation. This parameter
173 // can be null during browser-initiated navigations. 174 // can be null during browser-initiated navigations.
174 base::Optional<url::Origin> initiator_origin; 175 base::Optional<url::Origin> initiator_origin;
175 176
176 // If the transition type is a client side redirect, then this holds the URL 177 // If the transition type is a client side redirect, then this holds the URL
177 // of the page that had the client side redirect. 178 // of the page that had the client side redirect.
178 GURL client_side_redirect_url; 179 GURL client_side_redirect_url;
180
181 // ID provided by the renderer to identify this navigation. It will be sent
182 // back to the renderer if the browser decides to drop the navigation.
183 // -1 is this navigation wasn't renderer-initiated.
nasko 2017/03/16 21:23:14 nit: "Value of -1 means this ..." Why not define
clamy 2017/03/17 15:19:11 Done.
184 int renderer_navigation_id;
179 }; 185 };
180 186
181 // Provided by the browser ----------------------------------------------------- 187 // Provided by the browser -----------------------------------------------------
182 // 188 //
183 // These structs are sent by the browser to the renderer to start/commit a 189 // These structs are sent by the browser to the renderer to start/commit a
184 // navigation depending on whether browser-side navigation is enabled. 190 // navigation depending on whether browser-side navigation is enabled.
185 // Parameters used both in the current architecture and PlzNavigate should be 191 // Parameters used both in the current architecture and PlzNavigate should be
186 // put in RequestNavigationParams. Parameters only used by the current 192 // put in RequestNavigationParams. Parameters only used by the current
187 // architecture should go in StartNavigationParams. 193 // architecture should go in StartNavigationParams.
188 194
(...skipping 176 matching lines...) Expand 10 before | Expand all | Expand 10 after
365 ~NavigationParams(); 371 ~NavigationParams();
366 372
367 CommonNavigationParams common_params; 373 CommonNavigationParams common_params;
368 StartNavigationParams start_params; 374 StartNavigationParams start_params;
369 RequestNavigationParams request_params; 375 RequestNavigationParams request_params;
370 }; 376 };
371 377
372 } // namespace content 378 } // namespace content
373 379
374 #endif // CONTENT_COMMON_NAVIGATION_PARAMS_H_ 380 #endif // CONTENT_COMMON_NAVIGATION_PARAMS_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698