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

Unified Diff: third_party/WebKit/Source/core/dom/DOMTokenList.cpp

Issue 2898063003: DOMTokenList.contains() should not throw. (Closed)
Patch Set: Move the comment Created 3 years, 7 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: third_party/WebKit/Source/core/dom/DOMTokenList.cpp
diff --git a/third_party/WebKit/Source/core/dom/DOMTokenList.cpp b/third_party/WebKit/Source/core/dom/DOMTokenList.cpp
index e7d658e2a95146c7b8e8692f64b1320db8fa6b13..1446659eac750382785f954e679268238a4d7bc6 100644
--- a/third_party/WebKit/Source/core/dom/DOMTokenList.cpp
+++ b/third_party/WebKit/Source/core/dom/DOMTokenList.cpp
@@ -67,10 +67,8 @@ bool DOMTokenList::ValidateTokenValue(const AtomicString&,
return false;
}
-bool DOMTokenList::contains(const AtomicString& token,
- ExceptionState& exception_state) const {
- if (!ValidateToken(token, exception_state))
- return false;
+// https://dom.spec.whatwg.org/#dom-domtokenlist-contains
+bool DOMTokenList::contains(const AtomicString& token) const {
return ContainsInternal(token);
}
« no previous file with comments | « third_party/WebKit/Source/core/dom/DOMTokenList.h ('k') | third_party/WebKit/Source/core/dom/DOMTokenList.idl » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698