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

Description Gavin Peters 2012-04-25 09:52:00 PDT
Add new ENABLE_LINK_PRERENDER define to control the Prerendering API
Comment 1 Gavin Peters 2012-04-25 09:57:50 PDT
Created attachment 138829 [details]
Patch
Comment 2 Gavin Peters 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
Comment 3 Gavin Peters 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.
Comment 4 WebKit Review Bot 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>
Comment 5 WebKit Review Bot 2012-04-27 09:37:56 PDT
All reviewed patches have been landed.  Closing bug.