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

Unified Diff: Source/core/html/HTMLSelectElement.h

Issue 120373005: OnChange event should fire if the multiple selection changes in listbox using mouse (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: removed unnecessary blank lines in layout test Created 7 years 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/HTMLSelectElement.h
diff --git a/Source/core/html/HTMLSelectElement.h b/Source/core/html/HTMLSelectElement.h
old mode 100644
new mode 100755
index 7564360b5cc198fae350c5ca71de5995332768a6..6f5178544b690ac244a9da7253d34e5fdc70e7c1
--- a/Source/core/html/HTMLSelectElement.h
+++ b/Source/core/html/HTMLSelectElement.h
@@ -69,6 +69,10 @@ public:
String value() const;
void setValue(const String&);
+ bool isOnChangeFired() const { return m_onChangeFired; }
+ void setOnChangeFired(bool fired) { m_onChangeFired = fired; }
+ bool isLastOnChangeSelectionEmpty() { return m_lastOnChangeSelection.isEmpty(); }
+
PassRefPtr<HTMLOptionsCollection> options();
PassRefPtr<HTMLCollection> selectedOptions();
@@ -209,6 +213,7 @@ private:
bool m_activeSelectionState;
mutable bool m_shouldRecalcListItems;
bool m_isParsingInProgress;
+ bool m_onChangeFired;
};
DEFINE_NODE_TYPE_CASTS(HTMLSelectElement, hasTagName(HTMLNames::selectTag));

Powered by Google App Engine
This is Rietveld 408576698