New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 884734 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Last visit > 30 days ago
Closed: Sep 18
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug



Sign in to add a comment

Move all quarantine constants in the same place.

Project Member Reported by cheyuw@google.com, Sep 17

Issue description

The quarantine parameters will be used in multiple places in the chrome cleanup tool.
It is better for reading and using if we put all related constants in the same place.
 
Project Member

Comment 1 by bugdroid1@chromium.org, Sep 18

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

commit 99800cff6767062ce6893749e884016e5ee42ea0
Author: Che-yu Wu <cheyuw@google.com>
Date: Tue Sep 18 21:28:23 2018

Add quarantine constants files.

Add files to define all constants related to the quarantine feature.

Bug:  crbug.com/884734 
Change-Id: I254cd63f1ae48596abb15c10b996c7969d50a63e
Reviewed-on: https://chromium-review.googlesource.com/1228299
Reviewed-by: Chris Sharp <csharp@chromium.org>
Commit-Queue: Che-yu Wu <cheyuw@google.com>
Cr-Commit-Position: refs/heads/master@{#592200}
[modify] https://crrev.com/99800cff6767062ce6893749e884016e5ee42ea0/chrome/chrome_cleaner/constants/BUILD.gn
[add] https://crrev.com/99800cff6767062ce6893749e884016e5ee42ea0/chrome/chrome_cleaner/constants/quarantine_constants.cc
[add] https://crrev.com/99800cff6767062ce6893749e884016e5ee42ea0/chrome/chrome_cleaner/constants/quarantine_constants.h

Status: Fixed (was: Untriaged)

Sign in to add a comment