New issue
Advanced search Search tips

Issue 712312 link

Starred by 1 user
Status: WontFix
Owner:
petewil@chromium.org
Closed: Aug 24
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

Minor Refactor in ChromeResourceHostDispatcher

Project Member Reported by petewil@chromium.org, Apr 17 2017 
We're adding a new method in ChromeResourceHostDispatcher for dispatching resource beginning operations, NotifyUIThreadOfRequestStarted().

If possible, we should unify the SafeBrowsing request and the PageLoadMetrics request so that we only have to do one thread transition to the UI thread when a request begins. (At the same time, look for other opportunities in case any other thread switches need to happen).

Comment 1 by petewil@chromium.org, Apr 18 2017 

Note from csharrison: In servicing your TODO it would be
nice to hook up into WebContentsObserver like the notification when a response
comes in.

Comment 2 by davidben@chromium.org, Apr 19 2017

Status: Available (was: Untriaged)

Comment 3 by davidben@chromium.org, Apr 19 2017

Status: Assigned (was: Available)
(Oh, there's already an owner. I was just removing it from Untriaged.)

Comment 4 by petewil@chromium.org, Aug 24

Status: WontFix (was: Assigned)
We're removing the code that prompted this refactoring request instead.
►

Sign in to add a comment