Add contextmenu attribute for HTMLElement.

Source/core/html/HTMLElement.cpp

 /*
  * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
  *           (C) 1999 Antti Koivisto (koivisto@kde.org)
  * Copyright (C) 2004-2008, 2013, 2014 Apple Inc. All rights reserved.
  * Copyright (C) 2009 Torch Mobile Inc. All rights reserved. (http://www.torchmobile.com/)
  * Copyright (C) 2011 Motorola Mobility. 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
@@ skipping 30 matching lines @@
 #include "core/dom/Text.h"
 #include "core/dom/shadow/ElementShadow.h"
 #include "core/dom/shadow/ShadowRoot.h"
 #include "core/editing/markup.h"
 #include "core/events/EventListener.h"
 #include "core/events/KeyboardEvent.h"
 #include "core/frame/Settings.h"
 #include "core/html/HTMLBRElement.h"
 #include "core/html/HTMLFormElement.h"
 #include "core/html/HTMLInputElement.h"
+#include "core/html/HTMLMenuElement.h"
 #include "core/html/HTMLTemplateElement.h"
 #include "core/html/HTMLTextFormControlElement.h"
 #include "core/html/parser/HTMLParserIdioms.h"
 #include "core/rendering/RenderObject.h"
 #include "platform/text/BidiResolver.h"
 #include "platform/text/BidiTextRun.h"
 #include "platform/text/TextRunIterator.h"
 #include "wtf/StdLibExtras.h"
 #include "wtf/text/CString.h"
 
@@ skipping 833 matching lines @@
         parsedColor.setRGB(parseColorStringWithCrazyLegacyRules(colorString));
 
     style->setProperty(propertyID, cssValuePool().createColorValue(parsedColor.rgb()));
 }
 
 bool HTMLElement::isInteractiveContent() const
 {
     return false;
 }
 
+
+HTMLMenuElement* HTMLElement::contextMenu() const
+{
+    const AtomicString& contextMenuId(fastGetAttribute(contextmenuAttr));
+    if (contextMenuId.isNull())
+        return nullptr;
+
+    Element* element = treeScope().getElementById(contextMenuId);
+    return isHTMLMenuElement(element) ? toHTMLMenuElement(element) : nullptr;

[pals, 2014/08/17 17:18:13]

Specification
http://www.whatwg.org/specs/web-apps/current-work/#reflect
under section
"If a reflecting IDL attribute has the type HTMLElement ..."

[tkent, 2014/08/18 02:24:52]

Off topic: Please refer to multipage version URL.  The single page version is
too slow.

We should have a comment that ignoring type attribute is intentional according
to the standard.

[pals, 2014/08/18 14:23:24]

Done.

+}
+
+void HTMLElement::setContextMenu(HTMLMenuElement* contextMenu)
+{
+    if (!contextMenu)
+        return;
+
+    const AtomicString& contextMenuId(contextMenu->fastGetAttribute(idAttr));
+    const AtomicString& contextMenuType(contextMenu->fastGetAttribute(typeAttr));
+
+    if (!contextMenuId.isNull() && equalIgnoringCase(contextMenuType, "popup") &&  contextMenu->inDocument())

[pals, 2014/08/17 17:18:13]

http://www.whatwg.org/specs/web-apps/current-work/#attr-contextmenu
"The value must be the ID of a menu element in the same home subtree whose type
attribute is in the popup menu state."

[tkent, 2014/08/18 02:24:52]

You misread the specification.  This paragraph is for web authors.  This
paragraph doesn't say UA should reject IDs of <menu type=popup> elements.
According to the reflect behavior specification, we should accept any <menu>
elements.

[tkent, 2014/08/18 02:27:27]

reject IDs of <menu type=popup> -> reject IDs of *no* <menu type=popup>

[pals, 2014/08/18 14:23:24]

Done.

+        setAttribute(contextmenuAttr, contextMenuId);
+    else
+        setAttribute(contextmenuAttr, "");

[pals, 2014/08/17 17:18:13]

Specification
http://www.whatwg.org/specs/web-apps/current-work/#reflect
under section
"On setting, if the given element has an id attribute, and has the same home
subtree as the element of the attribute being set, and the given element is the
first element in that home subtree whose ID is the value of that id attribute,
then the content attribute must be set to the value of that id attribute.
Otherwise, the content attribute must be set to the empty string."

[tkent, 2014/08/18 02:24:52]

"same home subtree" means we need to check treeScope identity in Blink because
the HTML specification doesn't care about Shadow DOM.

There is no code to check "the first element"

[pals, 2014/08/18 14:23:24]

Done. I was doing inDocument() check. Also added a test with shadow dom.

+}
+
 void HTMLElement::defaultEventHandler(Event* event)
 {
     if (event->type() == EventTypeNames::keypress && event->isKeyboardEvent()) {
         handleKeypressEvent(toKeyboardEvent(event));
         if (event->defaultHandled())
             return;
     }
 
     Element::defaultEventHandler(event);
 }
@@ skipping 45 matching lines @@
 #ifndef NDEBUG
 
 // For use in the debugger
 void dumpInnerHTML(blink::HTMLElement*);
 
 void dumpInnerHTML(blink::HTMLElement* element)
 {
     printf("%s\n", element->innerHTML().ascii().data());
 }
 #endif