[webkit-reviews] review granted: [Bug 56659] [GTK] Expose menu items and menus as children of a menu bar : [Attachment 89015] Patch proposal + Layout test

[bugzilla-daemon at webkit.org]

chris fleizach <cfleizach at apple.com> has granted Mario Sanchez Prada
<msanchez at igalia.com>'s request for review:
Bug 56659: [GTK] Expose menu items and menus as children of a menu bar
https://bugs.webkit.org/show_bug.cgi?id=56659

Attachment 89015: Patch proposal + Layout test
https://bugs.webkit.org/attachment.cgi?id=89015&action=review

------- Additional Comments from chris fleizach <cfleizach at apple.com>
View in context: https://bugs.webkit.org/attachment.cgi?id=89015&action=review

otherwise looks good. r=me

> Source/WebCore/accessibility/AccessibilityRenderObject.cpp:2968
>	       return MenuItemRole;

It looks like we're calling parentObjectUnignored()->ariaRoleAttribute()
potentially three times. I think that value should be stored local.
parentObjectUnignored() can be little expensive.