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

Side by Side Diff: chrome/browser/intents/register_intent_handler_infobar_delegate_unittest.cc

Issue 8417043: Add webkit_glue namespace. Improve some variable and test names. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix merge error Created 9 years, 1 month 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) 2011 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2011 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 "base/utf_string_conversions.h" 5 #include "base/utf_string_conversions.h"
6 #include "chrome/browser/intents/register_intent_handler_infobar_delegate.h" 6 #include "chrome/browser/intents/register_intent_handler_infobar_delegate.h"
7 #include "chrome/browser/intents/web_intents_registry.h" 7 #include "chrome/browser/intents/web_intents_registry.h"
8 #include "chrome/browser/intents/web_intents_registry_factory.h" 8 #include "chrome/browser/intents/web_intents_registry_factory.h"
9 #include "chrome/browser/ui/tab_contents/tab_contents_wrapper.h" 9 #include "chrome/browser/ui/tab_contents/tab_contents_wrapper.h"
10 #include "chrome/browser/ui/tab_contents/test_tab_contents_wrapper.h" 10 #include "chrome/browser/ui/tab_contents/test_tab_contents_wrapper.h"
11 #include "chrome/test/base/testing_profile.h" 11 #include "chrome/test/base/testing_profile.h"
12 #include "content/browser/tab_contents/test_tab_contents.h" 12 #include "content/browser/tab_contents/test_tab_contents.h"
13 #include "content/test/test_browser_thread.h" 13 #include "content/test/test_browser_thread.h"
14 #include "testing/gmock/include/gmock/gmock.h" 14 #include "testing/gmock/include/gmock/gmock.h"
15 #include "testing/gtest/include/gtest/gtest.h" 15 #include "testing/gtest/include/gtest/gtest.h"
16 #include "webkit/glue/web_intent_service_data.h" 16 #include "webkit/glue/web_intent_service_data.h"
17 17
18 namespace { 18 namespace {
19 19
20 class MockWebIntentsRegistry : public WebIntentsRegistry { 20 class MockWebIntentsRegistry : public WebIntentsRegistry {
21 public: 21 public:
22 MOCK_METHOD1(RegisterIntentProvider, void(const WebIntentServiceData&)); 22 MOCK_METHOD1(RegisterIntentProvider,
23 void(const webkit_glue::WebIntentServiceData&));
23 }; 24 };
24 25
25 ProfileKeyedService* BuildMockWebIntentsRegistry(Profile* profile) { 26 ProfileKeyedService* BuildMockWebIntentsRegistry(Profile* profile) {
26 return new MockWebIntentsRegistry; 27 return new MockWebIntentsRegistry;
27 } 28 }
28 29
29 MockWebIntentsRegistry* BuildForProfile(Profile* profile) { 30 MockWebIntentsRegistry* BuildForProfile(Profile* profile) {
30 return static_cast<MockWebIntentsRegistry*>( 31 return static_cast<MockWebIntentsRegistry*>(
31 WebIntentsRegistryFactory::GetInstance()->SetTestingFactoryAndUse( 32 WebIntentsRegistryFactory::GetInstance()->SetTestingFactoryAndUse(
32 profile, BuildMockWebIntentsRegistry)); 33 profile, BuildMockWebIntentsRegistry));
(...skipping 20 matching lines...) Expand all
53 54
54 MockWebIntentsRegistry* web_intents_registry_; 55 MockWebIntentsRegistry* web_intents_registry_;
55 56
56 private: 57 private:
57 content::TestBrowserThread ui_thread_; 58 content::TestBrowserThread ui_thread_;
58 59
59 DISALLOW_COPY_AND_ASSIGN(RegisterIntentHandlerInfoBarDelegateTest); 60 DISALLOW_COPY_AND_ASSIGN(RegisterIntentHandlerInfoBarDelegateTest);
60 }; 61 };
61 62
62 TEST_F(RegisterIntentHandlerInfoBarDelegateTest, Accept) { 63 TEST_F(RegisterIntentHandlerInfoBarDelegateTest, Accept) {
63 WebIntentServiceData service; 64 webkit_glue::WebIntentServiceData service;
64 service.service_url = GURL("google.com"); 65 service.service_url = GURL("google.com");
65 service.action = ASCIIToUTF16("http://webintents.org/share"); 66 service.action = ASCIIToUTF16("http://webintents.org/share");
66 service.type = ASCIIToUTF16("text/url"); 67 service.type = ASCIIToUTF16("text/url");
67 RegisterIntentHandlerInfoBarDelegate delegate( 68 RegisterIntentHandlerInfoBarDelegate delegate(
68 contents_wrapper()->infobar_tab_helper(), 69 contents_wrapper()->infobar_tab_helper(),
69 WebIntentsRegistryFactory::GetForProfile(profile()), 70 WebIntentsRegistryFactory::GetForProfile(profile()),
70 service); 71 service);
71 72
72 EXPECT_CALL(*web_intents_registry_, RegisterIntentProvider(service)); 73 EXPECT_CALL(*web_intents_registry_, RegisterIntentProvider(service));
73 delegate.Accept(); 74 delegate.Accept();
74 } 75 }
75 76
76 } // namespace 77 } // namespace
OLDNEW
« no previous file with comments | « chrome/browser/intents/register_intent_handler_infobar_delegate.cc ('k') | chrome/browser/intents/web_intents_registry.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698