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

Side by Side Diff: chrome/browser/printing/print_preview_tab_controller_unittest.cc

Issue 10911351: Switch PrintViewManager, PrintPreviewMessageHandler to use WebContentsUserData. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: rebase, fix typo Created 8 years, 3 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 | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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 "chrome/browser/prefs/pref_service.h" 5 #include "chrome/browser/prefs/pref_service.h"
6 #include "chrome/browser/printing/print_preview_tab_controller.h" 6 #include "chrome/browser/printing/print_preview_tab_controller.h"
7 #include "chrome/browser/printing/print_view_manager.h" 7 #include "chrome/browser/printing/print_view_manager.h"
8 #include "chrome/browser/ui/browser_commands.h" 8 #include "chrome/browser/ui/browser_commands.h"
9 #include "chrome/browser/ui/browser_list.h" 9 #include "chrome/browser/ui/browser_list.h"
10 #include "chrome/browser/ui/browser_tabstrip.h" 10 #include "chrome/browser/ui/browser_tabstrip.h"
(...skipping 41 matching lines...) Expand 10 before | Expand all | Expand 10 after
52 EXPECT_EQ(1, browser()->tab_count()); 52 EXPECT_EQ(1, browser()->tab_count());
53 53
54 // Create a reference to initiator tab contents. 54 // Create a reference to initiator tab contents.
55 TabContents* initiator_tab = chrome::GetActiveTabContents(browser()); 55 TabContents* initiator_tab = chrome::GetActiveTabContents(browser());
56 56
57 printing::PrintPreviewTabController* tab_controller = 57 printing::PrintPreviewTabController* tab_controller =
58 printing::PrintPreviewTabController::GetInstance(); 58 printing::PrintPreviewTabController::GetInstance();
59 ASSERT_TRUE(tab_controller); 59 ASSERT_TRUE(tab_controller);
60 60
61 // Get the preview tab for initiator tab. 61 // Get the preview tab for initiator tab.
62 initiator_tab->print_view_manager()->PrintPreviewNow(); 62 printing::PrintViewManager::FromWebContents(initiator_tab->web_contents())->
63 PrintPreviewNow();
63 TabContents* preview_tab = 64 TabContents* preview_tab =
64 tab_controller->GetOrCreatePreviewTab(initiator_tab); 65 tab_controller->GetOrCreatePreviewTab(initiator_tab);
65 66
66 // New print preview tab is created. 67 // New print preview tab is created.
67 EXPECT_EQ(1, browser()->tab_count()); 68 EXPECT_EQ(1, browser()->tab_count());
68 EXPECT_NE(initiator_tab, preview_tab); 69 EXPECT_NE(initiator_tab, preview_tab);
69 70
70 // Get the print preview tab for initiator tab. 71 // Get the print preview tab for initiator tab.
71 TabContents* new_preview_tab = 72 TabContents* new_preview_tab =
72 tab_controller->GetOrCreatePreviewTab(initiator_tab); 73 tab_controller->GetOrCreatePreviewTab(initiator_tab);
(...skipping 20 matching lines...) Expand all
93 chrome::NewTab(browser()); 94 chrome::NewTab(browser());
94 TabContents* tab_contents_2 = chrome::GetActiveTabContents(browser()); 95 TabContents* tab_contents_2 = chrome::GetActiveTabContents(browser());
95 ASSERT_TRUE(tab_contents_2); 96 ASSERT_TRUE(tab_contents_2);
96 EXPECT_EQ(2, browser()->tab_count()); 97 EXPECT_EQ(2, browser()->tab_count());
97 98
98 printing::PrintPreviewTabController* tab_controller = 99 printing::PrintPreviewTabController* tab_controller =
99 printing::PrintPreviewTabController::GetInstance(); 100 printing::PrintPreviewTabController::GetInstance();
100 ASSERT_TRUE(tab_controller); 101 ASSERT_TRUE(tab_controller);
101 102
102 // Create preview tab for |tab_contents_1| 103 // Create preview tab for |tab_contents_1|
103 tab_contents_1->print_view_manager()->PrintPreviewNow(); 104 printing::PrintViewManager::FromWebContents(tab_contents_1->web_contents())->
105 PrintPreviewNow();
104 TabContents* preview_tab_1 = 106 TabContents* preview_tab_1 =
105 tab_controller->GetOrCreatePreviewTab(tab_contents_1); 107 tab_controller->GetOrCreatePreviewTab(tab_contents_1);
106 108
107 EXPECT_NE(tab_contents_1, preview_tab_1); 109 EXPECT_NE(tab_contents_1, preview_tab_1);
108 EXPECT_EQ(2, browser()->tab_count()); 110 EXPECT_EQ(2, browser()->tab_count());
109 111
110 // Create preview tab for |tab_contents_2| 112 // Create preview tab for |tab_contents_2|
111 tab_contents_2->print_view_manager()->PrintPreviewNow(); 113 printing::PrintViewManager::FromWebContents(tab_contents_2->web_contents())->
114 PrintPreviewNow();
112 TabContents* preview_tab_2 = 115 TabContents* preview_tab_2 =
113 tab_controller->GetOrCreatePreviewTab(tab_contents_2); 116 tab_controller->GetOrCreatePreviewTab(tab_contents_2);
114 117
115 EXPECT_NE(tab_contents_2, preview_tab_2); 118 EXPECT_NE(tab_contents_2, preview_tab_2);
116 // 2 initiator tab and 2 preview tabs exist in the same browser. 119 // 2 initiator tab and 2 preview tabs exist in the same browser.
117 // The preview tabs are constrained in their respective initiator tabs. 120 // The preview tabs are constrained in their respective initiator tabs.
118 EXPECT_EQ(2, browser()->tab_count()); 121 EXPECT_EQ(2, browser()->tab_count());
119 122
120 TabStripModel* model = browser()->tab_strip_model(); 123 TabStripModel* model = browser()->tab_strip_model();
121 ASSERT_TRUE(model); 124 ASSERT_TRUE(model);
(...skipping 22 matching lines...) Expand all
144 EXPECT_EQ(1, browser()->tab_count()); 147 EXPECT_EQ(1, browser()->tab_count());
145 148
146 // Create a reference to initiator tab contents. 149 // Create a reference to initiator tab contents.
147 TabContents* initiator_tab = chrome::GetActiveTabContents(browser()); 150 TabContents* initiator_tab = chrome::GetActiveTabContents(browser());
148 151
149 printing::PrintPreviewTabController* tab_controller = 152 printing::PrintPreviewTabController* tab_controller =
150 printing::PrintPreviewTabController::GetInstance(); 153 printing::PrintPreviewTabController::GetInstance();
151 ASSERT_TRUE(tab_controller); 154 ASSERT_TRUE(tab_controller);
152 155
153 // Get the preview tab for initiator tab. 156 // Get the preview tab for initiator tab.
154 initiator_tab->print_view_manager()->PrintPreviewNow(); 157 printing::PrintViewManager::FromWebContents(initiator_tab->web_contents())->
158 PrintPreviewNow();
155 TabContents* preview_tab = 159 TabContents* preview_tab =
156 tab_controller->GetOrCreatePreviewTab(initiator_tab); 160 tab_controller->GetOrCreatePreviewTab(initiator_tab);
157 161
158 // New print preview tab is created. Current focus is on preview tab. 162 // New print preview tab is created. Current focus is on preview tab.
159 EXPECT_EQ(1, browser()->tab_count()); 163 EXPECT_EQ(1, browser()->tab_count());
160 EXPECT_NE(initiator_tab, preview_tab); 164 EXPECT_NE(initiator_tab, preview_tab);
161 165
162 // Clear the initiator tab details associated with the preview tab. 166 // Clear the initiator tab details associated with the preview tab.
163 tab_controller->EraseInitiatorTabInfo(preview_tab); 167 tab_controller->EraseInitiatorTabInfo(preview_tab);
164 168
165 // Get the print preview tab for initiator tab. 169 // Get the print preview tab for initiator tab.
166 TabContents* new_preview_tab = 170 TabContents* new_preview_tab =
167 tab_controller->GetOrCreatePreviewTab(initiator_tab); 171 tab_controller->GetOrCreatePreviewTab(initiator_tab);
168 172
169 // New preview tab is created. 173 // New preview tab is created.
170 EXPECT_EQ(1, browser()->tab_count()); 174 EXPECT_EQ(1, browser()->tab_count());
171 EXPECT_NE(new_preview_tab, preview_tab); 175 EXPECT_NE(new_preview_tab, preview_tab);
172 } 176 }
OLDNEW
« no previous file with comments | « chrome/browser/printing/print_preview_tab_controller_browsertest.cc ('k') | chrome/browser/printing/print_view_manager.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698