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

Side by Side Diff: content/public/test/test_download_request_handler.cc

Issue 2405483002: Make the request initiator Optional (Closed)
Patch Set: Addressed comments + rebase Created 4 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
OLDNEW
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 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 "content/public/test/test_download_request_handler.h" 5 #include "content/public/test/test_download_request_handler.h"
6 6
7 #include <inttypes.h> 7 #include <inttypes.h>
8 8
9 #include <memory> 9 #include <memory>
10 #include <utility> 10 #include <utility>
(...skipping 309 matching lines...) Expand 10 before | Expand all | Expand 10 after
320 return bytes_to_copy; 320 return bytes_to_copy;
321 } 321 }
322 322
323 void TestDownloadRequestHandler::PartialResponseJob::ReportCompletedRequest() { 323 void TestDownloadRequestHandler::PartialResponseJob::ReportCompletedRequest() {
324 if (interceptor_.get()) { 324 if (interceptor_.get()) {
325 std::unique_ptr<CompletedRequest> completed_request(new CompletedRequest); 325 std::unique_ptr<CompletedRequest> completed_request(new CompletedRequest);
326 completed_request->transferred_byte_count = read_byte_count_; 326 completed_request->transferred_byte_count = read_byte_count_;
327 completed_request->request_headers = request()->extra_request_headers(); 327 completed_request->request_headers = request()->extra_request_headers();
328 completed_request->referrer = request()->referrer(); 328 completed_request->referrer = request()->referrer();
329 completed_request->referrer_policy = request()->referrer_policy(); 329 completed_request->referrer_policy = request()->referrer_policy();
330 completed_request->initiator = request()->initiator(); 330 completed_request->initiator = request()->initiator().has_value()
331 ? request()->initiator().value()
332 : url::Origin();
331 completed_request->first_party_for_cookies = 333 completed_request->first_party_for_cookies =
332 request()->first_party_for_cookies(); 334 request()->first_party_for_cookies();
333 completed_request->first_party_url_policy = 335 completed_request->first_party_url_policy =
334 request()->first_party_url_policy(); 336 request()->first_party_url_policy();
335 interceptor_->AddCompletedRequest(std::move(completed_request)); 337 interceptor_->AddCompletedRequest(std::move(completed_request));
336 } 338 }
337 } 339 }
338 340
339 // static 341 // static
340 void TestDownloadRequestHandler::PartialResponseJob:: 342 void TestDownloadRequestHandler::PartialResponseJob::
(...skipping 346 matching lines...) Expand 10 before | Expand all | Expand 10 after
687 base::RunLoop run_loop; 689 base::RunLoop run_loop;
688 BrowserThread::PostTaskAndReply( 690 BrowserThread::PostTaskAndReply(
689 BrowserThread::IO, FROM_HERE, 691 BrowserThread::IO, FROM_HERE,
690 base::Bind(&Interceptor::GetAndResetCompletedRequests, interceptor_, 692 base::Bind(&Interceptor::GetAndResetCompletedRequests, interceptor_,
691 requests), 693 requests),
692 run_loop.QuitClosure()); 694 run_loop.QuitClosure());
693 run_loop.Run(); 695 run_loop.Run();
694 } 696 }
695 697
696 } // namespace content 698 } // namespace content
OLDNEW
« no previous file with comments | « content/public/common/url_fetcher.h ('k') | extensions/browser/guest_view/web_view/web_ui/web_ui_url_fetcher.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698