New issue
Advanced search Search tips

Issue 834002 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Apr 2018
Cc:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug



Sign in to add a comment

arc: cleanup OOBE OptIn UI

Project Member Reported by khmel@chromium.org, Apr 17 2018

Issue description

We have 2 version of ARC OOBE OptIn UI.
One non-md and second one md based.

Non-md is now completely deprecated and we should remove this to reduce maintenance cost.
 
Project Member

Comment 1 by bugdroid1@chromium.org, Apr 18 2018

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

commit 93885746db08d66d12743c161a9adabced3caba6
Author: khmel@google.com <khmel@google.com>
Date: Wed Apr 18 18:14:02 2018

arc: Cleanup OOBE OptIn UI

This removes non-md resources due MD is forced now.

Test: Manually
Bug:  834002 
Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation
Change-Id: Ic2af89f20efc69f971c6b5d34ffe9759d4818f90
Reviewed-on: https://chromium-review.googlesource.com/1015659
Reviewed-by: Alexander Alekseev <alemate@chromium.org>
Commit-Queue: Yury Khmel <khmel@google.com>
Cr-Commit-Position: refs/heads/master@{#551744}
[modify] https://crrev.com/93885746db08d66d12743c161a9adabced3caba6/chrome/browser/resources/chromeos/login/arc_terms_of_service.css
[modify] https://crrev.com/93885746db08d66d12743c161a9adabced3caba6/chrome/browser/resources/chromeos/login/arc_terms_of_service.html
[modify] https://crrev.com/93885746db08d66d12743c161a9adabced3caba6/chrome/browser/resources/chromeos/login/arc_terms_of_service.js
[modify] https://crrev.com/93885746db08d66d12743c161a9adabced3caba6/chrome/browser/resources/chromeos/login/screen_arc_terms_of_service.css
[modify] https://crrev.com/93885746db08d66d12743c161a9adabced3caba6/chrome/browser/resources/chromeos/login/screen_arc_terms_of_service.html
[modify] https://crrev.com/93885746db08d66d12743c161a9adabced3caba6/chrome/browser/resources/chromeos/login/screen_arc_terms_of_service.js

Comment 2 by khmel@chromium.org, Apr 18 2018

Status: Fixed (was: Started)

Sign in to add a comment