[Webkit-unassigned] [Bug 54744] Add Prerendering Motivation information to ResourceRequest

Fri Feb 18 12:45:44 PST 2011

https://bugs.webkit.org/show_bug.cgi?id=54744

--- Comment #9 from Gavin Peters <gavinp at chromium.org>  2011-02-18 12:45:44 PST ---
(In reply to comment #5)
> (From update of attachment 82955 [details])
> "triggeredByPrerendering" or "forPrerendering" seem better than "hasPrenderingMotivation" (which is quite a mouthful).

You are right.  I've uploaded a new patch.

> 
> I agree with Adam, seems this would be better to store off in Chromium land instead of in WebCore if possible.
> 
> Unless WebCore really needs to know about it being a prerender?

WebCore does not need to know about it being a prerender.

I think both you and Adam are right, and 49113 is the right way forward; requiring WebKit changes for load information to pass through each time Chrome gets more clever about request life is not a sustainable way forward.

Adam asked me to promise that I'd work on 49113 to make this ugliness go away: By these words I promise that I will hack at 49113.  This will come up in Chrome networking again as we improve prioritization of requests, since information about the origination of requests (a hidden tab?  a visible tab? ...) will motivate that.  As well I know other folks here who have similar FavIcon changes for Auth they want.  I'll try and talk them into waiting on 49113.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.