New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 775402 link

Starred by 3 users

Issue metadata

Status: Available
Owner: ----
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Task

Blocked on:
issue 736308
issue 752028

Blocking:
issue 860403



Sign in to add a comment

Onionsoup: content/renderer/fetcher

Project Member Reported by toyoshim@chromium.org, Oct 17 2017

Issue description

mojofication was finished, but still code exists in content/renderer.

files in content/renderer/fetcher/:
 - associated_resource_fetcher_impl.{cc,h}
 - manifest_fetcher.{cc,h}
 - multi_resolution_image_resource_fetcher.{cc, h}
 - resource_fetcher_browsertest.cc
 - resource_fetcher_impl.{cc, h}

Onionsoup wants moving these files to Blink

Notes:
 - ResourceFetcher is content public API and used from chrome/ now. This can be replaced with CORSURLLoader or SimpleURLLoader.
 - AssociatedResourceFetcher may be replaced with CORSURLLoader.
 - ManifestFetcher uses AssociatedResourceFetcher now, but once it's changed to use CORSURLLoader, remaining works will be a part of content/renderer/manifest/ Onionsoup efforts.
 - Ditto on MultiResourceImageResourceFetcher, and it can be a part of content/renderer/image_downloader/ efforts.

As a loading team, we will focus on CORSURLLoader impl first, and that would make this task easier.
 
Please note that in the end state we will not have //content/child - so we want to move all of this into blink.

Thanks!
Description: Show this description
Thank you for correcting. I slightly modified the bug description to say all should be moved to blink.
Blocking: 860403

Sign in to add a comment