Implement contextmenu attribute with basic support of <menu>

Source/core/html/HTMLMenuElement.cpp

 /*
  * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
  *           (C) 1999 Antti Koivisto (koivisto@kde.org)
  * Copyright (C) 2010 Apple Inc. All rights reserved.
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
  * License as published by the Free Software Foundation; either
  * version 2 of the License, or (at your option) any later version.
  *
  * This library is distributed in the hope that it will be useful,
  * but WITHOUT ANY WARRANTY; without even the implied warranty of
  * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
  * Library General Public License for more details.
  *
  * You should have received a copy of the GNU Library General Public License
  * along with this library; see the file COPYING.LIB.  If not, write to
  * the Free Software Foundation, Inc., 51 Franklin Street, Fifth Floor,
  * Boston, MA 02110-1301, USA.
  *
  */
 
 #include "config.h"
+
 #include "core/html/HTMLMenuElement.h"
 
-#include "HTMLNames.h"
+#include "core/dom/ElementTraversal.h"
+#include "core/events/RelatedEvent.h"
+#include "core/html/HTMLMenuElementEventListener.h"
+#include "core/html/HTMLMenuitemElement.h"
+#include "core/page/ContextMenuController.h"
+#include "core/page/Page.h"
 
 namespace WebCore {
 
 using namespace HTMLNames;
 
 inline HTMLMenuElement::HTMLMenuElement(Document& document)
     : HTMLElement(menuTag, document)
 {
     ScriptWrappable::init(this);
+    RefPtr<EventListener> listener = HTMLMenuElementEventListener::create(this);
+    addEventListener("show", listener.release(), false);

[esprehn, 2014/06/06 22:30:42]

We don't normally add event listeners like this. I'm not sure we want to do
this.

[pals, 2014/06/10 10:42:39]

Done. Removed HTMLMenuElementEventListener. Added defaultEventHandler.

 }
 
 PassRefPtrWillBeRawPtr<HTMLMenuElement> HTMLMenuElement::create(Document& document)
 {
     return adoptRefWillBeRefCountedGarbageCollected(new HTMLMenuElement(document));
 }
 
+CustomContextMenuProvider& HTMLMenuElement::ensureCustomProvider()

[esprehn, 2014/06/06 22:30:42]

ensureMenuProvider()

[pals, 2014/06/10 10:42:39]

Done.

+{
+    if (!m_menuProvider) {
+        ContextMenu menu;
+        populateContextMenuItems(this, menu);
+        m_menuProvider = CustomContextMenuProvider::create(this, menu.items());
+    }
+    return *m_menuProvider;
 }
+
+void HTMLMenuElement::populateContextMenuItems(HTMLMenuElement* menu, ContextMenu& contextMenu)
+{
+    for (Element* nextElement = ElementTraversal::firstWithin(*menu); nextElement;) {

[esprehn, 2014/06/06 22:30:42]

Lets use a while() loop, it'll be more clear.

[pals, 2014/06/10 10:42:39]

Done.

+        if (nextElement->hasTagName(HTMLNames::menuTag)) {
+            ContextMenu subMenu;
+            populateContextMenuItems(toHTMLMenuElement(nextElement), subMenu);
+            contextMenu.appendItem(ContextMenuItem(SubmenuType, ContextMenuItemCustomTagNoAction, nextElement->fastGetAttribute(HTMLNames::labelAttr), &subMenu));
+            nextElement = ElementTraversal::nextSibling(*nextElement);
+            continue;
+        } else if (nextElement->hasTagName(HTMLNames::menuitemTag)) {
+            m_menuItems.append(toHTMLElement(nextElement));
+            contextMenu.appendItem(ContextMenuItem(ActionType, static_cast<ContextMenuAction>(ContextMenuItemBaseCustomTag + m_menuItems.size() - 1), nextElement->fastGetAttribute(HTMLNames::labelAttr)));
+        }
+        nextElement = ElementTraversal::next(*nextElement, menu);
+    }
+}
+
+HTMLElement* HTMLMenuElement::selectedMenuItem(unsigned menuId)
+{
+    return m_menuItems[menuId - ContextMenuItemBaseCustomTag];

[esprehn, 2014/06/06 22:30:42]

This unsigned thing is really weird.

[pals, 2014/06/10 10:42:39]

Done. Added checks.

+}
+
+}