[MacViewsBrowser] bookmark folder does not open on mouse over
Reported by
jongkwon...@navercorp.com,
Sep 7 2017
|
||||||
Issue descriptionUserAgent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_12_6) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/62.0.3202.0 Safari/537.36 Steps to reproduce the problem: 1. build chrome with mac_views_browser=1 and run 2. open a bookmark folder on bookmark bar 3. move mouse pointer to another bookmark folder on bookmark bar What is the expected behavior? New bookmark folder should open automatically and old one should close. What went wrong? Old bookmark folder remain open and new bookmark folder does not open. Did this work before? No Chrome version: 62.0.3202.0 Channel: n/a OS Version: OS X 10.12.6 Flash Version:
,
Sep 7 2017
,
Sep 16 2017
Lots of these error messages in the console: [31757:775:0915/171529.541005:ERROR:screen_mac.mm(164)] Not implemented reached in virtual bool display::(anonymous namespace)::ScreenMac::IsWindowUnderCursor(gfx::NativeWindow)
,
Sep 18 2017
It's ready to be fixed. https://chromium-review.googlesource.com/c/chromium/src/+/654400
,
Dec 6 2017
Thanks for all of these! I'll do a big triage of Issue 671916 blockers when the current phase is complete. See go/macviewstracking
,
Feb 8 2018
[Bulk Edit] Applying M-68 milestone per email discussion with ellyjones@. Pls change it if milestone is incorrectly applied.
,
Feb 26 2018
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/b893f3bd4fa2dfe56d207066b644f4c63433794e commit b893f3bd4fa2dfe56d207066b644f4c63433794e Author: jongkwon.lee <jongkwon.lee@navercorp.com> Date: Mon Feb 26 16:51:32 2018 Implement IsWindowUnderCursor() MacViewsBrowser, it is necessary to open bookmark menus on mouse over. So, implement the unimplemented function IsWindowUnderCursor(). MenuController: :ShowSiblingMenu() use IsWindowUnderCursor(). On Bug: 762764 Change-Id: Ifbf93085a0546f7e3c2b6070267e62212b84bddd Reviewed-on: https://chromium-review.googlesource.com/654400 Reviewed-by: ccameron <ccameron@chromium.org> Commit-Queue: Leonard Grey <lgrey@chromium.org> Cr-Commit-Position: refs/heads/master@{#539165} [modify] https://crrev.com/b893f3bd4fa2dfe56d207066b644f4c63433794e/ui/display/mac/screen_mac.mm
,
Mar 23 2018
This looks Fixed. Thanks #7 :) |
||||||
►
Sign in to add a comment |
||||||
Comment 1 by ajha@chromium.org
, Sep 7 2017Labels: Needs-Triage-M62 TE-NeedsTriageHelp