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

Side by Side Diff: chrome/browser/ui/cocoa/web_intent_sheet_controller_browsertest.mm

Issue 11414153: Remove legacy constrained window dialogs (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 years 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
(Empty)
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
3 // found in the LICENSE file.
4
5 #include "base/memory/scoped_ptr.h"
6 #include "chrome/test/base/in_process_browser_test.h"
7 #include "chrome/browser/ui/browser.h"
8 #include "chrome/browser/ui/browser_tabstrip.h"
9 #include "chrome/browser/ui/cocoa/web_intent_picker_cocoa.h"
10 #import "chrome/browser/ui/cocoa/web_intent_sheet_controller.h"
11 #include "chrome/browser/ui/intents/web_intent_picker_delegate.h"
12 #include "chrome/browser/ui/intents/web_intent_picker_delegate_mock.h"
13 #include "chrome/browser/ui/intents/web_intent_picker_model.h"
14 #include "testing/gmock/include/gmock/gmock.h"
15 #include "webkit/glue/web_intent_service_data.h"
16
17 class WebIntentSheetControllerBrowserTest : public InProcessBrowserTest {
18 public:
19 void CreateBubble(content::WebContents* web_contents);
20 void CreatePicker();
21
22 WebIntentPickerSheetController* controller_; // Weak, owns self.
23 NSWindow* window_; // Weak, owned by controller.
24 scoped_ptr<WebIntentPickerCocoa> picker_;
25 WebIntentPickerDelegateMock delegate_;
26 WebIntentPickerModel model_; // The model used by the picker
27 };
28
29 void WebIntentSheetControllerBrowserTest::CreateBubble(
30 content::WebContents* web_contents) {
31 picker_.reset(new WebIntentPickerCocoa(web_contents, &delegate_, &model_));
32
33 controller_ =
34 [[WebIntentPickerSheetController alloc] initWithPicker:picker_.get()];
35 window_ = [controller_ window];
36 [controller_ showWindow:nil];
37 }
38
39 void WebIntentSheetControllerBrowserTest::CreatePicker() {
40 picker_.reset(new WebIntentPickerCocoa());
41 picker_->delegate_ = &delegate_;
42 picker_->model_ = &model_;
43 window_ = nil;
44 controller_ = nil;
45 }
46
47 IN_PROC_BROWSER_TEST_F(WebIntentSheetControllerBrowserTest, CloseWillClose) {
48 CreateBubble(chrome::GetActiveWebContents(browser()));
49
50 EXPECT_CALL(delegate_, OnUserCancelledPickerDialog()).Times(0);
51 EXPECT_CALL(delegate_, OnClosing());
52 picker_->Close();
53
54 ignore_result(picker_.release()); // Closing |picker_| will destruct it.
55 }
56
57 IN_PROC_BROWSER_TEST_F(WebIntentSheetControllerBrowserTest,
58 DontCancelAfterServiceInvokation) {
59 CreateBubble(chrome::GetActiveWebContents(browser()));
60
61 GURL url;
62 model_.AddInstalledService(string16(), url,
63 webkit_glue::WebIntentServiceData::DISPOSITION_WINDOW);
64
65 EXPECT_CALL(delegate_, OnServiceChosen(
66 url, webkit_glue::WebIntentServiceData::DISPOSITION_WINDOW,
67 WebIntentPickerDelegate::kEnableDefaults));
68 EXPECT_CALL(delegate_, OnUserCancelledPickerDialog()).Times(0);
69 EXPECT_CALL(delegate_, OnClosing());
70
71 picker_->OnServiceChosen(0);
72 picker_->Close();
73
74 ignore_result(picker_.release()); // Closing |picker_| will destruct it.
75 }
76
77 IN_PROC_BROWSER_TEST_F(WebIntentSheetControllerBrowserTest,
78 OnCancelledWillSignalClose) {
79 CreatePicker();
80 EXPECT_CALL(delegate_, OnUserCancelledPickerDialog());
81 EXPECT_CALL(delegate_, OnClosing());
82 picker_->OnCancelled();
83
84 ignore_result(picker_.release()); // Closing |picker_| will destruct it.
85 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698