New issue
Advanced search Search tips

Issue 657107 link

Starred by 2 users

Issue metadata

Status: Archived
Owner:
Closed: Jan 2017
Cc:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 3
Type: Bug



Sign in to add a comment

Move display::mojom::TestDisplayController into its own capability class

Project Member Reported by sky@chromium.org, Oct 18 2016

Issue description

Currently display::mojom::TestDisplayController is grouped in the capability class ui:window_manager. As TestDisplayController is really only for testing we should move it into its own capability class, maybe ui:window_manager-test or something.
 
Project Member

Comment 1 by bugdroid1@chromium.org, Jan 24 2017

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

commit 09b64d2271dec56ef2c739809abc499263926b69
Author: kylechar <kylechar@chromium.org>
Date: Tue Jan 24 20:42:37 2017

Move TestDisplayController into own capability class.

TestDisplayController isn't directly related to the window_manager
class and shouldn't be part of it.

BUG= 657107 

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

[modify] https://crrev.com/09b64d2271dec56ef2c739809abc499263926b69/ash/mus/manifest.json
[modify] https://crrev.com/09b64d2271dec56ef2c739809abc499263926b69/services/ui/manifest.json

Status: Fixed (was: Untriaged)

Comment 3 by dchan@google.com, Apr 17 2017

Labels: VerifyIn-59

Comment 4 by dchan@google.com, May 30 2017

Labels: VerifyIn-60

Comment 5 by dchan@chromium.org, Aug 1 2017

Labels: VerifyIn-61

Comment 6 by dchan@chromium.org, Oct 14 2017

Status: Archived (was: Fixed)

Sign in to add a comment