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

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

Issue 2666193002: Switch RenderViewContextMenu to use RequestOpenURL (Closed)
Patch Set: Address Charlie's comments Created 3 years, 10 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 508 matching lines...) Expand 10 before | Expand all | Expand 10 after
519 if (rwh_view) { 519 if (rwh_view) {
520 // If the page has focus, focus the interstitial. 520 // If the page has focus, focus the interstitial.
521 if (rwh_view->HasFocus()) 521 if (rwh_view->HasFocus())
522 Focus(); 522 Focus();
523 523
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) {
530 NOTREACHED();
531 return nullptr;
532 }
533
529 RendererPreferences InterstitialPageImpl::GetRendererPrefs( 534 RendererPreferences InterstitialPageImpl::GetRendererPrefs(
530 BrowserContext* browser_context) const { 535 BrowserContext* browser_context) const {
531 delegate_->OverrideRendererPrefs(&renderer_preferences_); 536 delegate_->OverrideRendererPrefs(&renderer_preferences_);
532 return renderer_preferences_; 537 return renderer_preferences_;
533 } 538 }
534 539
535 void InterstitialPageImpl::RenderWidgetDeleted( 540 void InterstitialPageImpl::RenderWidgetDeleted(
536 RenderWidgetHostImpl* render_widget_host) { 541 RenderWidgetHostImpl* render_widget_host) {
537 // TODO(creis): Remove this method once we verify the shutdown path is sane. 542 // TODO(creis): Remove this method once we verify the shutdown path is sane.
538 CHECK(!web_contents_); 543 CHECK(!web_contents_);
(...skipping 418 matching lines...) Expand 10 before | Expand all | Expand 10 after
957 void InterstitialPageImpl::UpdateDeviceScaleFactor(double device_scale_factor) { 962 void InterstitialPageImpl::UpdateDeviceScaleFactor(double device_scale_factor) {
958 WebContentsImpl* web_contents_impl = 963 WebContentsImpl* web_contents_impl =
959 static_cast<WebContentsImpl*>(web_contents_); 964 static_cast<WebContentsImpl*>(web_contents_);
960 if (!web_contents_impl) 965 if (!web_contents_impl)
961 return; 966 return;
962 967
963 web_contents_impl->UpdateDeviceScaleFactor(device_scale_factor); 968 web_contents_impl->UpdateDeviceScaleFactor(device_scale_factor);
964 } 969 }
965 970
966 } // namespace content 971 } // namespace content
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698