Implement HTMLMenuItemElement.label

third_party/WebKit/Source/core/html/HTMLMenuItemElement.h

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef HTMLMenuItemElement_h
 #define HTMLMenuItemElement_h
 
 #include "core/html/HTMLElement.h"
 
 namespace blink {
 
 class HTMLMenuItemElement final : public HTMLElement {
   DEFINE_WRAPPERTYPEINFO();
 
  public:
+  const String label() const;

[tkent, 2017/04/25 23:25:00]

We usually write declarations in the following order:

  constructors or factory functions
  destructor
  other boilerplate functions like DECLARE_VIRTUAL_TRACE
  member functions

So,

 public:
  DECLARE_NODE_FACTORY(HTMLMenuItemElement);

  String label() const;
  void setLabel(const AtomicString&);

is preferable.

[yuzuchan, 2017/04/26 08:54:57]

Done. Thanks!

+  void setLabel(const AtomicString&);
   DECLARE_NODE_FACTORY(HTMLMenuItemElement);
 
  private:
   explicit HTMLMenuItemElement(Document&);
   bool IsURLAttribute(const Attribute&) const override;
   void ParseAttribute(const AttributeModificationParams&) override;
   void DefaultEventHandler(Event*) override;
 };
 
 }  // namespace blink
 
 #endif  // HTMLMenuItemElement_h