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

Side by Side Diff: Source/core/html/HTMLMenuElement.h

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 unified diff | Download patch
OLDNEW
1 /* 1 /*
2 * Copyright (C) 1999 Lars Knoll (knoll@kde.org) 2 * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
3 * (C) 1999 Antti Koivisto (koivisto@kde.org) 3 * (C) 1999 Antti Koivisto (koivisto@kde.org)
4 * Copyright (C) 2010 Apple Inc. All rights reserved. 4 * Copyright (C) 2010 Apple Inc. All rights reserved.
5 * 5 *
6 * This library is free software; you can redistribute it and/or 6 * This library is free software; you can redistribute it and/or
7 * modify it under the terms of the GNU Library General Public 7 * modify it under the terms of the GNU Library General Public
8 * License as published by the Free Software Foundation; either 8 * License as published by the Free Software Foundation; either
9 * version 2 of the License, or (at your option) any later version. 9 * version 2 of the License, or (at your option) any later version.
10 * 10 *
11 * This library is distributed in the hope that it will be useful, 11 * This library is distributed in the hope that it will be useful,
12 * but WITHOUT ANY WARRANTY; without even the implied warranty of 12 * but WITHOUT ANY WARRANTY; without even the implied warranty of
13 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU 13 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
14 * Library General Public License for more details. 14 * Library General Public License for more details.
15 * 15 *
16 * You should have received a copy of the GNU Library General Public License 16 * You should have received a copy of the GNU Library General Public License
17 * along with this library; see the file COPYING.LIB. If not, write to 17 * along with this library; see the file COPYING.LIB. If not, write to
18 * the Free Software Foundation, Inc., 51 Franklin Street, Fifth Floor, 18 * the Free Software Foundation, Inc., 51 Franklin Street, Fifth Floor,
19 * Boston, MA 02110-1301, USA. 19 * Boston, MA 02110-1301, USA.
20 * 20 *
21 */ 21 */
22 22
23 #ifndef HTMLMenuElement_h 23 #ifndef HTMLMenuElement_h
24 #define HTMLMenuElement_h 24 #define HTMLMenuElement_h
25 25
26 #include "core/html/CustomContextMenuProvider.h"
26 #include "core/html/HTMLElement.h" 27 #include "core/html/HTMLElement.h"
28 #include "core/page/ContextMenuProvider.h"
29 #include "platform/ContextMenu.h"
30 #include "platform/ContextMenuItem.h"
27 31
28 namespace blink { 32 namespace blink {
29 33
34 class HTMLMenuElement;
tkent 2014/07/25 04:42:38 This is unnecessary.
pals 2014/07/30 09:47:44 Done.
35
30 class HTMLMenuElement FINAL : public HTMLElement { 36 class HTMLMenuElement FINAL : public HTMLElement {
31 public: 37 public:
32 DECLARE_NODE_FACTORY(HTMLMenuElement); 38 DECLARE_NODE_FACTORY(HTMLMenuElement);
33 39
40 virtual void defaultEventHandler(Event*) OVERRIDE;
41
42 CustomContextMenuProvider* menuProvider() { return m_menuProvider.get(); }
43 CustomContextMenuProvider& ensureMenuProvider();
44 void populateContextMenuItems(const HTMLMenuElement&, ContextMenu&);
45 HTMLMenuItemElement* menuItemAt(unsigned menuId);
46 HTMLElement* subjectElement() { return m_subject.get(); }
47
34 private: 48 private:
35 explicit HTMLMenuElement(Document&); 49 explicit HTMLMenuElement(Document&);
50
51 RefPtrWillBeMember<CustomContextMenuProvider> m_menuProvider;
tkent 2014/07/25 04:42:38 HTMLMenuElement should not keep CustomContextMenuP
pals 2014/07/30 09:47:44 Done.
52 RefPtrWillBeMember<HTMLElement> m_subject;
tkent 2014/07/25 04:42:38 This member is not referred.
pals 2014/07/30 09:47:44 This is referred in ContextMenuController.cpp.
53 Vector<HTMLMenuItemElement*> m_menuItems;
tkent 2014/07/25 04:42:38 should be WillBeHeapVector<RawPtrWillBeMember<HTML
pals 2014/07/30 09:47:44 Moved this code to ContextMenuController.cpp.
36 }; 54 };
37 55
38 } //namespace 56 } // namespace blink
39 57
40 #endif 58 #endif
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698