Fix typeahead for popup menu with attributes

Fix typeahead for popup menu with attributes

Popup menu item with attributes (<optgroup> for instance) will
break typeahead matching support on Mac. Use the same trick Apple
uses in WebKit2 [1] to fix this.

BUG=90064
[1] http://trac.webkit.org/browser/trunk/Source/WebKit2/UIProcess/mac/WebPopupMenuProxyMac.mm#L90

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=238377

[Avi (use Gerrit), 2013-12-02 14:53:06]

Sweet! LGTM with a link.

https://codereview.chromium.org/99043003/diff/1/content/browser/renderer_host...
File content/browser/renderer_host/webmenurunner_mac.mm (right):

https://codereview.chromium.org/99043003/diff/1/content/browser/renderer_host...
content/browser/renderer_host/webmenurunner_mac.mm:88: // what Apple uses in
WebKit as well.
If you're referring to "what Apple uses in WebKit", throw in a link here and be
nice to the next person :)

[commit-bot: I haz the power, 2013-12-02 15:05:55]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/jiangj@opera.com/99043003/20001

[Jiang Jiang, 2013-12-02 15:06:13]

On 2013/12/02 14:53:06, Avi wrote:
> Sweet! LGTM with a link.
> 
>
https://codereview.chromium.org/99043003/diff/1/content/browser/renderer_host...
> File content/browser/renderer_host/webmenurunner_mac.mm (right):
> 
>
https://codereview.chromium.org/99043003/diff/1/content/browser/renderer_host...
> content/browser/renderer_host/webmenurunner_mac.mm:88: // what Apple uses in
> WebKit as well.
> If you're referring to "what Apple uses in WebKit", throw in a link here and
be
> nice to the next person :)

Updated comment with the link.

[commit-bot: I haz the power, 2013-12-02 15:22:19]

Retried try job too often on mac_rel for step(s) base_unittests, browser_tests,
interactive_ui_tests, net_unittests, unit_tests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=mac_rel&nu...

[commit-bot: I haz the power, 2013-12-03 11:21:51]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/jiangj@opera.com/99043003/20001

[commit-bot: I haz the power, 2013-12-03 12:48:49]

Change committed as 238377