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

Side by Side Diff: third_party/WebKit/Source/core/dom/Element.cpp

Issue 2402223002: Fix class name which has only space (Closed)
Patch Set: Fix layout tests Created 4 years, 2 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 * (C) 2001 Peter Kelly (pmk@post.com) 4 * (C) 2001 Peter Kelly (pmk@post.com)
5 * (C) 2001 Dirk Mueller (mueller@kde.org) 5 * (C) 2001 Dirk Mueller (mueller@kde.org)
6 * (C) 2007 David Smith (catfish.man@gmail.com) 6 * (C) 2007 David Smith (catfish.man@gmail.com)
7 * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009, 2010, 2012, 2013 Apple Inc. 7 * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009, 2010, 2012, 2013 Apple Inc.
8 * All rights reserved. 8 * All rights reserved.
9 * (C) 2007 Eric Seidel (eric@webkit.org) 9 * (C) 2007 Eric Seidel (eric@webkit.org)
10 * 10 *
(...skipping 1359 matching lines...) Expand 10 before | Expand all | Expand 10 after
1370 const bool shouldFoldCase = document().inQuirksMode(); 1370 const bool shouldFoldCase = document().inQuirksMode();
1371 if (classStringContentType == ClassStringContent::HasClasses) { 1371 if (classStringContentType == ClassStringContent::HasClasses) {
1372 const SpaceSplitString oldClasses = elementData()->classNames(); 1372 const SpaceSplitString oldClasses = elementData()->classNames();
1373 elementData()->setClass(newClassString, shouldFoldCase); 1373 elementData()->setClass(newClassString, shouldFoldCase);
1374 const SpaceSplitString& newClasses = elementData()->classNames(); 1374 const SpaceSplitString& newClasses = elementData()->classNames();
1375 document().styleEngine().classChangedForElement(oldClasses, newClasses, 1375 document().styleEngine().classChangedForElement(oldClasses, newClasses,
1376 *this); 1376 *this);
1377 } else { 1377 } else {
1378 const SpaceSplitString& oldClasses = elementData()->classNames(); 1378 const SpaceSplitString& oldClasses = elementData()->classNames();
1379 document().styleEngine().classChangedForElement(oldClasses, *this); 1379 document().styleEngine().classChangedForElement(oldClasses, *this);
1380 if (classStringContentType == ClassStringContent::WhiteSpaceOnly) 1380 elementData()->setClass(newClassString, shouldFoldCase);
skobes 2016/10/14 20:41:30 I don't quite understand how this works. Setting
1381 elementData()->setClass(newClassString, shouldFoldCase);
1382 else
1383 elementData()->clearClass();
1384 } 1381 }
1385 1382
1386 if (hasRareData()) 1383 if (hasRareData())
1387 elementRareData()->clearClassListValueForQuirksMode(); 1384 elementRareData()->clearClassListValueForQuirksMode();
1388 } 1385 }
1389 1386
1390 bool Element::shouldInvalidateDistributionWhenAttributeChanged( 1387 bool Element::shouldInvalidateDistributionWhenAttributeChanged(
1391 ElementShadow* elementShadow, 1388 ElementShadow* elementShadow,
1392 const QualifiedName& name, 1389 const QualifiedName& name,
1393 const AtomicString& newValue) { 1390 const AtomicString& newValue) {
(...skipping 2688 matching lines...) Expand 10 before | Expand all | Expand 10 after
4082 } 4079 }
4083 4080
4084 DEFINE_TRACE_WRAPPERS(Element) { 4081 DEFINE_TRACE_WRAPPERS(Element) {
4085 if (hasRareData()) { 4082 if (hasRareData()) {
4086 visitor->traceWrappers(elementRareData()); 4083 visitor->traceWrappers(elementRareData());
4087 } 4084 }
4088 ContainerNode::traceWrappers(visitor); 4085 ContainerNode::traceWrappers(visitor);
4089 } 4086 }
4090 4087
4091 } // namespace blink 4088 } // namespace blink
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698