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

Unified Diff: chrome/browser/ui/webui/web_dialog_web_contents_delegate.cc

Issue 10796049: Move WebDialogWebContentsDelegate to ui/web_dialogs. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 years, 5 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
Index: chrome/browser/ui/webui/web_dialog_web_contents_delegate.cc
diff --git a/chrome/browser/ui/webui/web_dialog_web_contents_delegate.cc b/chrome/browser/ui/webui/web_dialog_web_contents_delegate.cc
deleted file mode 100644
index 1742e2282089dcb34fc6383931bde99574d4b09d..0000000000000000000000000000000000000000
--- a/chrome/browser/ui/webui/web_dialog_web_contents_delegate.cc
+++ /dev/null
@@ -1,59 +0,0 @@
-// Copyright (c) 2012 The Chromium Authors. All rights reserved.
-// Use of this source code is governed by a BSD-style license that can be
-// found in the LICENSE file.
-
-#include "chrome/browser/ui/webui/web_dialog_web_contents_delegate.h"
-
-#include "base/logging.h"
-#include "content/public/browser/web_contents.h"
-
-using content::BrowserContext;
-using content::OpenURLParams;
-using content::WebContents;
-
-// Incognito profiles are not long-lived, so we always want to store a
-// non-incognito profile.
-//
-// TODO(akalin): Should we make it so that we have a default incognito
-// profile that's long-lived? Of course, we'd still have to clear it out
-// when all incognito browsers close.
-WebDialogWebContentsDelegate::WebDialogWebContentsDelegate(
- content::BrowserContext* browser_context,
- WebContentsHandler* handler)
- : browser_context_(browser_context),
- handler_(handler) {
- CHECK(handler_.get());
-}
-
-WebDialogWebContentsDelegate::~WebDialogWebContentsDelegate() {
-}
-
-void WebDialogWebContentsDelegate::Detach() {
- browser_context_ = NULL;
-}
-
-WebContents* WebDialogWebContentsDelegate::OpenURLFromTab(
- WebContents* source, const OpenURLParams& params) {
- return handler_->OpenURLFromTab(browser_context_, source, params);
-}
-
-void WebDialogWebContentsDelegate::AddNewContents(
- WebContents* source, WebContents* new_contents,
- WindowOpenDisposition disposition, const gfx::Rect& initial_pos,
- bool user_gesture) {
- handler_->AddNewContents(browser_context_, source, new_contents, disposition,
- initial_pos, user_gesture);
-}
-
-bool WebDialogWebContentsDelegate::IsPopupOrPanel(
- const WebContents* source) const {
- // This needs to return true so that we are allowed to be resized by our
- // contents.
- return true;
-}
-
-bool WebDialogWebContentsDelegate::ShouldAddNavigationToHistory(
- const history::HistoryAddPageArgs& add_page_args,
- content::NavigationType navigation_type) {
- return false;
-}

Powered by Google App Engine
This is Rietveld 408576698