New issue
Advanced search Search tips

Issue 640309 link

Starred by 1 user
Status: Fixed
Owner: ----
Closed: Oct 2017
Cc:
sadrul@chromium.org
msw@chromium.org
jonr...@chromium.org
mustash-bugs@google.com
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

mash preferences should not be owned by chrome.

Project Member Reported by msw@chromium.org, Aug 23 2016 
mash preferences should not be owned by chrome.

In the long term, mash should own/access preferences without Chrome involvement.
For now, we use Chrome to get/set prefs for things like the ash shelf alignment.

Jon has a related doc at:
  https://docs.google.com/document/d/103kMqAF9Z0Hczd0MXph8HQYxWWzUWyQxDpayET8w_xM

Comment 1 by jonr...@chromium.org, Aug 23 2016

Cc: sadrul@chromium.org
I also have a related code review demonstrating a possible Mojom for that doc.

https://codereview.chromium.org/2092453002/

Ideally the mojom would use base::Value directly, which requires making struct_traits for those classes.

Also needs a followup with an implementation of the manager.

Comment 2 by dtapu...@chromium.org, Sep 2 2016

Components: MUS

Comment 3 by rjkroege@chromium.org, Oct 4 2016

Labels: Proj-Mustash

Comment 4 by rjkroege@chromium.org, Oct 4 2016

Components: Internals>MUS
Project Member

Comment 5 by sheriffbot@chromium.org, Oct 5 2017

Labels: Hotlist-Recharge-Cold
Status: Untriaged (was: Available)
This issue has been Available for over a year. If it's no longer important or seems unlikely to be fixed, please consider closing it out. If it is important, please re-triage the issue.

Sorry for the inconvenience if the bug really should have been left as Available. If you change it back, also remove the "Hotlist-Recharge-Cold" label.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot

Comment 6 by msw@chromium.org, Oct 5 2017

Status: Fixed (was: Untriaged)
Closing this as fixed, Ash has a synchronized PrefService and can own prefs.

Comment 7 by lafo...@chromium.org, Feb 26 2018

Components: -Internals>MUS Internals>Services>WindowService

Comment 8 by lafo...@chromium.org, Feb 26 2018

Components: -MUS
►

Sign in to add a comment