Issue metadata
Sign in to add a comment
|
Regression: Unnecessary separator line is seen below App info on the context menu of any app |
||||||||||||||||||||
Issue descriptionChrome Version: 67.0.3390.0/10559.0.0 dev channel Daisy,Kip and Reks OS: Chrome OS What steps will reproduce the problem? (1)Sign-in to user>> Click on App launcher and Right click on any app to get context menu and observe Actual: Unnecessary separator line is seen below App info on the context menu Expected: No such issue should be seen This is a Regression issue as same is working fine in 67.0.3383.0/10550.0.0 @Newcomer: Please confirm the issue
,
Apr 10 2018
,
May 8 2018
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/0453022ae1a429f8650446cb0aa2b882a1e4c297 commit 0453022ae1a429f8650446cb0aa2b882a1e4c297 Author: Alex Newcomer <newcomer@chromium.org> Date: Tue May 08 22:08:31 2018 cros:Add Actionable Submenu option to AppList extension context menus Part 2 of "Add Actionable Submenus To Extension App Context Menus" This part adds the actionable submenu option to the AppList. Also in this cl: - Re-ordered function in extension_app_context_menu.cc to match the header. - Fixed a mis-placed separator (incidental fix due to restructuring the context menu option ordering). Bug: 829940 Bug: 830339 Change-Id: If3f369260b4883206a6014754e08fcbae697b7ae Reviewed-on: https://chromium-review.googlesource.com/1050406 Reviewed-by: Xiyuan Xia <xiyuan@chromium.org> Commit-Queue: Alex Newcomer <newcomer@chromium.org> Cr-Commit-Position: refs/heads/master@{#556980} [modify] https://crrev.com/0453022ae1a429f8650446cb0aa2b882a1e4c297/chrome/browser/ui/app_list/app_context_menu_unittest.cc [modify] https://crrev.com/0453022ae1a429f8650446cb0aa2b882a1e4c297/chrome/browser/ui/app_list/extension_app_context_menu.cc [modify] https://crrev.com/0453022ae1a429f8650446cb0aa2b882a1e4c297/chrome/browser/ui/app_list/extension_app_context_menu.h
,
May 8 2018
|
|||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||
Comment 1 by kebalaji@chromium.org
, Apr 9 2018