New issue
Advanced search Search tips

Issue 620161 link

Starred by 1 user

Issue metadata

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



Sign in to add a comment

Move utility methods in data_reduction_proxy_util to util namespace

Project Member Reported by ryansturm@chromium.org, Jun 14 2016

Issue description

It's not always clear that a method is a utility method when it is being called.
 
Status: Started (was: Assigned)
Project Member

Comment 2 by bugdroid1@chromium.org, Jun 17 2016

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

commit d27a01d7bc41fc61ae27d5a853a76fe0bcf55770
Author: ryansturm <ryansturm@chromium.org>
Date: Fri Jun 17 20:12:48 2016

Moving utility methods in data_reduction_proxy to util namespace

BUG= 620161 

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

[modify] https://crrev.com/d27a01d7bc41fc61ae27d5a853a76fe0bcf55770/components/data_reduction_proxy/core/browser/data_reduction_proxy_bypass_protocol.cc
[modify] https://crrev.com/d27a01d7bc41fc61ae27d5a853a76fe0bcf55770/components/data_reduction_proxy/core/browser/data_reduction_proxy_bypass_protocol_unittest.cc
[modify] https://crrev.com/d27a01d7bc41fc61ae27d5a853a76fe0bcf55770/components/data_reduction_proxy/core/browser/data_reduction_proxy_config_service_client.cc
[modify] https://crrev.com/d27a01d7bc41fc61ae27d5a853a76fe0bcf55770/components/data_reduction_proxy/core/browser/data_reduction_proxy_delegate.cc
[modify] https://crrev.com/d27a01d7bc41fc61ae27d5a853a76fe0bcf55770/components/data_reduction_proxy/core/browser/data_reduction_proxy_network_delegate.cc
[modify] https://crrev.com/d27a01d7bc41fc61ae27d5a853a76fe0bcf55770/components/data_reduction_proxy/core/browser/data_reduction_proxy_pingback_client.cc
[modify] https://crrev.com/d27a01d7bc41fc61ae27d5a853a76fe0bcf55770/components/data_reduction_proxy/core/common/data_reduction_proxy_util.cc
[modify] https://crrev.com/d27a01d7bc41fc61ae27d5a853a76fe0bcf55770/components/data_reduction_proxy/core/common/data_reduction_proxy_util.h

Status: Fixed (was: Started)

Sign in to add a comment