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

Side by Side Diff: third_party/WebKit/Source/web/PopupMenuImpl.h

Issue 1610553003: Remove unused arguments of blink::PopupMenu::show(). (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 11 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 // Copyright (c) 2014 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef PopupMenuImpl_h 5 #ifndef PopupMenuImpl_h
6 #define PopupMenuImpl_h 6 #define PopupMenuImpl_h
7 7
8 #include "core/page/PagePopupClient.h" 8 #include "core/page/PagePopupClient.h"
9 #include "platform/PopupMenu.h" 9 #include "platform/PopupMenu.h"
10 10
(...skipping 20 matching lines...) Expand all
31 private: 31 private:
32 PopupMenuImpl(ChromeClientImpl*, HTMLSelectElement&); 32 PopupMenuImpl(ChromeClientImpl*, HTMLSelectElement&);
33 33
34 class ItemIterationContext; 34 class ItemIterationContext;
35 void addOption(ItemIterationContext&, HTMLOptionElement&); 35 void addOption(ItemIterationContext&, HTMLOptionElement&);
36 void addOptGroup(ItemIterationContext&, HTMLOptGroupElement&); 36 void addOptGroup(ItemIterationContext&, HTMLOptGroupElement&);
37 void addSeparator(ItemIterationContext&, HTMLHRElement&); 37 void addSeparator(ItemIterationContext&, HTMLHRElement&);
38 void addElementStyle(ItemIterationContext&, HTMLElement&); 38 void addElementStyle(ItemIterationContext&, HTMLElement&);
39 39
40 // PopupMenu functions: 40 // PopupMenu functions:
41 void show(const FloatQuad& controlPosition, const IntSize& controlSize, int index) override; 41 void show() override;
42 void hide() override; 42 void hide() override;
43 void disconnectClient() override; 43 void disconnectClient() override;
44 void updateFromElement() override; 44 void updateFromElement() override;
45 45
46 // PagePopupClient functions: 46 // PagePopupClient functions:
47 void writeDocument(SharedBuffer*) override; 47 void writeDocument(SharedBuffer*) override;
48 void selectFontsFromOwnerDocument(Document&) override; 48 void selectFontsFromOwnerDocument(Document&) override;
49 void setValueAndClosePopup(int, const String&) override; 49 void setValueAndClosePopup(int, const String&) override;
50 void setValue(const String&) override; 50 void setValue(const String&) override;
51 void closePopup() override; 51 void closePopup() override;
52 Element& ownerElement() override; 52 Element& ownerElement() override;
53 Locale& locale() override; 53 Locale& locale() override;
54 void didClosePopup() override; 54 void didClosePopup() override;
55 55
56 RawPtrWillBeMember<ChromeClientImpl> m_chromeClient; 56 RawPtrWillBeMember<ChromeClientImpl> m_chromeClient;
57 RawPtrWillBeMember<HTMLSelectElement> m_ownerElement; 57 RawPtrWillBeMember<HTMLSelectElement> m_ownerElement;
58 PagePopup* m_popup; 58 PagePopup* m_popup;
59 bool m_needsUpdate; 59 bool m_needsUpdate;
60 }; 60 };
61 61
62 } 62 }
63 63
64 #endif // PopupMenuImpl_h 64 #endif // PopupMenuImpl_h
OLDNEW
« no previous file with comments | « third_party/WebKit/Source/web/ExternalPopupMenu.cpp ('k') | third_party/WebKit/Source/web/PopupMenuImpl.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698