New issue
Advanced search Search tips

Issue 810535 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Feb 2018
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug

Blocking:
issue 779221



Sign in to add a comment

Clean up the legacy code in wallpaper.mojom

Project Member Reported by wzang@chromium.org, Feb 8 2018

Issue description

|wallpaper.mojom| contains legacy mojo calls that's no longer used.
 
Project Member

Comment 1 by bugdroid1@chromium.org, Feb 9 2018

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

commit ed752013680e93bb4bc51baac5435b37e57ef73b
Author: Wenzhao Zang <wzang@chromium.org>
Date: Fri Feb 09 23:21:35 2018

cros: Clean up legacy code in wallpaper.mojom

Deletes the dead code after wallpaper refactoring.

TBR=jamescook@chromium.org, xdai@chromium.org

Bug:  810535 
Change-Id: I7ef7124c825131810b13cf6e50aa93971fec5b4c
Reviewed-on: https://chromium-review.googlesource.com/909849
Commit-Queue: Wenzhao (Colin) Zang <wzang@chromium.org>
Reviewed-by: Daniel Cheng <dcheng@chromium.org>
Cr-Commit-Position: refs/heads/master@{#535876}
[delete] https://crrev.com/8a846d001e80605e46cbec141bc3c42d20a1c863/ash/public/cpp/wallpaper_struct_traits.cc
[modify] https://crrev.com/ed752013680e93bb4bc51baac5435b37e57ef73b/ash/public/cpp/wallpaper_struct_traits.h
[modify] https://crrev.com/ed752013680e93bb4bc51baac5435b37e57ef73b/ash/public/interfaces/wallpaper.mojom
[modify] https://crrev.com/ed752013680e93bb4bc51baac5435b37e57ef73b/ash/public/interfaces/wallpaper.typemap
[modify] https://crrev.com/ed752013680e93bb4bc51baac5435b37e57ef73b/ash/wallpaper/wallpaper_controller.cc
[modify] https://crrev.com/ed752013680e93bb4bc51baac5435b37e57ef73b/ash/wallpaper/wallpaper_controller.h
[modify] https://crrev.com/ed752013680e93bb4bc51baac5435b37e57ef73b/chrome/browser/ui/ash/test_wallpaper_controller.cc
[modify] https://crrev.com/ed752013680e93bb4bc51baac5435b37e57ef73b/chrome/browser/ui/ash/test_wallpaper_controller.h

Comment 2 by wzang@chromium.org, Feb 9 2018

Status: Fixed (was: Started)

Sign in to add a comment