New issue
Advanced search Search tips

Issue 824854 link

Starred by 1 user

Issue metadata

Status: Assigned
Owner:
Cc:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug

Blocked on:
issue 598073

Blocking:
issue 824840



Sign in to add a comment

WebPackageRequestHandler should work on the UI thread

Project Member Reported by jam@chromium.org, Mar 22 2018

Issue description

To improve performance and avoid extra/unnecessary thread hops (see parent bug), we want to move NavigationURLLoaderNetworkService and its dependent classes to the UI thread. This bug tracks converting WebPackageRequestHandler.
 

Comment 1 by jam@chromium.org, Mar 22 2018

Labels: -OS-Windows -Pri-3 Pri-2

Comment 2 by jam@chromium.org, Mar 22 2018

Blocking: -805851 824840

Comment 3 by horo@chromium.org, Mar 23 2018

Cc: kinuko@chromium.org ksakamoto@chromium.org kouhei@chromium.org

Comment 4 by kinuko@chromium.org, Mar 23 2018

To give extra context, we want to do this when Network Service is enabled (i.e. where we don't need to worry about interactions with RDHI etc)

Comment 5 by jam@chromium.org, Mar 23 2018

Kinuko: can we run it all the time on UI thread (e.g. even when NS is disabled), that way we can convert to NS without changing all the request handlers and throttles?

Comment 6 by jam@chromium.org, May 16 2018

Blockedon: 598073
btw we'll hold off on this until the network service ships. at that point, it'll be a lot easier (as the rest of the code will only be on UI).

Sign in to add a comment