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

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

Issue 2638423008: Convert ContentFaviconDriver to use the new navigation callbacks. (Closed)
Patch Set: add comment Created 3 years, 11 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 117 matching lines...) Expand 10 before | Expand all | Expand 10 after
128 bool new_method_is_post, 128 bool new_method_is_post,
129 const GURL& new_referrer_url, 129 const GURL& new_referrer_url,
130 bool new_is_external_protocol) override; 130 bool new_is_external_protocol) override;
131 NavigationThrottle::ThrottleCheckResult CallWillProcessResponseForTesting( 131 NavigationThrottle::ThrottleCheckResult CallWillProcessResponseForTesting(
132 RenderFrameHost* render_frame_host, 132 RenderFrameHost* render_frame_host,
133 const std::string& raw_response_header) override; 133 const std::string& raw_response_header) override;
134 void CallDidCommitNavigationForTesting(const GURL& url) override; 134 void CallDidCommitNavigationForTesting(const GURL& url) override;
135 bool WasStartedFromContextMenu() const override; 135 bool WasStartedFromContextMenu() const override;
136 const GURL& GetSearchableFormURL() override; 136 const GURL& GetSearchableFormURL() override;
137 const std::string& GetSearchableFormEncoding() override; 137 const std::string& GetSearchableFormEncoding() override;
138 ReloadType GetReloadType() override;
138 const GlobalRequestID& GetGlobalRequestID() override; 139 const GlobalRequestID& GetGlobalRequestID() override;
139 140
140 NavigationData* GetNavigationData() override; 141 NavigationData* GetNavigationData() override;
141 142
142 // The NavigatorDelegate to notify/query for various navigation events. 143 // The NavigatorDelegate to notify/query for various navigation events.
143 // Normally this is the WebContents, except if this NavigationHandle was 144 // Normally this is the WebContents, except if this NavigationHandle was
144 // created during a navigation to an interstitial page. In this case it will 145 // created during a navigation to an interstitial page. In this case it will
145 // be the InterstitialPage itself. 146 // be the InterstitialPage itself.
146 // 147 //
147 // Note: due to the interstitial navigation case, all calls that can possibly 148 // Note: due to the interstitial navigation case, all calls that can possibly
(...skipping 295 matching lines...) Expand 10 before | Expand all | Expand 10 after
443 // A callback to run on the IO thread if the navigation transfers. 444 // A callback to run on the IO thread if the navigation transfers.
444 base::Closure transfer_callback_; 445 base::Closure transfer_callback_;
445 446
446 // Whether the navigation ended up being a download or a stream. 447 // Whether the navigation ended up being a download or a stream.
447 bool is_download_; 448 bool is_download_;
448 bool is_stream_; 449 bool is_stream_;
449 450
450 // False by default unless the navigation started within a context menu. 451 // False by default unless the navigation started within a context menu.
451 bool started_from_context_menu_; 452 bool started_from_context_menu_;
452 453
454 // Stores the reload type, or NONE if it's not a reload.
455 ReloadType reload_type_;
456
453 GURL searchable_form_url_; 457 GURL searchable_form_url_;
454 std::string searchable_form_encoding_; 458 std::string searchable_form_encoding_;
455 459
456 base::WeakPtrFactory<NavigationHandleImpl> weak_factory_; 460 base::WeakPtrFactory<NavigationHandleImpl> weak_factory_;
457 461
458 DISALLOW_COPY_AND_ASSIGN(NavigationHandleImpl); 462 DISALLOW_COPY_AND_ASSIGN(NavigationHandleImpl);
459 }; 463 };
460 464
461 } // namespace content 465 } // namespace content
462 466
463 #endif // CONTENT_BROWSER_FRAME_HOST_NAVIGATION_HANDLE_IMPL_H_ 467 #endif // CONTENT_BROWSER_FRAME_HOST_NAVIGATION_HANDLE_IMPL_H_
OLDNEW
« no previous file with comments | « components/favicon/content/content_favicon_driver.cc ('k') | content/browser/frame_host/navigation_handle_impl.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698