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

Unified Diff: content/child/web_url_loader_impl_unittest.cc

Issue 2345253002: Remove content::RequestInfo (Closed)
Patch Set: Rebase Created 4 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
« no previous file with comments | « content/child/web_url_loader_impl.cc ('k') | content/common/resource_messages.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/child/web_url_loader_impl_unittest.cc
diff --git a/content/child/web_url_loader_impl_unittest.cc b/content/child/web_url_loader_impl_unittest.cc
index 55276552e0a0e7bb80a5d77f98024a801b4521de..3895a60b03a1676eff6cfbc0f65705ed29fd6c3a 100644
--- a/content/child/web_url_loader_impl_unittest.cc
+++ b/content/child/web_url_loader_impl_unittest.cc
@@ -20,7 +20,6 @@
#include "base/time/default_tick_clock.h"
#include "base/time/time.h"
#include "content/child/request_extra_data.h"
-#include "content/child/request_info.h"
#include "content/child/resource_dispatcher.h"
#include "content/child/sync_load_response.h"
#include "content/public/child/fixed_received_data.h"
@@ -67,24 +66,27 @@ class TestResourceDispatcher : public ResourceDispatcher {
// TestDispatcher implementation:
- void StartSync(const RequestInfo& request_info,
- ResourceRequestBodyImpl* request_body,
+ void StartSync(std::unique_ptr<ResourceRequest> request,
+ int routing_id,
SyncLoadResponse* response,
blink::WebURLRequest::LoadingIPCType ipc_type,
mojom::URLLoaderFactory* url_loader_factory) override {
*response = sync_load_response_;
}
- int StartAsync(const RequestInfo& request_info,
- ResourceRequestBodyImpl* request_body,
- std::unique_ptr<RequestPeer> peer,
- blink::WebURLRequest::LoadingIPCType ipc_type,
- mojom::URLLoaderFactory* url_loader_factory) override {
+ int StartAsync(
+ std::unique_ptr<ResourceRequest> request,
+ int routing_id,
+ scoped_refptr<base::SingleThreadTaskRunner> loading_task_runner,
+ const GURL& frame_origin,
+ std::unique_ptr<RequestPeer> peer,
+ blink::WebURLRequest::LoadingIPCType ipc_type,
+ mojom::URLLoaderFactory* url_loader_factory) override {
EXPECT_FALSE(peer_);
EXPECT_EQ(blink::WebURLRequest::LoadingIPCType::ChromeIPC, ipc_type);
peer_ = std::move(peer);
- url_ = request_info.url;
- stream_url_ = request_info.resource_body_stream_url;
+ url_ = request->url;
+ stream_url_ = request->resource_body_stream_url;
return 1;
}
@@ -639,7 +641,7 @@ TEST_F(WebURLLoaderImplTest, BrowserSideNavigationCommit) {
client()->loader()->loadAsynchronously(request, client());
- // The stream url should have been added to the RequestInfo.
+ // The stream url should have been added to the ResourceRequest.
ASSERT_TRUE(peer());
EXPECT_EQ(kNavigationURL, dispatcher()->url());
EXPECT_EQ(kStreamURL, dispatcher()->stream_url());
« no previous file with comments | « content/child/web_url_loader_impl.cc ('k') | content/common/resource_messages.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698