Bug 84871

Summary: Add new ENABLE_LINK_PRERENDER define to control the Prerendering API
Product: WebKit Reporter: Gavin Peters <gavinp>
Component: New BugsAssignee: Gavin Peters <gavinp>
Status: RESOLVED FIXED    
Severity: Normal CC: abarth, mrowe, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 85005    
Attachments:
Description Flags
Patch none

Gavin Peters
Reported 2012-04-25 09:52:00 PDT
Add new ENABLE_LINK_PRERENDER define to control the Prerendering API
Attachments
Patch (30.63 KB, patch)
2012-04-25 09:57 PDT, Gavin Peters 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Gavin Peters
Comment 1 2012-04-25 09:57:50 PDT
Created attachment 138829 [details] Patch
Gavin Peters
Comment 2 2012-04-25 10:00:57 PDT
Prerendering is currently covered by the ENABLE_LINK_PREFETCH macro, but the new Prerendering API separates it from prefetching. Having separate include guards lets ports enable prefetching, a relatively easy change, without needing to build the infrastructure for prerendering, which is considerably more complicated. This was carved out of the architecture review at https://bugs.webkit.org/show_bug.cgi?id=82478 , and got the LGTM from Adam in GitHub at https://github.com/gavinp/git.webkit.org/commit/fa83ece77731e42e280aaa3dd1ad594443666220
Gavin Peters
Comment 3 2012-04-25 10:01:20 PDT
Comment on attachment 138829 [details] Patch I'll wait for EWS, and probably a ping from Mark Rowe before landing this.
WebKit Review Bot
Comment 4 2012-04-27 09:37:51 PDT
Comment on attachment 138829 [details] Patch Clearing flags on attachment: 138829 Committed r115443: <http://trac.webkit.org/changeset/115443>
WebKit Review Bot
Comment 5 2012-04-27 09:37:56 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.