New issue
Advanced search Search tips

Issue 732566 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Aug 2017
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

Migrate views-based signin screen constants to a separate file

Project Member Reported by wzang@chromium.org, Jun 12 2017

Issue description

As the views-based signin screen is being developed, all the related
constants should be moved to a file similar to ash/shelf/shelf_constants.h

 
Project Member

Comment 1 by bugdroid1@chromium.org, Aug 2 2017

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

commit eb86464a8cb4944b52be8572fbf7e7e1387381c1
Author: Wenzhao Zang <wzang@chromium.org>
Date: Wed Aug 02 01:50:44 2017

Move all login screen related constants to a separate file

Login/lock screen related constants may be used from at least three
places: views-based lock, WebUi based login/lock and wallpaper. So
we want to move them together just as shelf_constants.h

Bug:  732566 ,  733409 
Change-Id: I791d5dfbd02f35cdcc6064b964b97b6978a376d2
Reviewed-on: https://chromium-review.googlesource.com/590483
Reviewed-by: Dan Erat <derat@chromium.org>
Reviewed-by: Jacob Dufault <jdufault@chromium.org>
Commit-Queue: Wenzhao (Colin) Zang <wzang@chromium.org>
Cr-Commit-Position: refs/heads/master@{#491196}
[modify] https://crrev.com/eb86464a8cb4944b52be8572fbf7e7e1387381c1/ash/BUILD.gn
[modify] https://crrev.com/eb86464a8cb4944b52be8572fbf7e7e1387381c1/ash/login/ui/lock_screen.cc
[add] https://crrev.com/eb86464a8cb4944b52be8572fbf7e7e1387381c1/ash/login/ui/login_constants.h
[modify] https://crrev.com/eb86464a8cb4944b52be8572fbf7e7e1387381c1/chrome/browser/ui/webui/chromeos/login/signin_screen_handler.cc

Comment 2 by wzang@chromium.org, Aug 2 2017

Status: Fixed (was: Assigned)
Project Member

Comment 3 by bugdroid1@chromium.org, Aug 10 2017

Labels: merge-merged-3163
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/810c5a28e18e3831911df6fb01efe1cac34e49cf

commit 810c5a28e18e3831911df6fb01efe1cac34e49cf
Author: Wenzhao Zang <wzang@chromium.org>
Date: Thu Aug 10 01:19:01 2017

Move all login screen related constants to a separate file

Login/lock screen related constants may be used from at least three
places: views-based lock, WebUi based login/lock and wallpaper. So
we want to move them together just as shelf_constants.h

TBR=wzang@chromium.org

(cherry picked from commit eb86464a8cb4944b52be8572fbf7e7e1387381c1)

Bug:  732566 ,  733409 
Change-Id: I791d5dfbd02f35cdcc6064b964b97b6978a376d2
Reviewed-on: https://chromium-review.googlesource.com/590483
Reviewed-by: Dan Erat <derat@chromium.org>
Reviewed-by: Jacob Dufault <jdufault@chromium.org>
Commit-Queue: Wenzhao (Colin) Zang <wzang@chromium.org>
Cr-Original-Commit-Position: refs/heads/master@{#491196}
Reviewed-on: https://chromium-review.googlesource.com/609546
Reviewed-by: Wenzhao (Colin) Zang <wzang@chromium.org>
Cr-Commit-Position: refs/branch-heads/3163@{#425}
Cr-Branched-From: ff259bab28b35d242e10186cd63af7ed404fae0d-refs/heads/master@{#488528}
[modify] https://crrev.com/810c5a28e18e3831911df6fb01efe1cac34e49cf/ash/BUILD.gn
[modify] https://crrev.com/810c5a28e18e3831911df6fb01efe1cac34e49cf/ash/login/ui/lock_screen.cc
[add] https://crrev.com/810c5a28e18e3831911df6fb01efe1cac34e49cf/ash/login/ui/login_constants.h
[modify] https://crrev.com/810c5a28e18e3831911df6fb01efe1cac34e49cf/chrome/browser/ui/webui/chromeos/login/signin_screen_handler.cc

Comment 4 by wzang@chromium.org, Aug 10 2017

This CL also points to  crbug.com/733409  which gets merge approval.

Sign in to add a comment