Remove PrerenderResourceThrottle and related infrastructure.

Remove PrerenderResourceThrottle and related infrastructure.

This was needed in the past to prevent double GETs for the top-level resource of a prerendered page (once when the page started being prerendered, another time when the page was navigated to but before the prerender was swapped in).

Now that prerenders are swapped in at OpenURL time, the second GET will not happen and this class is not needed anymore.

BUG=122410
TEST=existing tests, do a prerender and confirm that there is not a double-GET of the top-level resource by looking at net-internals.


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=131602

[cbentzel, 2012-04-06 19:26:31]

mmenke: Review the whole thing.
darin: You're in chrome/OWNERS and originally wrote the throttle. Just expecting
a rubberstamp here.

[mmenke, 2012-04-06 19:40:04]

http://codereview.chromium.org/10006041/diff/1/chrome/browser/prerender/prere...
File chrome/browser/prerender/prerender_tracker.h (right):

http://codereview.chromium.org/10006041/diff/1/chrome/browser/prerender/prere...
chrome/browser/prerender/prerender_tracker.h:13: #include
"base/callback_forward.h"
We don't need this anymore.

http://codereview.chromium.org/10006041/diff/1/chrome/browser/prerender/prere...
chrome/browser/prerender/prerender_tracker.h:50: typedef
base::Callback<void(bool /* proceed */)> CheckURLCallback;
Or this.

http://codereview.chromium.org/10006041/diff/1/chrome/browser/prerender/prere...
chrome/browser/prerender/prerender_tracker.h:164: URLCounter url_counter_;
I don't believe we even need this (Including URLCounter), since everything else
uses child/route pairs.

[darin (slow to review), 2012-04-07 05:55:42]

cool, LGTM

[cbentzel, 2012-04-10 17:13:10]

Matt - any concerns before I land this?

[mmenke, 2012-04-10 17:16:40]

On 2012/04/10 17:13:10, cbentzel wrote:
> Matt - any concerns before I land this?

Sorry, didn't realize it was ready for another review.

LGTM.

[commit-bot: I haz the power, 2012-04-10 17:18:35]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/cbentzel@chromium.org/10006041/6001

[commit-bot: I haz the power, 2012-04-10 17:37:41]

Try job failure for 10006041-6001 on linux_rel for step "update".
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=linux_rel&...

Step "update" is always a major failure.
Look at the try server FAQ for more details.

[commit-bot: I haz the power, 2012-04-10 17:43:20]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/cbentzel@chromium.org/10006041/6001

[commit-bot: I haz the power, 2012-04-10 19:15:32]

Change committed as 131602