Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1106)

Unified Diff: Source/core/html/HTMLMenuElement.cpp

Issue 243403006: Implement contextmenu attribute with basic support of <menu> (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Move to oilpan Created 6 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: Source/core/html/HTMLMenuElement.cpp
diff --git a/Source/core/html/HTMLMenuElement.cpp b/Source/core/html/HTMLMenuElement.cpp
index 36a44dc57318e6d07d22f415390cf8273c6f7bea..5e93e675362c83ed0ee7ef554b9eb0dcf8de8767 100644
--- a/Source/core/html/HTMLMenuElement.cpp
+++ b/Source/core/html/HTMLMenuElement.cpp
@@ -23,7 +23,11 @@
#include "config.h"
#include "core/html/HTMLMenuElement.h"
-#include "core/HTMLNames.h"
+#include "core/dom/ElementTraversal.h"
+#include "core/events/RelatedEvent.h"
+#include "core/html/HTMLMenuItemElement.h"
+#include "core/page/ContextMenuController.h"
+#include "core/page/Page.h"
namespace blink {
@@ -37,4 +41,50 @@ inline HTMLMenuElement::HTMLMenuElement(Document& document)
DEFINE_NODE_FACTORY(HTMLMenuElement)
+CustomContextMenuProvider& HTMLMenuElement::ensureMenuProvider()
+{
+ if (!m_menuProvider) {
+ ContextMenu menu;
+ populateContextMenuItems(*this, menu);
+ m_menuProvider = CustomContextMenuProvider::create(this, menu.items());
+ }
+ return *m_menuProvider;
+}
+
+void HTMLMenuElement::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(toHTMLMenuItemElement(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);
+ }
tkent 2014/07/25 04:42:38 if nextElement is neither <menu> nor <menuitem>, t
pals 2014/07/30 09:47:44 Done.
+ }
}
+
+void HTMLMenuElement::defaultEventHandler(Event* event)
+{
+ if (event->type() == EventTypeNames::show && event->isRelatedEvent()) {
+ m_subject = toHTMLElement(toRelatedEvent(event)->relatedTarget()->toNode());
+ ensureMenuProvider();
+ event->setDefaultHandled();
+ }
+}
+
+HTMLMenuItemElement* HTMLMenuElement::menuItemAt(unsigned menuId)
+{
+ int itemIndex = menuId - ContextMenuItemBaseCustomTag;
+ if (itemIndex < 0 || static_cast<unsigned long>(itemIndex) >= m_menuItems.size())
+ return 0;
+ return m_menuItems[itemIndex];
+}
+
+} // namespace blink

Powered by Google App Engine
This is Rietveld 408576698