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/EventListener.h"
+#include "core/events/RelatedEvent.h"
+#include "core/html/HTMLMenuitemElement.h"
+#include "core/page/ContextMenuController.h"
+#include "core/page/Page.h"
 
 namespace WebCore {
 
 using namespace HTMLNames;
 
+class HTMLMenuElementEventListener : public EventListener {

[esprehn, 2014/05/20 05:51:34]

FINAL

This class should also have it's own .h and .cpp file.

[pals, 2014/05/21 07:55:12]

Done.

+public:
+    static PassRefPtr<HTMLMenuElementEventListener> create(HTMLMenuElement* menu) { return adoptRef(new HTMLMenuElementEventListener(menu)); }
+    static const HTMLMenuElementEventListener* cast(const EventListener* listener)

[esprehn, 2014/05/20 05:51:34]

This should be an instance method, toHTMLMenuElementEventListener() or a free
function of that same name. Remove ::cast()

[pals, 2014/05/21 07:55:12]

Done.

+    {
+        return listener->type() == NativeEventListenerType
+            ? static_cast<const HTMLMenuElementEventListener*>(listener)
+            : 0;
+    }
+
+    virtual bool operator==(const EventListener& other);
+
+private:
+    HTMLMenuElementEventListener(HTMLMenuElement* menu)

[esprehn, 2014/05/20 05:51:34]

missing explicit

[pals, 2014/05/21 07:55:12]

Done.

+        : EventListener(NativeEventListenerType)
+        , m_menu(menu)
+    {
+    }
+
+    virtual void handleEvent(ExecutionContext*, Event*);
+
+    HTMLMenuElement* m_menu;
+};
+
+void HTMLMenuElementEventListener::handleEvent(ExecutionContext*, Event* event)
+{
+    if (event->type() == EventTypeNames::show && toRelatedEvent(event)->isRelatedEvent()) {
+        Node* node = toRelatedEvent(event)->relatedTarget()->toNode();
+        if (node && node->isElementNode()) {
+            const HTMLElement& element = toHTMLElement(*node);
+            HTMLMenuElement* menuElement = element.menuElement();
+            if (menuElement) {
+                menuElement->buildCustomProvider();
+                event->setDefaultHandled();
+            }
+        }
+    }
+}
+
+bool HTMLMenuElementEventListener::operator==(const EventListener& listener)
+{
+    if (const HTMLMenuElementEventListener* menuEventListener = HTMLMenuElementEventListener::cast(&listener))
+        return m_menu == menuEventListener->m_menu;
+    return false;
+}
+
 inline HTMLMenuElement::HTMLMenuElement(Document& document)
     : HTMLElement(menuTag, document)
 {
     ScriptWrappable::init(this);
+    // Add event listeners

[esprehn, 2014/05/20 05:51:34]

remove comment, it doesn't add value.

[pals, 2014/05/21 07:55:12]

Done.

+    RefPtr<EventListener> listener = HTMLMenuElementEventListener::create(this);
+    this->addEventListener("show", listener.release(), false);

[esprehn, 2014/05/20 05:51:34]

Remove this->, you don't need it.

[pals, 2014/05/21 07:55:12]

Done.

 }
 
 PassRefPtr<HTMLMenuElement> HTMLMenuElement::create(Document& document)
 {
     return adoptRef(new HTMLMenuElement(document));
 }
 
+void HTMLMenuElement::buildCustomProvider()
+{
+    m_menuItems.clear();
+    ContextMenu menu;
+    populateContextMenuItems(this, menu);
+    m_menuProvider = CustomContextMenuProvider::create(this, menu.items());
 }
+
+void HTMLMenuElement::populateContextMenuItems(HTMLMenuElement* menu, ContextMenu& contextMenu)
+{
+    for (Element* nextElement = ElementTraversal::firstWithin(*menu); nextElement; nextElement = ElementTraversal::next(*nextElement)) {

[esprehn, 2014/05/20 05:51:34]

Missing scoping node, this will traverse all the way back up to the Document.
You need ::next(*nextElement, menu)

[pals, 2014/05/21 07:55:12]

Done.

+        if (nextElement->hasTagName(HTMLNames::menuTag)) {
+            ContextMenu subMenu;
+            populateContextMenuItems(toHTMLMenuElement(nextElement), subMenu);
+            contextMenu.appendItem(ContextMenuItem(SubmenuType, ContextMenuItemCustomTagNoAction, nextElement->fastGetAttribute(HTMLNames::labelAttr), &subMenu));
+        } else {
+            m_menuItems.append(toHTMLElement(nextElement));

[esprehn, 2014/05/20 05:51:34]

Does the spec really say that anything is a menu item? This makes <div> and
<span> menu items?

Also this cast is not safe, what if I put an <svg> in there?

[pals, 2014/05/21 07:55:12]

Done.

+            contextMenu.appendItem(ContextMenuItem(ActionType, static_cast<ContextMenuAction>(ContextMenuItemBaseCustomTag + m_menuItems.size() - 1), nextElement->fastGetAttribute(HTMLNames::labelAttr)));

[esprehn, 2014/05/20 05:51:34]

This enum stuff doesn't look right.

[pals, 2014/05/21 07:55:12]

I mostly followed the custom contextmenu implementations from inspector, where
they specify an Id for the custom menu items.
https://code.google.com/p/chromium/codesearch#chromium/src/third_party/WebKit...

But we don't have anything similar for menuitem so I am using menuitem
index/position as the id. any suggestion for an alternative?

+        }
+    }
+}
+
+HTMLElement* HTMLMenuElement::getSelectedMenuItem(unsigned menuId)
+{
+    return m_menuItems[menuId - ContextMenuItemBaseCustomTag];
+}
+
+}