AuthorStyleSheets logic can be simplified. This should be done by the client if needed.
Created attachment 289746 [details] patch
Created attachment 289748 [details] patch
Comment on attachment 289748 [details] patch View in context: https://bugs.webkit.org/attachment.cgi?id=289748&action=review > Source/WebCore/dom/AuthorStyleSheets.h:66 > + enum class ChangeType { ActiveSet, ContentsOrInterpretation }; > + WEBCORE_EXPORT void didChange(ChangeType); Does this really need to be a single function call with an argument? Itβs kind of wordy to call it that way. Maybe we should make this two different named functions. They can share code inside the class and we can use the ChangeType f we find it handy internally, but I suspect there is no real advantage to callers that this is an argument rather than a separate function.
Yeah, that was the plan though you right that I can as well do it in this patch.
Created attachment 289776 [details] patch
Comment on attachment 289776 [details] patch Clearing flags on attachment: 289776 Committed r206361: <http://trac.webkit.org/changeset/206361>
All reviewed patches have been landed. Closing bug.