New issue
Advanced search Search tips

Issue 780117 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Nov 2017
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 1
Type: Bug



Sign in to add a comment

Warmup URL is not being fetched by data saver on Android

Project Member Reported by tbansal@chromium.org, Oct 31 2017

Issue description

Data reduction proxy code should fetch the warmup URL on startup and on connection change events. However, it is not doing so on M63+ Android.
https://uma.googleplex.com/p/chrome/timeline_v2/?sid=c6d0481baabb7123d14d6fea5a5221bc
 

Comment 1 by bengr@chromium.org, Oct 31 2017

Labels: -Pri-2 Pri-1
Until we understand this, we should raise the priority.
Project Member

Comment 2 by bugdroid1@chromium.org, Nov 1 2017

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

commit 09f9fdb847c34a540f93d9e588ea47c2aed7ac75
Author: Tarun Bansal <tbansal@chromium.org>
Date: Wed Nov 01 05:43:33 2017

Add more enum values to record status of the data reduction proxy
QUIC usage and warmup URL fetching.

Also, enable fetching of the warmup URL even if usage of QUIC proxies
is disabled.

Bug:  780117 
Change-Id: I06706aed704701ca6f85553913b8cc94eb4a7afe
Reviewed-on: https://chromium-review.googlesource.com/747131
Reviewed-by: Ryan Sturm <ryansturm@chromium.org>
Commit-Queue: Tarun Bansal <tbansal@chromium.org>
Cr-Commit-Position: refs/heads/master@{#513085}
[modify] https://crrev.com/09f9fdb847c34a540f93d9e588ea47c2aed7ac75/components/data_reduction_proxy/core/browser/data_reduction_proxy_config.cc
[modify] https://crrev.com/09f9fdb847c34a540f93d9e588ea47c2aed7ac75/components/data_reduction_proxy/core/browser/data_reduction_proxy_config_unittest.cc
[modify] https://crrev.com/09f9fdb847c34a540f93d9e588ea47c2aed7ac75/components/data_reduction_proxy/core/browser/data_reduction_proxy_delegate.cc
[modify] https://crrev.com/09f9fdb847c34a540f93d9e588ea47c2aed7ac75/components/data_reduction_proxy/core/browser/data_reduction_proxy_delegate.h
[modify] https://crrev.com/09f9fdb847c34a540f93d9e588ea47c2aed7ac75/components/data_reduction_proxy/core/browser/data_reduction_proxy_delegate_unittest.cc
[modify] https://crrev.com/09f9fdb847c34a540f93d9e588ea47c2aed7ac75/components/data_reduction_proxy/core/common/data_reduction_proxy_params.cc
[modify] https://crrev.com/09f9fdb847c34a540f93d9e588ea47c2aed7ac75/components/data_reduction_proxy/core/common/data_reduction_proxy_params_unittest.cc
[modify] https://crrev.com/09f9fdb847c34a540f93d9e588ea47c2aed7ac75/tools/metrics/histograms/enums.xml

Comment 3 by bengr@chromium.org, Nov 1 2017

Status: Started (was: Assigned)

Comment 4 by bengr@chromium.org, Nov 2 2017

Labels: M-63
Status: Fixed (was: Started)

Sign in to add a comment