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

Unified Diff: chrome/browser/ui/intents/web_intent_picker_controller_unittest.cc

Issue 7782015: content: Move web intent constrained window to TabContentsWrapper. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 9 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/ui/intents/web_intent_picker_controller_unittest.cc
diff --git a/chrome/browser/ui/intents/web_intent_picker_controller_unittest.cc b/chrome/browser/ui/intents/web_intent_picker_controller_unittest.cc
index 89a2e1413328f3f4d032131f9fb63dbaefd40433..407d709cac60c58c164cac8e2bd50a8458535287 100644
--- a/chrome/browser/ui/intents/web_intent_picker_controller_unittest.cc
+++ b/chrome/browser/ui/intents/web_intent_picker_controller_unittest.cc
@@ -87,16 +87,16 @@ class WebIntentPickerMock : public WebIntentPicker {
class WebIntentPickerFactoryMock : public WebIntentPickerFactory {
public:
MOCK_METHOD2(Create,
- WebIntentPicker*(TabContents* tab_contents,
+ WebIntentPicker*(TabContentsWrapper* wrapper,
WebIntentPickerDelegate* delegate));
MOCK_METHOD1(ClosePicker, void(WebIntentPicker* picker));
};
class TestWebIntentPickerController : public WebIntentPickerController {
public:
- TestWebIntentPickerController(TabContents* tab_contents,
+ TestWebIntentPickerController(TabContentsWrapper* wrapper,
WebIntentPickerFactory* factory)
- : WebIntentPickerController(tab_contents, factory) {
+ : WebIntentPickerController(wrapper, factory) {
}
MOCK_METHOD1(OnServiceChosen, void(size_t index));
@@ -133,8 +133,8 @@ class WebIntentPickerControllerTest : public TabContentsWrapperTestHarness {
registry->Initialize(web_data_service_);
picker_factory_ = new WebIntentPickerFactoryMock();
- controller_.reset(new TestWebIntentPickerController(contents(),
- picker_factory_));
+ controller_.reset(new TestWebIntentPickerController(contents_wrapper(),
+ picker_factory_));
}
virtual void TearDown() {
« no previous file with comments | « chrome/browser/ui/intents/web_intent_picker_controller.cc ('k') | chrome/browser/ui/intents/web_intent_picker_factory.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698