Bug 149702 – Implement the CSS4 'revert' keyword (including inside the "all" property)

• Status: RESOLVED
• Resolution: FIXED
• Component: CSS

[Ryosuke Niwa]

In order to reset all properties back to UA-stylesheet at the shadow boundary,
we need the support for `all: revert`.

[Dave Hyatt]

Created attachment 263386 [details]
Patch

[Dave Hyatt]

Created attachment 263387 [details]
Patch

[WebKit Commit Bot]

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.

[Dave Hyatt]

Created attachment 263390 [details]
Patch

[WebKit Commit Bot]

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.

[Simon Fraser (smfr)]

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.

[Dave Hyatt]

Fixed in r191252.

[Radar WebKit Bug Importer]

<rdar://problem/23771199>

[yisibl]

http://trac.webkit.org/changeset/191252