1284464 - Missing arrow indicating submenus in the context menu

Status: RESOLVED FIXED

(Core :: Widget: Cocoa, defect)

Description

[Florian Quèze [:florian]]

Attachment: Screenshot

See attached screenshot. The ">" arrow at the end of the "This Frame" context menu item is missing on current 50a1 trunk builds (on Mac at least).
I can reproduce both on the internal retina screen and the external low DPI screen.

Works correctly on release (47) builds.

Comment 1

[Florian Quèze [:florian]]

[Tracking Requested - why for this release]: Visible regression in Fx 50.

mozregression says https://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?fromchange=259c39227fd6e1ccbc244b39bb6c02136c614cee&tochange=22f34da350173fda29c5048974e6d99e2db028e6 -> bug 1279017

In case this matters, I'm running OS X 10.10.4 (14E46).

Comment 2

[Stefan [:stefanh]]

I see what I did wrong (and OS version actually matters here)...

Comment 3

[Stefan [:stefanh]]

Attachment: 1284464.1.diff

I obviously slipped... thanks to Florian who noticed this.

Comment 4

[Markus Stange [:mstange]]

Comment on attachment 8768103 [details] [diff] [review]
1284464.1.diff

Review of attachment 8768103 [details] [diff] [review]:
-----------------------------------------------------------------

Looks like I slipped as well.

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/1bd2b6599c55
Missing submenu arrows in xul menus on Mac OS X 10.9-10.10. r=mstange.

Comment 6

[Marcia Knous [:marcia]]

Tracking 50+ for this visible Mac regression.

Comment 8

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/1bd2b6599c55