Integrate the LocalPredictor with the Prerender Service.

Integrate the LocalPredictor with the Prerender Service.
R=jam@chromium.org

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=220674

[jam, 2013-08-30 20:34:30]

https://codereview.chromium.org/23622012/diff/113001/chrome/browser/prerender...
File chrome/browser/prerender/prerender_local_predictor.cc (right):

https://codereview.chromium.org/23622012/diff/113001/chrome/browser/prerender...
chrome/browser/prerender/prerender_local_predictor.cc:772: for (int i = 0; i <
static_cast<int>(list->GetSize()); i++) {
nit: instead of casting, just use "size_t i", also below

https://codereview.chromium.org/23622012/diff/113001/chrome/browser/prerender...
File chrome/browser/prerender/prerender_local_predictor.h (right):

https://codereview.chromium.org/23622012/diff/113001/chrome/browser/prerender...
chrome/browser/prerender/prerender_local_predictor.h:136: void virtual
OnURLFetchComplete(const net::URLFetcher* source) OVERRIDE;
usually we write

// net::URLFetcherDelegate implementation:

above this line

https://codereview.chromium.org/23622012/diff/113001/chrome/browser/prerender...
chrome/browser/prerender/prerender_local_predictor.h:168: OutstandingFetchers;
indent

[tburkard, 2013-08-30 20:59:01]

https://codereview.chromium.org/23622012/diff/113001/chrome/browser/prerender...
File chrome/browser/prerender/prerender_local_predictor.cc (right):

https://codereview.chromium.org/23622012/diff/113001/chrome/browser/prerender...
chrome/browser/prerender/prerender_local_predictor.cc:772: for (int i = 0; i <
static_cast<int>(list->GetSize()); i++) {
On 2013/08/30 20:34:30, jam wrote:
> nit: instead of casting, just use "size_t i", also below

Done.

https://codereview.chromium.org/23622012/diff/113001/chrome/browser/prerender...
File chrome/browser/prerender/prerender_local_predictor.h (right):

https://codereview.chromium.org/23622012/diff/113001/chrome/browser/prerender...
chrome/browser/prerender/prerender_local_predictor.h:136: void virtual
OnURLFetchComplete(const net::URLFetcher* source) OVERRIDE;
On 2013/08/30 20:34:30, jam wrote:
> usually we write
> 
> // net::URLFetcherDelegate implementation:
> 
> above this line

Done.

https://codereview.chromium.org/23622012/diff/113001/chrome/browser/prerender...
chrome/browser/prerender/prerender_local_predictor.h:168: OutstandingFetchers;
On 2013/08/30 20:34:30, jam wrote:
> indent

Done.

[jam, 2013-08-30 21:00:37]

lgtm

[tburkard, 2013-08-30 22:09:49]

Committed patchset #9 manually as r220674 (presubmit successful).