Bug 154699 – RefCounter value changed callback should be called on all changes (not just zero edge).

• Status: RESOLVED
• Resolution: FIXED
• Component: Web Template Framework

[Gavin Barraclough]

.

[Gavin Barraclough]

Created attachment 272255 [details]
WIP

[Gavin Barraclough]

Created attachment 272261 [details]
Fix

[Anders Carlsson]

Comment on attachment 272261 [details]
Fix

View in context: https://bugs.webkit.org/attachment.cgi?id=272261&action=review

> Source/WTF/wtf/RefCounter.h:60
> +    enum Event { Decrement, Increment };

I would use enum class here.

> Source/WTF/wtf/RefCounter.h:61
> +    using ValueChangeFunction = std::function<void(Event)>;

Space after the return type.

> Source/WTF/wtf/RefCounter.h:63
> +    RefCounter(ValueChangeFunction = [](bool) { });

Maybe use nullptr here (and check m_valueChangeFunction)

[Gavin Barraclough]

Created attachment 272273 [details]
Fix

[WebKit Commit Bot]

Attachment 272273 [details] did not pass style-queue:


ERROR: Source/WebKit2/UIProcess/WebProcessPool.cpp:162:  Code inside a namespace should not be indented.  [whitespace/indent] [4]
ERROR: Source/WebKit2/UIProcess/ProcessThrottler.cpp:39:  Code inside a namespace should not be indented.  [whitespace/indent] [4]
ERROR: Source/WTF/wtf/RefCounter.h:61:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WebCore/platform/VNodeTracker.cpp:43:  Code inside a namespace should not be indented.  [whitespace/indent] [4]
ERROR: Source/WebCore/platform/VNodeTracker.cpp:42:  More than one command on the same line in if  [whitespace/parens] [4]
Total errors found: 5 in 14 files


If any of these errors are false positives, please file a bug against check-webkit-style.

[Geoffrey Garen]

Comment on attachment 272273 [details]
Fix

r=me

[Gavin Barraclough]

Transmitting file data ..............
Committed revision 197178.