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

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

Issue 317703004: Simplify AreURLsInPageNavigation (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 6 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 | Annotate | Revision Log
« no previous file with comments | « no previous file | content/browser/frame_host/navigation_controller_impl.cc » ('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 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 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_CONTROLLER_IMPL_H_ 5 #ifndef CONTENT_BROWSER_FRAME_HOST_NAVIGATION_CONTROLLER_IMPL_H_
6 #define CONTENT_BROWSER_FRAME_HOST_NAVIGATION_CONTROLLER_IMPL_H_ 6 #define CONTENT_BROWSER_FRAME_HOST_NAVIGATION_CONTROLLER_IMPL_H_
7 7
8 #include "base/callback.h" 8 #include "base/callback.h"
9 #include "base/compiler_specific.h" 9 #include "base/compiler_specific.h"
10 #include "base/gtest_prod_util.h" 10 #include "base/gtest_prod_util.h"
(...skipping 149 matching lines...) Expand 10 before | Expand all | Expand 10 after
160 // be a reload, while only a different ref would be in-page (pages can't clear 160 // be a reload, while only a different ref would be in-page (pages can't clear
161 // refs without reload, only change to "#" which we don't count as empty). 161 // refs without reload, only change to "#" which we don't count as empty).
162 // 162 //
163 // The situation is made murkier by history.replaceState(), which could 163 // The situation is made murkier by history.replaceState(), which could
164 // provide the same URL as part of an in-page navigation, not a reload. So 164 // provide the same URL as part of an in-page navigation, not a reload. So
165 // we need to let the (untrustworthy) renderer resolve the ambiguity, but 165 // we need to let the (untrustworthy) renderer resolve the ambiguity, but
166 // only when the URLs are on the same origin. 166 // only when the URLs are on the same origin.
167 bool IsURLInPageNavigation( 167 bool IsURLInPageNavigation(
168 const GURL& url, 168 const GURL& url,
169 bool renderer_says_in_page, 169 bool renderer_says_in_page,
170 NavigationType navigation_type) const; 170 RenderFrameHost* rfh) const;
171 171
172 // Sets the SessionStorageNamespace for the given |partition_id|. This is 172 // Sets the SessionStorageNamespace for the given |partition_id|. This is
173 // used during initialization of a new NavigationController to allow 173 // used during initialization of a new NavigationController to allow
174 // pre-population of the SessionStorageNamespace objects. Session restore, 174 // pre-population of the SessionStorageNamespace objects. Session restore,
175 // prerendering, and the implementaion of window.open() are the primary users 175 // prerendering, and the implementaion of window.open() are the primary users
176 // of this API. 176 // of this API.
177 // 177 //
178 // Calling this function when a SessionStorageNamespace has already been 178 // Calling this function when a SessionStorageNamespace has already been
179 // associated with a |partition_id| will CHECK() fail. 179 // associated with a |partition_id| will CHECK() fail.
180 void SetSessionStorageNamespace( 180 void SetSessionStorageNamespace(
(...skipping 232 matching lines...) Expand 10 before | Expand all | Expand 10 after
413 TimeSmoother time_smoother_; 413 TimeSmoother time_smoother_;
414 414
415 scoped_ptr<NavigationEntryScreenshotManager> screenshot_manager_; 415 scoped_ptr<NavigationEntryScreenshotManager> screenshot_manager_;
416 416
417 DISALLOW_COPY_AND_ASSIGN(NavigationControllerImpl); 417 DISALLOW_COPY_AND_ASSIGN(NavigationControllerImpl);
418 }; 418 };
419 419
420 } // namespace content 420 } // namespace content
421 421
422 #endif // CONTENT_BROWSER_FRAME_HOST_NAVIGATION_CONTROLLER_IMPL_H_ 422 #endif // CONTENT_BROWSER_FRAME_HOST_NAVIGATION_CONTROLLER_IMPL_H_
OLDNEW
« no previous file with comments | « no previous file | content/browser/frame_host/navigation_controller_impl.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698