In order to reset all properties back to UA-stylesheet at the shadow boundary, we need the support for `all: revert`.
Created attachment 263386 [details] Patch
Created attachment 263387 [details] Patch
Attachment 263387 [details] did not pass style-queue: ERROR: Source/WebCore/css/StyleResolver.cpp:1854: An else if statement should be written as an if statement when the prior "if" concludes with a return, break, continue or goto statement. [readability/control_flow] [4] ERROR: Source/WebCore/css/StyleResolver.h:309: The parameter name "id" adds no information, so it should be removed. [readability/parameter_name] [5] ERROR: Source/WebCore/css/StyleResolver.h:535: The parameter name "linkMatchMask" adds no information, so it should be removed. [readability/parameter_name] [5] ERROR: Source/WebCore/css/StyleResolver.h:535: The parameter name "matchResult" adds no information, so it should be removed. [readability/parameter_name] [5] ERROR: Source/WebCore/css/CSSValue.h:50: enum members should use InterCaps with an initial capital letter. [readability/enum_casing] [4] ERROR: Source/WebCore/css/CSSValue.h:51: enum members should use InterCaps with an initial capital letter. [readability/enum_casing] [4] Total errors found: 6 in 31 files If any of these errors are false positives, please file a bug against check-webkit-style.
Created attachment 263390 [details] Patch
Attachment 263390 [details] did not pass style-queue: ERROR: Source/WebCore/css/CSSValue.h:50: enum members should use InterCaps with an initial capital letter. [readability/enum_casing] [4] ERROR: Source/WebCore/css/CSSValue.h:51: enum members should use InterCaps with an initial capital letter. [readability/enum_casing] [4] Total errors found: 2 in 31 files If any of these errors are false positives, please file a bug against check-webkit-style.
Comment on attachment 263390 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=263390&action=review > Source/WebCore/WebCore.xcodeproj/project.pbxproj:5503 > BC7D8FF01BD03B6400FFE540 /* CSSUnsetValue.h in Headers */ = {isa = PBXBuildFile; fileRef = BC7D8FEE1BD03B6400FFE540 /* CSSUnsetValue.h */; }; > + BC7D8FF31BD1A47900FFE540 /* CSSRevertValue.cpp in Sources */ = {isa = PBXBuildFile; fileRef = BC7D8FF11BD1A47900FFE540 /* CSSRevertValue.cpp */; settings = {ASSET_TAGS = (); }; }; > + BC7D8FF41BD1A47900FFE540 /* CSSRevertValue.h in Headers */ = {isa = PBXBuildFile; fileRef = BC7D8FF21BD1A47900FFE540 /* CSSRevertValue.h */; settings = {ASSET_TAGS = (); }; }; Please run ./Tools/Scripts/sort-Xcode-project-file on this file. > Source/WebCore/css/StyleResolver.cpp:1858 > + authorRollback = new CascadedProperties(direction, writingMode); unique_ptr? > Source/WebCore/css/StyleResolver.h:425 > - > + whitespace > Source/WebCore/css/StyleResolver.h:457 > + void setAuthorRollback(CascadedProperties* rollback) { m_authorRollback.reset(rollback); } Should take a std::unique_ptr<CascadedProperties>&& and the caller should std::move I think.
Fixed in r191252.
<rdar://problem/23771199>
http://trac.webkit.org/changeset/191252