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

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

Issue 2359503002: Let TRACE_EVENT_XXX_WITH_TIMESTAMP accept TimeTicks instead of int64_t (Closed)
Patch Set: Rebase Created 4 years, 3 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 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 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_handle_impl.h" 5 #include "content/browser/frame_host/navigation_handle_impl.h"
6 6
7 #include <utility> 7 #include <utility>
8 8
9 #include "base/logging.h" 9 #include "base/logging.h"
10 #include "content/browser/browsing_data/clear_site_data_throttle.h" 10 #include "content/browser/browsing_data/clear_site_data_throttle.h"
(...skipping 64 matching lines...) Expand 10 before | Expand all | Expand 10 after
75 next_index_(0), 75 next_index_(0),
76 navigation_start_(navigation_start), 76 navigation_start_(navigation_start),
77 pending_nav_entry_id_(pending_nav_entry_id), 77 pending_nav_entry_id_(pending_nav_entry_id),
78 request_context_type_(REQUEST_CONTEXT_TYPE_UNSPECIFIED) { 78 request_context_type_(REQUEST_CONTEXT_TYPE_UNSPECIFIED) {
79 DCHECK(!navigation_start.is_null()); 79 DCHECK(!navigation_start.is_null());
80 GetDelegate()->DidStartNavigation(this); 80 GetDelegate()->DidStartNavigation(this);
81 81
82 if (IsInMainFrame()) { 82 if (IsInMainFrame()) {
83 TRACE_EVENT_ASYNC_BEGIN_WITH_TIMESTAMP1( 83 TRACE_EVENT_ASYNC_BEGIN_WITH_TIMESTAMP1(
84 "navigation", "Navigation StartToCommit", this, 84 "navigation", "Navigation StartToCommit", this,
85 navigation_start.ToInternalValue(), "Initial URL", url_.spec()); 85 navigation_start, "Initial URL", url_.spec());
86 } 86 }
87 } 87 }
88 88
89 NavigationHandleImpl::~NavigationHandleImpl() { 89 NavigationHandleImpl::~NavigationHandleImpl() {
90 GetDelegate()->DidFinishNavigation(this); 90 GetDelegate()->DidFinishNavigation(this);
91 91
92 // Cancel the navigation on the IO thread if the NavigationHandle is being 92 // Cancel the navigation on the IO thread if the NavigationHandle is being
93 // destroyed in the middle of the NavigationThrottles checks. 93 // destroyed in the middle of the NavigationThrottles checks.
94 if (!IsBrowserSideNavigationEnabled() && !complete_callback_.is_null()) 94 if (!IsBrowserSideNavigationEnabled() && !complete_callback_.is_null())
95 RunCompleteCallback(NavigationThrottle::CANCEL_AND_IGNORE); 95 RunCompleteCallback(NavigationThrottle::CANCEL_AND_IGNORE);
(...skipping 504 matching lines...) Expand 10 before | Expand all | Expand 10 after
600 throttles_to_register.push_back(std::move(clear_site_data_throttle)); 600 throttles_to_register.push_back(std::move(clear_site_data_throttle));
601 601
602 if (throttles_to_register.size() > 0) { 602 if (throttles_to_register.size() > 0) {
603 throttles_.insert(throttles_.begin(), throttles_to_register.begin(), 603 throttles_.insert(throttles_.begin(), throttles_to_register.begin(),
604 throttles_to_register.end()); 604 throttles_to_register.end());
605 throttles_to_register.weak_clear(); 605 throttles_to_register.weak_clear();
606 } 606 }
607 } 607 }
608 608
609 } // namespace content 609 } // namespace content
OLDNEW
« no previous file with comments | « content/browser/devtools/devtools_frame_trace_recorder.cc ('k') | content/browser/frame_host/navigation_request.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698