Implement HTMLMenuItemElement.label

third_party/WebKit/Source/core/page/CustomContextMenuProvider.cpp

Index: third_party/WebKit/Source/core/page/CustomContextMenuProvider.cpp
diff --git a/third_party/WebKit/Source/core/page/CustomContextMenuProvider.cpp b/third_party/WebKit/Source/core/page/CustomContextMenuProvider.cpp
index bbfe4bc7864dc3dc3c5e7f5e09fc675d83b12193..561c647a807eea825b3597223e1891ad4fca6a77 100644
--- a/third_party/WebKit/Source/core/page/CustomContextMenuProvider.cpp
+++ b/third_party/WebKit/Source/core/page/CustomContextMenuProvider.cpp
@@ -69,7 +69,7 @@ void CustomContextMenuProvider::AppendSeparator(ContextMenu& context_menu) {
 void CustomContextMenuProvider::AppendMenuItem(HTMLMenuItemElement* menu_item,
                                                ContextMenu& context_menu) {
   // Avoid menuitems with no label.
-  String label_string = menu_item->FastGetAttribute(labelAttr);
+  String label_string = menu_item->label();

[tkent, 2017/04/26 08:59:58]

Can you check Firefox behavior for <menuitem label="">text content</menuitem>? 
If Firefox shows "text content" in the menu, it uses conceptualLabel() here.

[yuzuchan, 2017/04/26 10:28:30]

Firefox does not render the menuitem when label is empty and text content
exists, 
which means label is set to an empty string.
To match the behavior I'll keep it this way here.

   if (label_string.IsEmpty())
     return;