In some WebCore, there are a few file with the same header included twice.
Created attachment 147179 [details] patch proposal
Comment on attachment 147179 [details] patch proposal View in context: https://bugs.webkit.org/attachment.cgi?id=147179&action=review > Source/WebCore/bindings/scripts/test/JS/JSTestObj.cpp:-38 > -#include "JSTestObj.h" This is a generated file. Please don't edit it. (Note: It should say that at the top.)
Comment on attachment 147179 [details] patch proposal View in context: https://bugs.webkit.org/attachment.cgi?id=147179&action=review Looks good, but you should try to remove the one so not to make the sorting worse. Also r- based on Adam's comment. > Source/WebCore/rendering/InlineFlowBox.cpp:-39 > -#include "RootInlineBox.h" I think you should remove the first occurrence of it, otherwise you violate (or keep violating) the alphabetic sorting rule.
Created attachment 147396 [details] updated patch
Comment on attachment 147396 [details] updated patch Clearing flags on attachment: 147396 Committed r120279: <http://trac.webkit.org/changeset/120279>
All reviewed patches have been landed. Closing bug.