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

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

Issue 2746043003: Fix user agent overridding with PlzNavigate. (Closed)
Patch Set: review comments Created 3 years, 9 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/interstitial_page_impl.h" 5 #include "content/browser/frame_host/interstitial_page_impl.h"
6 6
7 #include <utility> 7 #include <utility>
8 #include <vector> 8 #include <vector>
9 9
10 #include "base/bind.h" 10 #include "base/bind.h"
(...skipping 513 matching lines...) Expand 10 before | Expand all | Expand 10 after
524 // Hide the original RVH since we're showing the interstitial instead. 524 // Hide the original RVH since we're showing the interstitial instead.
525 rwh_view->Hide(); 525 rwh_view->Hide();
526 } 526 }
527 } 527 }
528 528
529 WebContents* InterstitialPageImpl::OpenURL(const OpenURLParams& params) { 529 WebContents* InterstitialPageImpl::OpenURL(const OpenURLParams& params) {
530 NOTREACHED(); 530 NOTREACHED();
531 return nullptr; 531 return nullptr;
532 } 532 }
533 533
534 const std::string& InterstitialPageImpl::GetUserAgentOverride() const {
535 return base::EmptyString();
536 }
537
534 RendererPreferences InterstitialPageImpl::GetRendererPrefs( 538 RendererPreferences InterstitialPageImpl::GetRendererPrefs(
535 BrowserContext* browser_context) const { 539 BrowserContext* browser_context) const {
536 delegate_->OverrideRendererPrefs(&renderer_preferences_); 540 delegate_->OverrideRendererPrefs(&renderer_preferences_);
537 return renderer_preferences_; 541 return renderer_preferences_;
538 } 542 }
539 543
540 void InterstitialPageImpl::RenderWidgetDeleted( 544 void InterstitialPageImpl::RenderWidgetDeleted(
541 RenderWidgetHostImpl* render_widget_host) { 545 RenderWidgetHostImpl* render_widget_host) {
542 // TODO(creis): Remove this method once we verify the shutdown path is sane. 546 // TODO(creis): Remove this method once we verify the shutdown path is sane.
543 CHECK(!web_contents_); 547 CHECK(!web_contents_);
(...skipping 423 matching lines...) Expand 10 before | Expand all | Expand 10 after
967 void InterstitialPageImpl::UpdateDeviceScaleFactor(double device_scale_factor) { 971 void InterstitialPageImpl::UpdateDeviceScaleFactor(double device_scale_factor) {
968 WebContentsImpl* web_contents_impl = 972 WebContentsImpl* web_contents_impl =
969 static_cast<WebContentsImpl*>(web_contents_); 973 static_cast<WebContentsImpl*>(web_contents_);
970 if (!web_contents_impl) 974 if (!web_contents_impl)
971 return; 975 return;
972 976
973 web_contents_impl->UpdateDeviceScaleFactor(device_scale_factor); 977 web_contents_impl->UpdateDeviceScaleFactor(device_scale_factor);
974 } 978 }
975 979
976 } // namespace content 980 } // namespace content
OLDNEW
« no previous file with comments | « content/browser/frame_host/interstitial_page_impl.h ('k') | content/browser/frame_host/navigation_request.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698