Implement contextmenu attribute with basic support of <menu>

Source/core/page/ContextMenuController.cpp

 /*
  * Copyright (C) 2006, 2007 Apple Inc. All rights reserved.
  * Copyright (C) 2010 Igalia S.L
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
@@ skipping 10 matching lines @@
  * PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY
  * OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
  * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
 #include "config.h"
 #include "core/page/ContextMenuController.h"
 
 #include "core/dom/Document.h"
+#include "core/dom/ElementTraversal.h"
+#include "core/dom/Node.h"
 #include "core/events/Event.h"
+#include "core/events/EventDispatchMediator.h"
+#include "core/events/EventDispatcher.h"
 #include "core/events/MouseEvent.h"
-#include "core/dom/Node.h"
+#include "core/events/RelatedEvent.h"
 #include "core/frame/LocalFrame.h"
+#include "core/html/HTMLMenuElement.h"
 #include "core/page/ContextMenuClient.h"
 #include "core/page/ContextMenuProvider.h"
+#include "core/page/CustomContextMenuProvider.h"
 #include "core/page/EventHandler.h"
 #include "platform/ContextMenu.h"
 #include "platform/ContextMenuItem.h"
 
 namespace blink {
 
+using namespace HTMLNames;
+
 ContextMenuController::ContextMenuController(Page*, ContextMenuClient* client)
     : m_client(client)
 {
     ASSERT_ARG(client, client);
 }
 
 ContextMenuController::~ContextMenuController()
 {
 }
 
@@ skipping 14 matching lines @@
 
 void ContextMenuController::documentDetached(Document* document)
 {
     if (Node* innerNode = m_hitTestResult.innerNode()) {
         // Invalidate the context menu info if its target document is detached.
         if (innerNode->document() == document)
             clearContextMenu();
     }
 }
 
+void ContextMenuController::populateContextMenuItems(const HTMLMenuElement& menu, ContextMenu& contextMenu)
+{
+    HTMLElement* nextElement = Traversal<HTMLElement>::firstWithin(menu);
+    while (nextElement) {
+        if (isHTMLMenuElement(*nextElement)) {
+            ContextMenu subMenu;
+            populateContextMenuItems(*toHTMLMenuElement(nextElement), subMenu);
+            contextMenu.appendItem(ContextMenuItem(SubmenuType, ContextMenuItemCustomTagNoAction, nextElement->fastGetAttribute(labelAttr), &subMenu));
+            nextElement = Traversal<HTMLElement>::nextSibling(*nextElement);
+        } else if (isHTMLMenuItemElement(*nextElement)) {
+            m_menuItems.append(nextElement);
+            contextMenu.appendItem(ContextMenuItem(ActionType, static_cast<ContextMenuAction>(ContextMenuItemBaseCustomTag + m_menuItems.size() - 1), nextElement->fastGetAttribute(labelAttr)));
+            if (ContextMenuItemBaseCustomTag + m_menuItems.size() >= ContextMenuItemLastCustomTag)
+                break;
+            nextElement = Traversal<HTMLElement>::next(*nextElement, &menu);
+        } else {
+            nextElement = Traversal<HTMLElement>::next(*nextElement, &menu);
+        }
+    }
+}
+
+HTMLElement* ContextMenuController::menuItemAt(unsigned menuId)
+{
+    int itemIndex = menuId - ContextMenuItemBaseCustomTag;
+    if (itemIndex < 0 || static_cast<unsigned long>(itemIndex) >= m_menuItems.size())
+        return 0;
+    return m_menuItems[itemIndex];
+}
+
 void ContextMenuController::handleContextMenuEvent(Event* event)
 {
     m_contextMenu = createContextMenu(event);
     if (!m_contextMenu)
         return;
 
+    Node* node = event->target()->toNode();

[tkent, 2014/07/31 06:07:56]

I recommend to make new function for this block of new code.

[pals, 2014/08/01 10:03:08]

Done.

+    if (node && node->isHTMLElement() && RuntimeEnabledFeatures::contextMenuEnabled()) {
+        const HTMLElement& element = toHTMLElement(*node);
+        if (RefPtrWillBeRawPtr<HTMLMenuElement> menuElement = element.contextMenu()) {
+            RefPtr<RelatedEvent> relatedEvent = RelatedEvent::create(EventTypeNames::show);
+            relatedEvent->setRelatedTarget(node);
+            m_subject = toHTMLElement(node);

[tkent, 2014/07/31 06:07:56]

toHTMLElement is unnecessary we already have |element|.

[pals, 2014/08/01 10:03:08]

Done.

+            bool notCanceled = menuElement->dispatchEvent(relatedEvent.release());

[tkent, 2014/07/31 06:07:56]

|notCanceled| is unnecessary.
  if (menuElement->dispatchEvet...) {

If you'd like to add a variable for code readability, the name should not
contain 'not'.  It should be |shouldProceed| or something.

[pals, 2014/08/01 10:03:08]

Done.

+            if (notCanceled) {
+                ContextMenu menu;

[tkent, 2014/07/31 06:07:56]

Because dispatchEvent runs arbitrary JavaScript code, |menuElement| might be
disassociated with the subject element. We need to check
HTMLElement::contextMenu() again.

[pals, 2014/08/01 10:03:08]

Done.

+                populateContextMenuItems(*menuElement, menu);

[tkent, 2014/07/31 06:07:56]

I'm sorry I changed my mind.  populateContextMenuItems() should be a member
function of CustomContextMenuProvider.

[pals, 2014/08/01 10:03:08]

Done. Moved to CustomContextMenuProvider. After this move the code looks more
logically organised. Thanks. 

+                m_menuProvider = CustomContextMenuProvider::create(&menuElement->document(), menu.items());
+                m_menuProvider->populateContextMenu(m_contextMenu.get());
+            }
+        }
+    }
     showContextMenu(event);
 }
 
 void ContextMenuController::showContextMenu(Event* event, PassRefPtr<ContextMenuProvider> menuProvider)
 {
     m_menuProvider = menuProvider;
 
     m_contextMenu = createContextMenu(event);
     if (!m_contextMenu) {
         clearContextMenu();
@@ skipping 57 matching lines @@
     ASSERT(item->type() == ActionType || item->type() == CheckableActionType);
 
     if (item->action() < ContextMenuItemBaseCustomTag || item->action() > ContextMenuItemLastCustomTag)
         return;
 
     ASSERT(m_menuProvider);
     m_menuProvider->contextMenuItemSelected(item);
 }
 
 } // namespace blink