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

Side by Side Diff: net/proxy/dhcp_proxy_script_fetcher_factory_unittest.cc

Issue 10299002: Stop refcounting URLRequestContext. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Initialize to NULL Created 8 years, 7 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) 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 "net/proxy/dhcp_proxy_script_fetcher.h" 5 #include "net/proxy/dhcp_proxy_script_fetcher.h"
6 #include "net/proxy/dhcp_proxy_script_fetcher_factory.h" 6 #include "net/proxy/dhcp_proxy_script_fetcher_factory.h"
7 #include "net/url_request/url_request_test_util.h" 7 #include "net/url_request/url_request_test_util.h"
8 #include "testing/gtest/include/gtest/gtest.h" 8 #include "testing/gtest/include/gtest/gtest.h"
9 9
10 namespace net { 10 namespace net {
11 namespace { 11 namespace {
12 12
13 TEST(DhcpProxyScriptFetcherFactoryTest, DoNothingWhenDisabled) { 13 TEST(DhcpProxyScriptFetcherFactoryTest, DoNothingWhenDisabled) {
14 DhcpProxyScriptFetcherFactory factory; 14 DhcpProxyScriptFetcherFactory factory;
15 factory.set_enabled(false); 15 factory.set_enabled(false);
16 scoped_ptr<DhcpProxyScriptFetcher> fetcher(factory.Create(NULL)); 16 scoped_ptr<DhcpProxyScriptFetcher> fetcher(factory.Create(NULL));
17 EXPECT_EQ("", fetcher->GetFetcherName()); 17 EXPECT_EQ("", fetcher->GetFetcherName());
18 } 18 }
19 19
20 #if defined(OS_WIN) 20 #if defined(OS_WIN)
21 TEST(DhcpProxyScriptFetcherFactoryTest, WindowsFetcherOnWindows) { 21 TEST(DhcpProxyScriptFetcherFactoryTest, WindowsFetcherOnWindows) {
22 DhcpProxyScriptFetcherFactory factory; 22 DhcpProxyScriptFetcherFactory factory;
23 factory.set_enabled(true); 23 factory.set_enabled(true);
24 24
25 scoped_refptr<TestURLRequestContext> context(new TestURLRequestContext()); 25 scoped_ptr<TestURLRequestContext> context(new TestURLRequestContext());
26 scoped_ptr<DhcpProxyScriptFetcher> fetcher(factory.Create(context)); 26 scoped_ptr<DhcpProxyScriptFetcher> fetcher(factory.Create(context.get()));
27 EXPECT_EQ("win", fetcher->GetFetcherName()); 27 EXPECT_EQ("win", fetcher->GetFetcherName());
28 } 28 }
29 #endif // defined(OS_WIN) 29 #endif // defined(OS_WIN)
30 30
31 TEST(DhcpProxyScriptFetcherFactoryTest, IsSupported) { 31 TEST(DhcpProxyScriptFetcherFactoryTest, IsSupported) {
32 #if defined(OS_WIN) 32 #if defined(OS_WIN)
33 ASSERT_TRUE(DhcpProxyScriptFetcherFactory::IsSupported()); 33 ASSERT_TRUE(DhcpProxyScriptFetcherFactory::IsSupported());
34 #else 34 #else
35 ASSERT_FALSE(DhcpProxyScriptFetcherFactory::IsSupported()); 35 ASSERT_FALSE(DhcpProxyScriptFetcherFactory::IsSupported());
36 #endif // defined(OS_WIN) 36 #endif // defined(OS_WIN)
(...skipping 13 matching lines...) Expand all
50 factory.set_enabled(true); 50 factory.set_enabled(true);
51 #if defined(OS_WIN) 51 #if defined(OS_WIN)
52 EXPECT_TRUE(factory.enabled()); 52 EXPECT_TRUE(factory.enabled());
53 #else 53 #else
54 EXPECT_FALSE(factory.enabled()); 54 EXPECT_FALSE(factory.enabled());
55 #endif // defined(OS_WIN) 55 #endif // defined(OS_WIN)
56 } 56 }
57 57
58 } // namespace 58 } // namespace
59 } // namespace net 59 } // namespace net
OLDNEW
« no previous file with comments | « net/proxy/dhcp_proxy_script_adapter_fetcher_win_unittest.cc ('k') | net/proxy/dhcp_proxy_script_fetcher_win_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698