Use WTF::Function instead of std::function in more places in WebCore/ to avoid copying.
Created attachment 313260 [details] Patch
Attachment 313260 [details] did not pass style-queue: ERROR: Source/WebCore/css/CSSValue.cpp:104: Extra space before ( in function call [whitespace/parens] [4] ERROR: Source/WebCore/css/CSSFontFaceSrcValue.cpp:88: Extra space before ( in function call [whitespace/parens] [4] ERROR: Source/WebCore/css/CSSCrossfadeValue.h:55: Extra space before ( in function call [whitespace/parens] [4] ERROR: Source/WebCore/page/FrameView.cpp:2565: Extra space before ( in function call [whitespace/parens] [4] ERROR: Source/WebCore/page/FrameView.h:686: Extra space before ( in function call [whitespace/parens] [4] ERROR: Source/WebCore/loader/archive/cf/LegacyWebArchive.cpp:409: Extra space before ( in function call [whitespace/parens] [4] ERROR: Source/WebCore/loader/archive/cf/LegacyWebArchive.cpp:468: Extra space before ( in function call [whitespace/parens] [4] ERROR: Source/WebCore/css/CSSValue.h:125: Extra space before ( in function call [whitespace/parens] [4] ERROR: Source/WebCore/css/CSSFilterImageValue.h:67: Extra space before ( in function call [whitespace/parens] [4] ERROR: Source/WebCore/contentextensions/CombinedURLFilters.h:33: Alphabetical sorting problem. [build/include_order] [4] ERROR: Source/WebCore/loader/cache/MemoryCache.cpp:276: Extra space before ( in function call [whitespace/parens] [4] ERROR: Source/WebCore/platform/graphics/MediaPlayerPrivate.h:33: Alphabetical sorting problem. [build/include_order] [4] ERROR: Source/WebCore/css/CSSValueList.cpp:139: Extra space before ( in function call [whitespace/parens] [4] ERROR: Source/WebCore/css/StyleSheetContents.h:91: Extra space before ( in function call [whitespace/parens] [4] ERROR: Source/WebCore/page/DOMWindow.cpp:2180: Extra space before ( in function call [whitespace/parens] [4] ERROR: Source/WebCore/page/DOMWindow.cpp:2305: Extra space before ( in function call [whitespace/parens] [4] ERROR: Source/WebCore/css/CSSCrossfadeValue.cpp:209: Extra space before ( in function call [whitespace/parens] [4] ERROR: Source/WebCore/css/CSSImageValue.h:52: Extra space before ( in function call [whitespace/parens] [4] ERROR: Source/WebCore/css/CSSFontFaceSrcValue.h:68: Extra space before ( in function call [whitespace/parens] [4] ERROR: Source/WebCore/loader/archive/cf/LegacyWebArchive.h:63: Extra space before ( in function call [whitespace/parens] [4] ERROR: Source/WebCore/css/CSSImageValue.cpp:84: Extra space before ( in function call [whitespace/parens] [4] ERROR: Source/WebCore/css/StyleProperties.h:136: Extra space before ( in function call [whitespace/parens] [4] ERROR: Source/WebCore/css/StyleProperties.cpp:1150: Extra space before ( in function call [whitespace/parens] [4] ERROR: Source/WebCore/css/CSSImageSetValue.cpp:155: Extra space before ( in function call [whitespace/parens] [4] ERROR: Source/WebCore/css/CSSFilterImageValue.cpp:167: Extra space before ( in function call [whitespace/parens] [4] ERROR: Source/WebCore/css/CSSImageSetValue.h:57: Extra space before ( in function call [whitespace/parens] [4] ERROR: Source/WebCore/css/CSSValueList.h:72: Extra space before ( in function call [whitespace/parens] [4] ERROR: Source/WebCore/css/StyleSheetContents.cpp:438: Extra space before ( in function call [whitespace/parens] [4] ERROR: Source/WebCore/css/StyleSheetContents.cpp:483: Extra space before ( in function call [whitespace/parens] [4] Total errors found: 29 in 76 files If any of these errors are false positives, please file a bug against check-webkit-style.
Comment on attachment 313260 [details] Patch Attachment 313260 [details] did not pass mac-ews (mac): Output: http://webkit-queues.webkit.org/results/3955562 New failing tests: inspector/canvas/create-canvas-contexts.html
Created attachment 313262 [details] Archive of layout-test-results from ews102 for mac-elcapitan The attached test failures were seen while running run-webkit-tests on the mac-ews. Bot: ews102 Port: mac-elcapitan Platform: Mac OS X 10.11.6
Created attachment 313314 [details] Patch
Attachment 313314 [details] did not pass style-queue: ERROR: Source/WebCore/css/CSSValue.cpp:104: Extra space before ( in function call [whitespace/parens] [4] ERROR: Source/WebCore/css/CSSFontFaceSrcValue.cpp:88: Extra space before ( in function call [whitespace/parens] [4] ERROR: Source/WebCore/css/CSSCrossfadeValue.h:55: Extra space before ( in function call [whitespace/parens] [4] ERROR: Source/WebCore/contentextensions/CombinedURLFilters.h:33: Alphabetical sorting problem. [build/include_order] [4] ERROR: Source/WebCore/page/FrameView.cpp:2565: Extra space before ( in function call [whitespace/parens] [4] ERROR: Source/WebCore/page/FrameView.h:684: Extra space before ( in function call [whitespace/parens] [4] ERROR: Source/WebCore/loader/archive/cf/LegacyWebArchive.cpp:409: Extra space before ( in function call [whitespace/parens] [4] ERROR: Source/WebCore/loader/archive/cf/LegacyWebArchive.cpp:468: Extra space before ( in function call [whitespace/parens] [4] ERROR: Source/WebCore/css/CSSValue.h:125: Extra space before ( in function call [whitespace/parens] [4] ERROR: Source/WebCore/css/CSSFilterImageValue.h:67: Extra space before ( in function call [whitespace/parens] [4] ERROR: Source/WebCore/loader/cache/MemoryCache.cpp:276: Extra space before ( in function call [whitespace/parens] [4] ERROR: Source/WebCore/platform/graphics/MediaPlayerPrivate.h:33: Alphabetical sorting problem. [build/include_order] [4] ERROR: Source/WebCore/css/CSSValueList.cpp:139: Extra space before ( in function call [whitespace/parens] [4] ERROR: Source/WebCore/css/StyleSheetContents.h:91: Extra space before ( in function call [whitespace/parens] [4] ERROR: Source/WebCore/page/DOMWindow.cpp:2180: Extra space before ( in function call [whitespace/parens] [4] ERROR: Source/WebCore/page/DOMWindow.cpp:2305: Extra space before ( in function call [whitespace/parens] [4] ERROR: Source/WebCore/css/CSSCrossfadeValue.cpp:209: Extra space before ( in function call [whitespace/parens] [4] ERROR: Source/WebCore/css/CSSImageValue.h:52: Extra space before ( in function call [whitespace/parens] [4] ERROR: Source/WebCore/css/CSSFontFaceSrcValue.h:68: Extra space before ( in function call [whitespace/parens] [4] ERROR: Source/WebCore/loader/archive/cf/LegacyWebArchive.h:63: Extra space before ( in function call [whitespace/parens] [4] ERROR: Source/WebCore/css/CSSImageValue.cpp:84: Extra space before ( in function call [whitespace/parens] [4] ERROR: Source/WebCore/css/StyleProperties.h:136: Extra space before ( in function call [whitespace/parens] [4] ERROR: Source/WebCore/css/StyleProperties.cpp:1150: Extra space before ( in function call [whitespace/parens] [4] ERROR: Source/WebCore/css/CSSImageSetValue.cpp:155: Extra space before ( in function call [whitespace/parens] [4] ERROR: Source/WebCore/css/CSSFilterImageValue.cpp:167: Extra space before ( in function call [whitespace/parens] [4] ERROR: Source/WebCore/css/CSSImageSetValue.h:57: Extra space before ( in function call [whitespace/parens] [4] ERROR: Source/WebCore/css/CSSValueList.h:72: Extra space before ( in function call [whitespace/parens] [4] ERROR: Source/WebCore/css/StyleSheetContents.cpp:438: Extra space before ( in function call [whitespace/parens] [4] ERROR: Source/WebCore/css/StyleSheetContents.cpp:483: Extra space before ( in function call [whitespace/parens] [4] Total errors found: 29 in 77 files If any of these errors are false positives, please file a bug against check-webkit-style.
Comment on attachment 313314 [details] Patch rs=me
Comment on attachment 313314 [details] Patch Clearing flags on attachment: 313314 Committed r218501: <http://trac.webkit.org/changeset/218501>
All reviewed patches have been landed. Closing bug.