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

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

Issue 2954503003: Don't show current RenderWidgetHostView while interstitial is showing. (Closed)
Patch Set: Created 3 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
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 526 matching lines...) Expand 10 before | Expand all | Expand 10 after
537 537
538 WebContents* InterstitialPageImpl::OpenURL(const OpenURLParams& params) { 538 WebContents* InterstitialPageImpl::OpenURL(const OpenURLParams& params) {
539 NOTREACHED(); 539 NOTREACHED();
540 return nullptr; 540 return nullptr;
541 } 541 }
542 542
543 const std::string& InterstitialPageImpl::GetUserAgentOverride() const { 543 const std::string& InterstitialPageImpl::GetUserAgentOverride() const {
544 return base::EmptyString(); 544 return base::EmptyString();
545 } 545 }
546 546
547 bool InterstitialPageImpl::ShowingInterstitialPage() const {
548 // An interstitial page never shows a second interstitial.
549 return false;
550 }
551
547 RendererPreferences InterstitialPageImpl::GetRendererPrefs( 552 RendererPreferences InterstitialPageImpl::GetRendererPrefs(
548 BrowserContext* browser_context) const { 553 BrowserContext* browser_context) const {
549 delegate_->OverrideRendererPrefs(&renderer_preferences_); 554 delegate_->OverrideRendererPrefs(&renderer_preferences_);
550 return renderer_preferences_; 555 return renderer_preferences_;
551 } 556 }
552 557
553 void InterstitialPageImpl::RenderWidgetDeleted( 558 void InterstitialPageImpl::RenderWidgetDeleted(
554 RenderWidgetHostImpl* render_widget_host) { 559 RenderWidgetHostImpl* render_widget_host) {
555 // TODO(creis): Remove this method once we verify the shutdown path is sane. 560 // TODO(creis): Remove this method once we verify the shutdown path is sane.
556 CHECK(!web_contents_); 561 CHECK(!web_contents_);
(...skipping 439 matching lines...) Expand 10 before | Expand all | Expand 10 after
996 RenderWidgetHostInputEventRouter* InterstitialPageImpl::GetInputEventRouter() { 1001 RenderWidgetHostInputEventRouter* InterstitialPageImpl::GetInputEventRouter() {
997 WebContentsImpl* web_contents_impl = 1002 WebContentsImpl* web_contents_impl =
998 static_cast<WebContentsImpl*>(web_contents_); 1003 static_cast<WebContentsImpl*>(web_contents_);
999 if (!web_contents_impl) 1004 if (!web_contents_impl)
1000 return nullptr; 1005 return nullptr;
1001 1006
1002 return web_contents_impl->GetInputEventRouter(); 1007 return web_contents_impl->GetInputEventRouter();
1003 } 1008 }
1004 1009
1005 } // namespace content 1010 } // namespace content
OLDNEW
« no previous file with comments | « content/browser/frame_host/interstitial_page_impl.h ('k') | content/browser/frame_host/interstitial_page_impl_browsertest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698