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

Unified Diff: chrome/browser/external_tab/external_tab_container_win.cc

Issue 10536058: TabContentsWrapper -> TabContents, part 6. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: GetOwningTabContentsForWebContents Created 8 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/external_tab/external_tab_container_win.cc
diff --git a/chrome/browser/external_tab/external_tab_container_win.cc b/chrome/browser/external_tab/external_tab_container_win.cc
index 227c6562cd130d6f1191438703cbe17810e20a78..38020253497cf77d27157866df6c9d0b175ecaa8 100644
--- a/chrome/browser/external_tab/external_tab_container_win.cc
+++ b/chrome/browser/external_tab/external_tab_container_win.cc
@@ -33,7 +33,7 @@
#include "chrome/browser/ui/browser.h"
#include "chrome/browser/ui/browser_dialogs.h"
#include "chrome/browser/ui/browser_window.h"
-#include "chrome/browser/ui/tab_contents/tab_contents_wrapper.h"
+#include "chrome/browser/ui/tab_contents/tab_contents.h"
#include "chrome/browser/ui/views/infobars/infobar_container_view.h"
#include "chrome/common/automation_messages.h"
#include "chrome/common/chrome_constants.h"
@@ -150,7 +150,7 @@ bool ExternalTabContainer::Init(Profile* profile,
DWORD style,
bool load_requests_via_automation,
bool handle_top_level_requests,
- TabContentsWrapper* existing_contents,
+ TabContents* existing_contents,
const GURL& initial_url,
const GURL& referrer,
bool infobars_enabled,
@@ -185,7 +185,7 @@ bool ExternalTabContainer::Init(Profile* profile,
} else {
WebContents* new_contents = WebContents::Create(
profile, NULL, MSG_ROUTING_NONE, NULL, NULL);
- tab_contents_.reset(new TabContentsWrapper(new_contents));
+ tab_contents_.reset(new TabContents(new_contents));
}
if (!infobars_enabled)
@@ -450,23 +450,23 @@ void ExternalTabContainer::AddNewContents(WebContents* source,
// Make sure that ExternalTabContainer instance is initialized with
// an unwrapped Profile.
- scoped_ptr<TabContentsWrapper> wrapper(new TabContentsWrapper(new_contents));
+ scoped_ptr<TabContents> tab_contents(new TabContents(new_contents));
bool result = new_container->Init(
- wrapper->profile()->GetOriginalProfile(),
+ tab_contents->profile()->GetOriginalProfile(),
NULL,
initial_pos,
WS_CHILD,
load_requests_via_automation_,
handle_top_level_requests_,
- wrapper.get(),
+ tab_contents.get(),
GURL(),
GURL(),
true,
route_all_top_level_navigations_);
if (result) {
- Profile* profile = wrapper->profile();
- wrapper.release(); // Ownership has been transferred.
+ Profile* profile = tab_contents->profile();
+ tab_contents.release(); // Ownership has been transferred.
if (route_all_top_level_navigations_) {
return;
}
@@ -521,8 +521,8 @@ void ExternalTabContainer::MoveContents(WebContents* source,
automation_->Send(new AutomationMsg_MoveWindow(tab_handle_, pos));
}
-TabContentsWrapper* ExternalTabContainer::GetConstrainingContentsWrapper(
- TabContentsWrapper* source) {
+TabContents* ExternalTabContainer::GetConstrainingContentsWrapper(
+ TabContents* source) {
return source;
}
@@ -717,7 +717,7 @@ void ExternalTabContainer::ShowRepostFormWarningDialog(
WebContents* source) {
browser::ShowTabModalConfirmDialog(
new RepostFormWarningController(source),
- TabContentsWrapper::GetCurrentWrapperForContents(source));
+ TabContents::GetOwningTabContentsForWebContents(source));
}
void ExternalTabContainer::RunFileChooser(

Powered by Google App Engine
This is Rietveld 408576698