New issue
Advanced search Search tips

Issue 906684 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Dec 7
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: ----
Type: ----


Participants' hotlists:
Launcher-Tech-Debt


Sign in to add a comment

TACM dead code cleanup

Project Member Reported by newcomer@chromium.org, Nov 19

Issue description

All in the title
 
Project Member

Comment 1 by bugdroid1@chromium.org, Nov 19

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

commit 3ca0664f1915966cdebea25f9b5e4c0dc49f11b8
Author: Alex Newcomer <newcomer@chromium.org>
Date: Mon Nov 19 19:46:16 2018

cros: Remove dead touchable app context menu code

For ash/app_list.

Bug:  906684 
Change-Id: I8b3b79ba1e4220d0c2a3e3df8e822300ac6f63a7
Reviewed-on: https://chromium-review.googlesource.com/c/1342621
Reviewed-by: Weidong Guo <weidongg@chromium.org>
Commit-Queue: Alex Newcomer <newcomer@chromium.org>
Cr-Commit-Position: refs/heads/master@{#609401}
[modify] https://crrev.com/3ca0664f1915966cdebea25f9b5e4c0dc49f11b8/ash/app_list/views/app_list_item_view.cc
[modify] https://crrev.com/3ca0664f1915966cdebea25f9b5e4c0dc49f11b8/ash/app_list/views/search_result_tile_item_view.cc

Project Member

Comment 2 by bugdroid1@chromium.org, Nov 20

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

commit e6dfd5097264d08ed3a37e9bc7a4984c048b14d7
Author: Alex Newcomer <newcomer@chromium.org>
Date: Tue Nov 20 00:25:27 2018

cros: Touchable app context menu dead code cleanup

More cleanups, this is just within ash, not within app_list or shelf.

Bug:  906684 
Change-Id: I42b6a0b6765481685a1ec5e612b4d849f8f3f0ff
Reviewed-on: https://chromium-review.googlesource.com/c/1342766
Reviewed-by: Mitsuru Oshima <oshima@chromium.org>
Commit-Queue: Alex Newcomer <newcomer@chromium.org>
Cr-Commit-Position: refs/heads/master@{#609525}
[modify] https://crrev.com/e6dfd5097264d08ed3a37e9bc7a4984c048b14d7/ash/root_window_controller.cc

Project Member

Comment 3 by bugdroid1@chromium.org, Nov 27

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

commit 1ef59e1da6b5bf3a51ef7060c4fe8fe7bd762297
Author: Alex Newcomer <newcomer@chromium.org>
Date: Tue Nov 27 03:00:10 2018

cros: Touchable App Context Menu dead code cleanup

Remove code that is dead now that touchable app menus are enabled by
default.

Bug:  906684 
Change-Id: I28bb46da596ba93e0bbc01c0685ed1cc1b7ec0ac
Reviewed-on: https://chromium-review.googlesource.com/c/1341064
Commit-Queue: Alex Newcomer <newcomer@chromium.org>
Reviewed-by: Michael Wasserman <msw@chromium.org>
Cr-Commit-Position: refs/heads/master@{#611008}
[modify] https://crrev.com/1ef59e1da6b5bf3a51ef7060c4fe8fe7bd762297/ash/shelf/shelf_application_menu_model.cc
[modify] https://crrev.com/1ef59e1da6b5bf3a51ef7060c4fe8fe7bd762297/ash/shelf/shelf_context_menu_model.cc
[modify] https://crrev.com/1ef59e1da6b5bf3a51ef7060c4fe8fe7bd762297/ash/shelf/shelf_controller.h
[modify] https://crrev.com/1ef59e1da6b5bf3a51ef7060c4fe8fe7bd762297/ash/shelf/shelf_controller_unittest.cc
[modify] https://crrev.com/1ef59e1da6b5bf3a51ef7060c4fe8fe7bd762297/ash/shelf/shelf_view.cc
[modify] https://crrev.com/1ef59e1da6b5bf3a51ef7060c4fe8fe7bd762297/ash/shelf/shelf_view.h
[modify] https://crrev.com/1ef59e1da6b5bf3a51ef7060c4fe8fe7bd762297/ash/shelf/shelf_view_unittest.cc
[modify] https://crrev.com/1ef59e1da6b5bf3a51ef7060c4fe8fe7bd762297/ash/shelf/shelf_window_watcher_item_delegate.cc

Project Member

Comment 4 by bugdroid1@chromium.org, Dec 7

Status: Fixed (was: Started)

Sign in to add a comment