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

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

Issue 1310743003: Consistently use LoFi for an entire page (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: bengr comments and rebase Created 5 years, 2 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 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 #include "content/browser/frame_host/render_frame_host_impl.h" 5 #include "content/browser/frame_host/render_frame_host_impl.h"
6 6
7 #include "base/bind.h" 7 #include "base/bind.h"
8 #include "base/command_line.h" 8 #include "base/command_line.h"
9 #include "base/containers/hash_tables.h" 9 #include "base/containers/hash_tables.h"
10 #include "base/lazy_instance.h" 10 #include "base/lazy_instance.h"
(...skipping 1721 matching lines...) Expand 10 before | Expand all | Expand 10 after
1732 // done here either. 1732 // done here either.
1733 if (!common_params.url.SchemeIs(url::kJavaScriptScheme)) 1733 if (!common_params.url.SchemeIs(url::kJavaScriptScheme))
1734 frame_tree_node_->DidStartLoading(true); 1734 frame_tree_node_->DidStartLoading(true);
1735 } 1735 }
1736 1736
1737 void RenderFrameHostImpl::NavigateToInterstitialURL(const GURL& data_url) { 1737 void RenderFrameHostImpl::NavigateToInterstitialURL(const GURL& data_url) {
1738 DCHECK(data_url.SchemeIs(url::kDataScheme)); 1738 DCHECK(data_url.SchemeIs(url::kDataScheme));
1739 CommonNavigationParams common_params( 1739 CommonNavigationParams common_params(
1740 data_url, Referrer(), ui::PAGE_TRANSITION_LINK, 1740 data_url, Referrer(), ui::PAGE_TRANSITION_LINK,
1741 FrameMsg_Navigate_Type::NORMAL, false, false, base::TimeTicks::Now(), 1741 FrameMsg_Navigate_Type::NORMAL, false, false, base::TimeTicks::Now(),
1742 FrameMsg_UILoadMetricsReportType::NO_REPORT, GURL(), GURL()); 1742 FrameMsg_UILoadMetricsReportType::NO_REPORT, GURL(), GURL(),
1743 LOFI_OFF);
1743 if (base::CommandLine::ForCurrentProcess()->HasSwitch( 1744 if (base::CommandLine::ForCurrentProcess()->HasSwitch(
1744 switches::kEnableBrowserSideNavigation)) { 1745 switches::kEnableBrowserSideNavigation)) {
1745 CommitNavigation(nullptr, nullptr, common_params, 1746 CommitNavigation(nullptr, nullptr, common_params,
1746 RequestNavigationParams()); 1747 RequestNavigationParams());
1747 } else { 1748 } else {
1748 Navigate(common_params, StartNavigationParams(), RequestNavigationParams()); 1749 Navigate(common_params, StartNavigationParams(), RequestNavigationParams());
1749 } 1750 }
1750 } 1751 }
1751 1752
1752 void RenderFrameHostImpl::OpenURL(const FrameHostMsg_OpenURL_Params& params, 1753 void RenderFrameHostImpl::OpenURL(const FrameHostMsg_OpenURL_Params& params,
(...skipping 517 matching lines...) Expand 10 before | Expand all | Expand 10 after
2270 BrowserPluginInstanceIDToAXTreeID(value))); 2271 BrowserPluginInstanceIDToAXTreeID(value)));
2271 break; 2272 break;
2272 case AX_CONTENT_INT_ATTRIBUTE_LAST: 2273 case AX_CONTENT_INT_ATTRIBUTE_LAST:
2273 NOTREACHED(); 2274 NOTREACHED();
2274 break; 2275 break;
2275 } 2276 }
2276 } 2277 }
2277 } 2278 }
2278 2279
2279 } // namespace content 2280 } // namespace content
OLDNEW
« no previous file with comments | « content/browser/frame_host/navigator_impl.cc ('k') | content/browser/loader/resource_dispatcher_host_browsertest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698