Speculative resource prefetching - Not modify resource_type for MAIN_FRAME requests.

Speculative resource prefetching - Not modify resource_type for MAIN_FRAME requests.

We override resource_types for subresources based on mime_types to not have to
deal with types such as PREFETCH, SUBRESOURCE, etc. But we should not do this
for main frame requests which we want to treat separately.

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

[Shishir, 2012-08-06 18:22:18]

Please take a look.

Adding jar for chrome/browser/net/...

[jar (doing other things), 2012-08-06 19:02:53]

Please fix nearby nit while editing the file.

The change LGTM.

https://chromiumcodereview.appspot.com/10829195/diff/3001/chrome/browser/net/...
File chrome/browser/net/resource_prefetch_predictor_observer.cc (right):

https://chromiumcodereview.appspot.com/10829195/diff/3001/chrome/browser/net/...
chrome/browser/net/resource_prefetch_predictor_observer.cc:60: :
predictor_(predictor->AsWeakPtr()) {
nit: indent ":" only 4 spaces from start of function definition.

[Shishir, 2012-08-06 19:11:10]

https://chromiumcodereview.appspot.com/10829195/diff/3001/chrome/browser/net/...
File chrome/browser/net/resource_prefetch_predictor_observer.cc (right):

https://chromiumcodereview.appspot.com/10829195/diff/3001/chrome/browser/net/...
chrome/browser/net/resource_prefetch_predictor_observer.cc:60: :
predictor_(predictor->AsWeakPtr()) {
On 2012/08/06 19:02:53, jar wrote:
> nit: indent ":" only 4 spaces from start of function definition.

Done.

[jar (doing other things), 2012-08-06 19:22:44]

lgtm

[dominich, 2012-08-06 19:54:46]

lgtm

[commit-bot: I haz the power, 2012-08-07 00:01:20]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/shishir@chromium.org/10829195/1002

[commit-bot: I haz the power, 2012-08-07 01:28:35]

Change committed as 150221