Issue metadata
Sign in to add a comment
|
Regression: [MAC]Context menu is not seen as per MacViews flag when enabled.
Reported by
aiman.an...@etouch.net,
Jul 17
|
||||||||||||||||||||||
Issue descriptionChrome Version: 69.0.3493.0 (Official Build) Revision 7066e76287893adffd8bbd8c71c369cdc995c0c3-refs/branch-heads/3493@{#1} (32/64-bit) OS: Mac(10.12.6, 10.13.1, 10.13.6, 10.14). Pre-Condiiton: Enable 'Use Views browser windows instead of Cocoa' flag from chrome://flags and relaunch the browser. Steps to reproduce: 1. Launch chrome, Right click on NTP and observe the Context menu. Actual Result: Context menu is not seen as per MacViews flag. Expected Result: Context menu should be seen as per MacViews. This is a regression issue, broken in 'M-69', and below is the bisect provided using per-revision script. Good Build:69.0.3489.0 (Revision:574446) Bad Build: 69.0.3491.0 (Revision:575129) You are probably looking for a change made after 574561 (known good), but no later than 574562 (first known bad). CHANGE-LOG URL: The script might not always return single CL as suspect as some perf builds might get missing due to failure. https://chromium.googlesource.com/chromium/src/+log/06c7d7e68470ca975c0d10340a89e3e69ebab6ad..c7a09eb8d51c7e5d4d6ae68b4de0c8d3d9fcbabb Suspect: https://chromium.googlesource.com/chromium/src/+/c7a09eb8d51c7e5d4d6ae68b4de0c8d3d9fcbabb @ellyjones: Could you please check whether this is caused with respect to your change, if not please help us in assigning it to the right owner. Kindly refer the attached screen-cast for reference. Note: Issue is not seen on Windows(7,8,8.1,10), Linux(14.04 LTS) OS. Thank You! |
|||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||
Comment 1 by ellyjo...@chromium.org
, Jul 17This is working as intended - the linked CL ("use cocoa context menus for web content") caused MacViews builds to use Cocoa context menus for web content.