New issue
Advanced search Search tips

Issue 750180 link

Starred by 2 users
Status: Fixed
Owner:
xiaochu@chromium.org
Closed: Aug 2017
Cc:
dpa...@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug



Sign in to add a comment

platform_AddPrinter.py should use chrome://settings

Project Member Reported by steve...@chromium.org, Jul 28 2017 
MD Settings has been the default settings since 59.

We need to update platform_AddPrinter.py to use chrome://settings instead of chrome://md-settings before chrome://md-settings is deprecated in 62.
Project Member

Comment 1 by bugdroid1@chromium.org, Jul 30 2017 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/autotest/+/4333fe6024cf39adbd18247cfbaa97c256785a24

commit 4333fe6024cf39adbd18247cfbaa97c256785a24
Author: Xiaochu Liu <xiaochu@chromium.org>
Date: Sun Jul 30 20:46:10 2017

platform_AddPrinter: move to chrome://setting URL

Since chrome://md-setting is deprecating in 62, moving to
chrome://setting.

BUG= chromium:750180 
TEST=test_that -b <board> <IP> platform_AddPrinter

Change-Id: I625a809ae86f2b26c7dda1b022e32d9174e167d9
Reviewed-on: https://chromium-review.googlesource.com/592332
Commit-Ready: Xiaochu Liu <xiaochu@chromium.org>
Tested-by: Xiaochu Liu <xiaochu@chromium.org>
Reviewed-by: Steven Bennetts <stevenjb@chromium.org>

[modify] https://crrev.com/4333fe6024cf39adbd18247cfbaa97c256785a24/client/site_tests/platform_AddPrinter/platform_AddPrinter.py

Comment 2 by xiaochu@chromium.org, Aug 3 2017

Status: Fixed (was: Assigned)
►

Sign in to add a comment