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

Side by Side Diff: content/browser/frame_host/navigation_request.cc

Issue 2449553002: Remove dead DocumentState metrics tracking code (Closed)
Patch Set: fix tests 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 unified diff | Download patch
« no previous file with comments | « content/browser/frame_host/navigation_entry_impl.cc ('k') | content/common/frame_messages.h » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 #include "content/browser/frame_host/navigation_request.h" 5 #include "content/browser/frame_host/navigation_request.h"
6 6
7 #include <utility> 7 #include <utility>
8 8
9 #include "content/browser/child_process_security_policy_impl.h" 9 #include "content/browser/child_process_security_policy_impl.h"
10 #include "content/browser/devtools/render_frame_devtools_agent_host.h" 10 #include "content/browser/devtools/render_frame_devtools_agent_host.h"
(...skipping 169 matching lines...) Expand 10 before | Expand all | Expand 10 after
180 // TODO(clamy): Check if some PageState should be provided here. 180 // TODO(clamy): Check if some PageState should be provided here.
181 // TODO(clamy): See how we should handle override of the user agent when the 181 // TODO(clamy): See how we should handle override of the user agent when the
182 // navigation may start in a renderer and commit in another one. 182 // navigation may start in a renderer and commit in another one.
183 // TODO(clamy): See if the navigation start time should be measured in the 183 // TODO(clamy): See if the navigation start time should be measured in the
184 // renderer and sent to the browser instead of being measured here. 184 // renderer and sent to the browser instead of being measured here.
185 // TODO(clamy): The pending history list offset should be properly set. 185 // TODO(clamy): The pending history list offset should be properly set.
186 RequestNavigationParams request_params( 186 RequestNavigationParams request_params(
187 false, // is_overriding_user_agent 187 false, // is_overriding_user_agent
188 std::vector<GURL>(), // redirects 188 std::vector<GURL>(), // redirects
189 false, // can_load_local_resources 189 false, // can_load_local_resources
190 base::Time::Now(), // request_time
191 PageState(), // page_state 190 PageState(), // page_state
192 0, // nav_entry_id 191 0, // nav_entry_id
193 false, // is_same_document_history_load 192 false, // is_same_document_history_load
194 false, // is_history_navigation_in_new_child 193 false, // is_history_navigation_in_new_child
195 std::map<std::string, bool>(), // subframe_unique_names 194 std::map<std::string, bool>(), // subframe_unique_names
196 frame_tree_node->has_committed_real_load(), 195 frame_tree_node->has_committed_real_load(),
197 false, // intended_as_new_entry 196 false, // intended_as_new_entry
198 -1, // pending_history_list_offset 197 -1, // pending_history_list_offset
199 current_history_list_offset, current_history_list_length, 198 current_history_list_offset, current_history_list_length,
200 false, // is_view_source 199 false, // is_view_source
(...skipping 389 matching lines...) Expand 10 before | Expand all | Expand 10 after
590 DCHECK_EQ(request_params_.has_user_gesture, begin_params_.has_user_gesture); 589 DCHECK_EQ(request_params_.has_user_gesture, begin_params_.has_user_gesture);
591 590
592 render_frame_host->CommitNavigation(response_.get(), std::move(body_), 591 render_frame_host->CommitNavigation(response_.get(), std::move(body_),
593 common_params_, request_params_, 592 common_params_, request_params_,
594 is_view_source_); 593 is_view_source_);
595 594
596 frame_tree_node_->ResetNavigationRequest(true); 595 frame_tree_node_->ResetNavigationRequest(true);
597 } 596 }
598 597
599 } // namespace content 598 } // namespace content
OLDNEW
« no previous file with comments | « content/browser/frame_host/navigation_entry_impl.cc ('k') | content/common/frame_messages.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698