Issue 939233004: Fixing remote wallpaper change in multi profile cases

Issue 939233004: Fixing remote wallpaper change in multi profile cases (Closed)

Created:
5 years, 10 months ago by Mr4D (OOO till 08-26)

Modified:
5 years, 10 months ago

Reviewers:
Nikita (slow), bshe

CC:
chromium-reviews, chromium-apps-reviews_chromium.org, stevenjb+watch_chromium.org, davemoore+watch_chromium.org, oshima+watch_chromium.org, extensions-reviews_chromium.org, xdai1

Base URL:
https://chromium.googlesource.com/chromium/src.git@master

Target Ref:
refs/pending/heads/master

Project:
chromium

Visibility:
Public.

More Reviews

Description

Fixing remote wallpaper change in multi profile cases The problem was that if the wallpaper of the non active user got changed on a different system, the remote system would replace the wallpaper of the active desktop with the change, no matter which user currently was active. JS is supplying the browser-context to the api call which can be used to get the applications owner - and that is used to identify the proper user. Creating a test case for this scenario is a lot of work (setting up multi profile, running the wallpaper manager for both users, triggering a change in the app of one user to force a change, waiting until the change has trickled through the system, ..). Since the work required for an automated test feels rather complex and time consuming for this setup and the change itself is fairly straight forward and simple, so I did not create a test. BUG=459599 TEST=visual Committed: https://crrev.com/cdc6fb42259bb6364cdf26100947dfccc36a0c4b Cr-Commit-Position: refs/heads/master@{#317322}