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

Side by Side Diff: chrome/browser/extensions/api/web_request/web_request_api_unittest.cc

Issue 11931024: Removed static factories for data, ftp, file, and about jobs. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: sync (r198785) Created 7 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) 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 <map> 5 #include <map>
6 #include <queue> 6 #include <queue>
7 7
8 #include "base/basictypes.h" 8 #include "base/basictypes.h"
9 #include "base/bind.h" 9 #include "base/bind.h"
10 #include "base/callback.h" 10 #include "base/callback.h"
(...skipping 10 matching lines...) Expand all
21 #include "base/strings/string_piece.h" 21 #include "base/strings/string_piece.h"
22 #include "base/time.h" 22 #include "base/time.h"
23 #include "base/utf_string_conversions.h" 23 #include "base/utf_string_conversions.h"
24 #include "chrome/browser/content_settings/cookie_settings.h" 24 #include "chrome/browser/content_settings/cookie_settings.h"
25 #include "chrome/browser/extensions/api/web_request/upload_data_presenter.h" 25 #include "chrome/browser/extensions/api/web_request/upload_data_presenter.h"
26 #include "chrome/browser/extensions/api/web_request/web_request_api.h" 26 #include "chrome/browser/extensions/api/web_request/web_request_api.h"
27 #include "chrome/browser/extensions/api/web_request/web_request_api_constants.h" 27 #include "chrome/browser/extensions/api/web_request/web_request_api_constants.h"
28 #include "chrome/browser/extensions/api/web_request/web_request_api_helpers.h" 28 #include "chrome/browser/extensions/api/web_request/web_request_api_helpers.h"
29 #include "chrome/browser/extensions/event_router_forwarder.h" 29 #include "chrome/browser/extensions/event_router_forwarder.h"
30 #include "chrome/browser/extensions/extension_warning_set.h" 30 #include "chrome/browser/extensions/extension_warning_set.h"
31 #include "chrome/browser/net/about_protocol_handler.h"
31 #include "chrome/browser/net/chrome_network_delegate.h" 32 #include "chrome/browser/net/chrome_network_delegate.h"
32 #include "chrome/common/extensions/extension_messages.h" 33 #include "chrome/common/extensions/extension_messages.h"
33 #include "chrome/common/extensions/features/feature.h" 34 #include "chrome/common/extensions/features/feature.h"
34 #include "chrome/common/pref_names.h" 35 #include "chrome/common/pref_names.h"
35 #include "chrome/test/base/testing_browser_process.h" 36 #include "chrome/test/base/testing_browser_process.h"
36 #include "chrome/test/base/testing_pref_service_syncable.h" 37 #include "chrome/test/base/testing_pref_service_syncable.h"
37 #include "chrome/test/base/testing_profile.h" 38 #include "chrome/test/base/testing_profile.h"
38 #include "chrome/test/base/testing_profile_manager.h" 39 #include "chrome/test/base/testing_profile_manager.h"
40 #include "content/public/common/url_constants.h"
39 #include "content/public/test/test_browser_thread.h" 41 #include "content/public/test/test_browser_thread.h"
40 #include "net/base/auth.h" 42 #include "net/base/auth.h"
41 #include "net/base/capturing_net_log.h" 43 #include "net/base/capturing_net_log.h"
42 #include "net/base/net_util.h" 44 #include "net/base/net_util.h"
43 #include "net/base/upload_bytes_element_reader.h" 45 #include "net/base/upload_bytes_element_reader.h"
44 #include "net/base/upload_data_stream.h" 46 #include "net/base/upload_data_stream.h"
45 #include "net/base/upload_file_element_reader.h" 47 #include "net/base/upload_file_element_reader.h"
46 #include "net/dns/mock_host_resolver.h" 48 #include "net/dns/mock_host_resolver.h"
49 #include "net/url_request/url_request_job_factory_impl.h"
47 #include "net/url_request/url_request_test_util.h" 50 #include "net/url_request/url_request_test_util.h"
48 #include "testing/gtest/include/gtest/gtest-message.h" 51 #include "testing/gtest/include/gtest/gtest-message.h"
49 #include "testing/gtest/include/gtest/gtest.h" 52 #include "testing/gtest/include/gtest/gtest.h"
50 53
51 namespace helpers = extension_web_request_api_helpers; 54 namespace helpers = extension_web_request_api_helpers;
52 namespace keys = extension_web_request_api_constants; 55 namespace keys = extension_web_request_api_constants;
53 56
54 using base::BinaryValue; 57 using base::BinaryValue;
55 using base::DictionaryValue; 58 using base::DictionaryValue;
56 using base::ListValue; 59 using base::ListValue;
(...skipping 166 matching lines...) Expand 10 before | Expand all | Expand 10 after
223 base::WeakPtrFactory<TestIPCSender> ipc_sender_factory(&ipc_sender_); 226 base::WeakPtrFactory<TestIPCSender> ipc_sender_factory(&ipc_sender_);
224 ExtensionWebRequestEventRouter::GetInstance()->AddEventListener( 227 ExtensionWebRequestEventRouter::GetInstance()->AddEventListener(
225 &profile_, extension1_id, extension1_id, kEventName, kEventName + "/1", 228 &profile_, extension1_id, extension1_id, kEventName, kEventName + "/1",
226 filter, ExtensionWebRequestEventRouter::ExtraInfoSpec::BLOCKING, -1, -1, 229 filter, ExtensionWebRequestEventRouter::ExtraInfoSpec::BLOCKING, -1, -1,
227 ipc_sender_factory.GetWeakPtr()); 230 ipc_sender_factory.GetWeakPtr());
228 ExtensionWebRequestEventRouter::GetInstance()->AddEventListener( 231 ExtensionWebRequestEventRouter::GetInstance()->AddEventListener(
229 &profile_, extension2_id, extension2_id, kEventName, kEventName + "/2", 232 &profile_, extension2_id, extension2_id, kEventName, kEventName + "/2",
230 filter, ExtensionWebRequestEventRouter::ExtraInfoSpec::BLOCKING, -1, -1, 233 filter, ExtensionWebRequestEventRouter::ExtraInfoSpec::BLOCKING, -1, -1,
231 ipc_sender_factory.GetWeakPtr()); 234 ipc_sender_factory.GetWeakPtr());
232 235
236 net::URLRequestJobFactoryImpl job_factory;
237 job_factory.SetProtocolHandler(
238 chrome::kAboutScheme,
239 new chrome_browser_net::AboutProtocolHandler());
240 context_->set_job_factory(&job_factory);
241
233 GURL redirect_url("about:redirected"); 242 GURL redirect_url("about:redirected");
234 GURL not_chosen_redirect_url("about:not_chosen"); 243 GURL not_chosen_redirect_url("about:not_chosen");
235 244
236 net::URLRequest request(GURL("about:blank"), &delegate_, context_.get()); 245 net::URLRequest request(GURL("about:blank"), &delegate_, context_.get());
237 { 246 {
238 // onBeforeRequest will be dispatched twice initially. The second response - 247 // onBeforeRequest will be dispatched twice initially. The second response -
239 // the redirect - should win, since it has a later |install_time|. The 248 // the redirect - should win, since it has a later |install_time|. The
240 // redirect will dispatch another pair of onBeforeRequest. There, the first 249 // redirect will dispatch another pair of onBeforeRequest. There, the first
241 // response should win (later |install_time|). 250 // response should win (later |install_time|).
242 ExtensionWebRequestEventRouter::EventResponse* response = NULL; 251 ExtensionWebRequestEventRouter::EventResponse* response = NULL;
(...skipping 1883 matching lines...) Expand 10 before | Expand all | Expand 10 after
2126 EXPECT_TRUE(credentials_set); 2135 EXPECT_TRUE(credentials_set);
2127 EXPECT_FALSE(auth3.Empty()); 2136 EXPECT_FALSE(auth3.Empty());
2128 EXPECT_EQ(username, auth1.username()); 2137 EXPECT_EQ(username, auth1.username());
2129 EXPECT_EQ(password, auth1.password()); 2138 EXPECT_EQ(password, auth1.password());
2130 EXPECT_EQ(1u, warning_set.size()); 2139 EXPECT_EQ(1u, warning_set.size());
2131 EXPECT_TRUE(HasWarning(warning_set, "extid2")); 2140 EXPECT_TRUE(HasWarning(warning_set, "extid2"));
2132 EXPECT_EQ(3u, capturing_net_log.GetSize()); 2141 EXPECT_EQ(3u, capturing_net_log.GetSize());
2133 } 2142 }
2134 2143
2135 } // namespace extensions 2144 } // namespace extensions
OLDNEW
« no previous file with comments | « chrome/browser/custom_handlers/protocol_handler_registry_unittest.cc ('k') | chrome/browser/io_thread.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698