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

Unified Diff: ios/web/web_state/navigation_context_impl.mm

Issue 2845913003: Cleaned up unused methods from WebStateImpl and NavigationContextImpl. (Closed)
Patch Set: Fixed tests Created 3 years, 7 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « ios/web/web_state/navigation_context_impl.h ('k') | ios/web/web_state/navigation_context_impl_unittest.mm » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ios/web/web_state/navigation_context_impl.mm
diff --git a/ios/web/web_state/navigation_context_impl.mm b/ios/web/web_state/navigation_context_impl.mm
index 39df2fa68ea4f649af5bdadf219b39e19787f8ae..a1dc6f2dd73942609d48f54bc98e32372fe8fb79 100644
--- a/ios/web/web_state/navigation_context_impl.mm
+++ b/ios/web/web_state/navigation_context_impl.mm
@@ -21,40 +21,6 @@ NavigationContextImpl::CreateNavigationContext(WebState* web_state,
return result;
}
-// static
-std::unique_ptr<NavigationContextImpl>
-NavigationContextImpl::CreateNavigationContext(
- WebState* web_state,
- const GURL& url,
- const scoped_refptr<net::HttpResponseHeaders>& response_headers) {
- std::unique_ptr<NavigationContextImpl> resut(
- new NavigationContextImpl(web_state, url, false /* is_same_document */,
- false /* is_error_page */, response_headers));
- return resut;
-}
-
-// static
-std::unique_ptr<NavigationContextImpl>
-NavigationContextImpl::CreateSameDocumentNavigationContext(WebState* web_state,
- const GURL& url) {
- std::unique_ptr<NavigationContextImpl> result(new NavigationContextImpl(
- web_state, url, true /* is_same_document */, false /* is_error_page */,
- nullptr /* response_headers */));
- return result;
-}
-
-// static
-std::unique_ptr<NavigationContextImpl>
-NavigationContextImpl::CreateErrorPageNavigationContext(
- WebState* web_state,
- const GURL& url,
- const scoped_refptr<net::HttpResponseHeaders>& response_headers) {
- std::unique_ptr<NavigationContextImpl> result(
- new NavigationContextImpl(web_state, url, false /* is_same_document */,
- true /* is_error_page */, response_headers));
- return result;
-}
-
#ifndef NDEBUG
NSString* NavigationContextImpl::GetDescription() const {
return [NSString stringWithFormat:@"web::WebState: %ld, url: %s, "
« no previous file with comments | « ios/web/web_state/navigation_context_impl.h ('k') | ios/web/web_state/navigation_context_impl_unittest.mm » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698