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

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

Issue 2668883002: Convert NavigationMetricsRecorder to use the new navigation callbacks. (Closed)
Patch Set: Created 3 years, 10 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 #ifndef CONTENT_BROWSER_FRAME_HOST_NAVIGATION_HANDLE_IMPL_H_ 5 #ifndef CONTENT_BROWSER_FRAME_HOST_NAVIGATION_HANDLE_IMPL_H_
6 #define CONTENT_BROWSER_FRAME_HOST_NAVIGATION_HANDLE_IMPL_H_ 6 #define CONTENT_BROWSER_FRAME_HOST_NAVIGATION_HANDLE_IMPL_H_
7 7
8 #include "content/public/browser/navigation_handle.h" 8 #include "content/public/browser/navigation_handle.h"
9 9
10 #include <stddef.h> 10 #include <stddef.h>
(...skipping 93 matching lines...) Expand 10 before | Expand all | Expand 10 after
104 bool IsPost() override; 104 bool IsPost() override;
105 const Referrer& GetReferrer() override; 105 const Referrer& GetReferrer() override;
106 bool HasUserGesture() override; 106 bool HasUserGesture() override;
107 ui::PageTransition GetPageTransition() override; 107 ui::PageTransition GetPageTransition() override;
108 bool IsExternalProtocol() override; 108 bool IsExternalProtocol() override;
109 net::Error GetNetErrorCode() override; 109 net::Error GetNetErrorCode() override;
110 RenderFrameHostImpl* GetRenderFrameHost() override; 110 RenderFrameHostImpl* GetRenderFrameHost() override;
111 bool IsSamePage() override; 111 bool IsSamePage() override;
112 bool HasCommitted() override; 112 bool HasCommitted() override;
113 bool IsErrorPage() override; 113 bool IsErrorPage() override;
114 const GURL& GetPreviousURL() override;
114 const net::HttpResponseHeaders* GetResponseHeaders() override; 115 const net::HttpResponseHeaders* GetResponseHeaders() override;
115 net::HttpResponseInfo::ConnectionInfo GetConnectionInfo() override; 116 net::HttpResponseInfo::ConnectionInfo GetConnectionInfo() override;
116 void Resume() override; 117 void Resume() override;
117 void CancelDeferredNavigation( 118 void CancelDeferredNavigation(
118 NavigationThrottle::ThrottleCheckResult result) override; 119 NavigationThrottle::ThrottleCheckResult result) override;
119 void RegisterThrottleForTesting( 120 void RegisterThrottleForTesting(
120 std::unique_ptr<NavigationThrottle> navigation_throttle) override; 121 std::unique_ptr<NavigationThrottle> navigation_throttle) override;
121 NavigationThrottle::ThrottleCheckResult CallWillStartRequestForTesting( 122 NavigationThrottle::ThrottleCheckResult CallWillStartRequestForTesting(
122 bool is_post, 123 bool is_post,
123 const Referrer& sanitized_referrer, 124 const Referrer& sanitized_referrer,
(...skipping 148 matching lines...) Expand 10 before | Expand all | Expand 10 after
272 // Called when the navigation is ready to be committed in 273 // Called when the navigation is ready to be committed in
273 // |render_frame_host|. This will update the |state_| and inform the 274 // |render_frame_host|. This will update the |state_| and inform the
274 // delegate. 275 // delegate.
275 void ReadyToCommitNavigation(RenderFrameHostImpl* render_frame_host); 276 void ReadyToCommitNavigation(RenderFrameHostImpl* render_frame_host);
276 277
277 // Called when the navigation was committed in |render_frame_host|. This will 278 // Called when the navigation was committed in |render_frame_host|. This will
278 // update the |state_|. 279 // update the |state_|.
279 void DidCommitNavigation( 280 void DidCommitNavigation(
280 const FrameHostMsg_DidCommitProvisionalLoad_Params& params, 281 const FrameHostMsg_DidCommitProvisionalLoad_Params& params,
281 bool same_page, 282 bool same_page,
283 const GURL& previous_url,
282 RenderFrameHostImpl* render_frame_host); 284 RenderFrameHostImpl* render_frame_host);
283 285
284 // Called during commit. Takes ownership of the embedder's NavigationData 286 // Called during commit. Takes ownership of the embedder's NavigationData
285 // instance. This NavigationData may have been cloned prior to being added 287 // instance. This NavigationData may have been cloned prior to being added
286 // here. 288 // here.
287 void set_navigation_data(std::unique_ptr<NavigationData> navigation_data) { 289 void set_navigation_data(std::unique_ptr<NavigationData> navigation_data) {
288 navigation_data_ = std::move(navigation_data); 290 navigation_data_ = std::move(navigation_data);
289 } 291 }
290 292
291 SSLStatus ssl_status() { return ssl_status_; } 293 SSLStatus ssl_status() { return ssl_status_; }
(...skipping 159 matching lines...) Expand 10 before | Expand all | Expand 10 after
451 453
452 // False by default unless the navigation started within a context menu. 454 // False by default unless the navigation started within a context menu.
453 bool started_from_context_menu_; 455 bool started_from_context_menu_;
454 456
455 // Stores the reload type, or NONE if it's not a reload. 457 // Stores the reload type, or NONE if it's not a reload.
456 ReloadType reload_type_; 458 ReloadType reload_type_;
457 459
458 GURL searchable_form_url_; 460 GURL searchable_form_url_;
459 std::string searchable_form_encoding_; 461 std::string searchable_form_encoding_;
460 462
463 GURL previous_url_;
464
461 base::WeakPtrFactory<NavigationHandleImpl> weak_factory_; 465 base::WeakPtrFactory<NavigationHandleImpl> weak_factory_;
462 466
463 DISALLOW_COPY_AND_ASSIGN(NavigationHandleImpl); 467 DISALLOW_COPY_AND_ASSIGN(NavigationHandleImpl);
464 }; 468 };
465 469
466 } // namespace content 470 } // namespace content
467 471
468 #endif // CONTENT_BROWSER_FRAME_HOST_NAVIGATION_HANDLE_IMPL_H_ 472 #endif // CONTENT_BROWSER_FRAME_HOST_NAVIGATION_HANDLE_IMPL_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698