New issue
Advanced search Search tips

Issue 754284 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Closed: Oct 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: iOS
Pri: 3
Type: Bug



Sign in to add a comment

Disable prefetching on iOS

Project Member Reported by rohitrao@chromium.org, Aug 10 2017

Issue description

PreloadController currently tries to prefetch SRPs, but it does so using URLFetcher, which does not actually warm up the WKWebView caches.

We should disable the feature and delete all of the associated code.
 
Summary: Disable prefetching on iOS (was: Disable prefetching)
Project Member

Comment 2 by bugdroid1@chromium.org, Aug 10 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/3aab85a2e128b01a4a612c9b254faad2723aba20

commit 3aab85a2e128b01a4a612c9b254faad2723aba20
Author: Rohit Rao <rohitrao@google.com>
Date: Thu Aug 10 16:07:32 2017

[ios] Disables prefetching.

Prefetching using URLFetcher is not effective for warming up WKWebView caches.

BUG= 754284 

Change-Id: Id84afbcc4306d4abf19f577f0f563e1a81938e22
Reviewed-on: https://chromium-review.googlesource.com/610322
Reviewed-by: Sylvain Defresne <sdefresne@chromium.org>
Reviewed-by: Mark Cogan <marq@chromium.org>
Commit-Queue: Rohit Rao (ping after 24h) <rohitrao@chromium.org>
Cr-Commit-Position: refs/heads/master@{#493407}
[modify] https://crrev.com/3aab85a2e128b01a4a612c9b254faad2723aba20/ios/chrome/browser/prerender/preload_controller.mm
[modify] https://crrev.com/3aab85a2e128b01a4a612c9b254faad2723aba20/ios/chrome/browser/prerender/preload_controller_unittest.mm

Status: Fixed (was: Started)

Sign in to add a comment