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

Issue 686258 link

Starred by 2 users
Status: Verified
Owner:
varkha@chromium.org
Closed: Mar 2017
Cc:
moh...@chromium.org
█ tdander...@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug

Blocking:
issue 686261



Sign in to add a comment

Clean up of non-MD VPN UI code in the Ash system menu

Project Member Reported by tdander...@chromium.org, Jan 27 2017 
Clean up non-MD code paths in ash/common/system/chromeos/network/vpn_list_view.cc.

Comment 1 by tdander...@chromium.org, Jan 27 2017

Blocking: 686261

Comment 2 by tdander...@chromium.org, Feb 7 2017

Labels: Hotlist-CodeHealth

Comment 3 by tdander...@chromium.org, Mar 7 2017

Labels: -M-58 M-59

Comment 4 by varkha@chromium.org, Mar 22 2017

Cc: tdander...@chromium.org
Status: Started (was: Assigned)
Draft CL at https://codereview.chromium.org/2770933002.
Project Member

Comment 5 by bugdroid1@chromium.org, Mar 24 2017 

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

commit 5e488721e5f9119b9cd772ca02891188a6d83383
Author: varkha <varkha@chromium.org>
Date: Fri Mar 24 00:14:49 2017

[ash-md] Cleans up non-MD code path from VPN detailed page
Code deletion, there should be no functional change.

BUG= 686258 

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

[modify] https://crrev.com/5e488721e5f9119b9cd772ca02891188a6d83383/ash/common/system/chromeos/network/vpn_list_view.cc

Comment 6 by varkha@chromium.org, Mar 24 2017

Status: Fixed (was: Started)

Comment 7 by rookrishna@chromium.org, Mar 29 2017

Status: Verified (was: Fixed)
►

Sign in to add a comment