New issue
Advanced search Search tips

Issue 660136 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Oct 2016
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 3
Type: Bug



Sign in to add a comment

Remove the code to migrate prefs

Project Member Reported by tbansal@chromium.org, Oct 27 2016

Issue description

Remove the data reduction proxy code that migrates the compression prefs. Also, remove the code that enables compression stats logging without a valid DataReductionProxyStore. Both of these are currently unreachable.
 
Labels: M-56
Status: Fixed (was: Started)
Project Member

Comment 2 by bugdroid1@chromium.org, Oct 27 2016

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

commit 7839ed7cd5aae2d76abfeccef3fd2867a6b4e864
Author: tbansal <tbansal@chromium.org>
Date: Thu Oct 27 22:59:52 2016

Remove unused DRP code for migrating prefs

Remove the data reduction proxy code that migrates the
compression prefs. Also, remove the code that enables
compression stats logging without a valid
DataReductionProxyStore. Both of these are currently
unreachable.

BUG= 660136 

Review-Url: https://codereview.chromium.org/2447013003
Cr-Commit-Position: refs/heads/master@{#428185}

[modify] https://crrev.com/7839ed7cd5aae2d76abfeccef3fd2867a6b4e864/components/data_reduction_proxy/core/browser/data_reduction_proxy_prefs.cc
[modify] https://crrev.com/7839ed7cd5aae2d76abfeccef3fd2867a6b4e864/components/data_reduction_proxy/core/browser/data_reduction_proxy_prefs.h
[modify] https://crrev.com/7839ed7cd5aae2d76abfeccef3fd2867a6b4e864/components/data_reduction_proxy/core/browser/data_reduction_proxy_prefs_unittest.cc
[modify] https://crrev.com/7839ed7cd5aae2d76abfeccef3fd2867a6b4e864/components/data_reduction_proxy/core/browser/data_reduction_proxy_service.cc
[modify] https://crrev.com/7839ed7cd5aae2d76abfeccef3fd2867a6b4e864/components/data_reduction_proxy/core/browser/data_reduction_proxy_service.h
[modify] https://crrev.com/7839ed7cd5aae2d76abfeccef3fd2867a6b4e864/components/data_reduction_proxy/core/common/data_reduction_proxy_pref_names.cc
[modify] https://crrev.com/7839ed7cd5aae2d76abfeccef3fd2867a6b4e864/components/data_reduction_proxy/core/common/data_reduction_proxy_pref_names.h

Sign in to add a comment