New issue
Advanced search Search tips

Issue 911027 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Dec 20
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Task

Blocking:
issue 894819



Sign in to add a comment

SyncLoadContext should work with data pipe

Project Member Reported by yhirano@chromium.org, Dec 3

Issue description

Status: Assigned (was: Untriaged)
Status: Started (was: Assigned)
Project Member

Comment 3 by bugdroid1@chromium.org, Dec 19

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/f62b472cb8c427d1b5a17128c2451dc7d6d610f9

commit f62b472cb8c427d1b5a17128c2451dc7d6d610f9
Author: Makoto Shimazu <shimazu@chromium.org>
Date: Wed Dec 19 07:49:43 2018

Make SyncLoadContext work with mojo data pipe

Previously SyncLoadContext::OnStartLoadingResponseBody() is used for downloading
to a blob, but this CL makes it usable in both of downloading to a blob and
downloading to a SyncLoadResponse.

Bug:  911027 
Change-Id: I9287f3b4e6908aaf55b5d49ab8d085792357cfde
Reviewed-on: https://chromium-review.googlesource.com/c/1377960
Commit-Queue: Makoto Shimazu <shimazu@chromium.org>
Reviewed-by: Yutaka Hirano <yhirano@chromium.org>
Cr-Commit-Position: refs/heads/master@{#617761}
[modify] https://crrev.com/f62b472cb8c427d1b5a17128c2451dc7d6d610f9/content/renderer/loader/resource_dispatcher.h
[modify] https://crrev.com/f62b472cb8c427d1b5a17128c2451dc7d6d610f9/content/renderer/loader/sync_load_context.cc
[modify] https://crrev.com/f62b472cb8c427d1b5a17128c2451dc7d6d610f9/content/renderer/loader/sync_load_context.h
[add] https://crrev.com/f62b472cb8c427d1b5a17128c2451dc7d6d610f9/content/renderer/loader/sync_load_context_unittest.cc
[modify] https://crrev.com/f62b472cb8c427d1b5a17128c2451dc7d6d610f9/content/test/BUILD.gn

Status: Fixed (was: Started)

Sign in to add a comment