Chromium Code Reviews

Unified Diff: Source/core/css/RuleFeature.cpp

Issue 313963006: Missing return after class invalidation short-cut. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Use existing "root" element instead of creating a new one. Created 6 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
« no previous file with comments | « PerformanceTests/CSS/ClassInvalidation.html ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/core/css/RuleFeature.cpp
diff --git a/Source/core/css/RuleFeature.cpp b/Source/core/css/RuleFeature.cpp
index 5e687d8fc44904f0e903dd10125386f6444d80cd..e92b68727c4c68d3b0ea83e5ad49c3bf61b2398d 100644
--- a/Source/core/css/RuleFeature.cpp
+++ b/Source/core/css/RuleFeature.cpp
@@ -446,8 +446,10 @@ void RuleFeatureSet::scheduleStyleInvalidationForClassChange(const SpaceSplitStr
void RuleFeatureSet::scheduleStyleInvalidationForClassChange(const SpaceSplitString& oldClasses, const SpaceSplitString& newClasses, Element& element)
{
- if (!oldClasses.size())
+ if (!oldClasses.size()) {
scheduleStyleInvalidationForClassChange(newClasses, element);
+ return;
+ }
// Class vectors tend to be very short. This is faster than using a hash table.
BitVector remainingClassBits;
« no previous file with comments | « PerformanceTests/CSS/ClassInvalidation.html ('k') | no next file » | no next file with comments »

Powered by Google App Engine