Bug 121820

Summary: Remove *WebKit* prefix in WebKitCSS*Value classes
Product: WebKit Reporter: Gyuyoung Kim <gyuyoung.kim>
Component: CSSAssignee: Gyuyoung Kim <gyuyoung.kim>
Status: RESOLVED LATER    
Severity: Normal CC: allan.jensen, buildbot, cdumez, commit-queue, darin, esprehn+autocc, glenn, gyuyoung.kim, kling, kondapallykalyan, macpherson, menard, rakuco, rniwa
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 121776    
Attachments:
Description Flags
Patch
none
Patch
buildbot: commit-queue-
Archive of layout-test-results from webkit-ews-14 for mac-mountainlion-wk2
none
Archive of layout-test-results from webkit-ews-03 for mac-mountainlion
none
Archive of layout-test-results from webkit-ews-08 for mac-mountainlion none

Gyuyoung Kim
Reported 2013-09-23 18:50:37 PDT
As Kling suggested on irc, we need to remove *WebKit* prefix from some isFooValue() functions to apply CSS_VALUE_TYPE_CASTS to all child CSSValue classes.
Attachments
Patch (10.66 KB, patch)
2013-09-23 18:56 PDT, Gyuyoung Kim 		no flags
DetailsFormatted DiffDiff
Patch (295.66 KB, patch)
2013-09-23 22:00 PDT, Gyuyoung Kim 		buildbot: commit-queue-
DetailsFormatted DiffDiff
Archive of layout-test-results from webkit-ews-14 for mac-mountainlion-wk2 (641.83 KB, application/zip)
2013-09-23 23:54 PDT, Build Bot 		no flags
Details
Archive of layout-test-results from webkit-ews-03 for mac-mountainlion (745.47 KB, application/zip)
2013-09-24 01:16 PDT, Build Bot 		no flags
Details
Archive of layout-test-results from webkit-ews-08 for mac-mountainlion (663.91 KB, application/zip)
2013-09-24 02:30 PDT, Build Bot 		no flags
Details
Show Obsolete (2) View All Add attachment proposed patch, testcase, etc.
Gyuyoung Kim
Comment 1 2013-09-23 18:56:03 PDT
Created attachment 212419 [details] Patch
Gyuyoung Kim
Comment 2 2013-09-23 19:18:15 PDT
Comment on attachment 212419 [details] Patch Unfortunately, we should *WebKit* prefix from class names to apply CSS_VALUE_TYPE_CASTS to WebKitCSS*Value classes. I wonder if I can remove it from WebKitCSSArrayFunctionValue, WebKitCSSMatFunctionValue, WebKitCSSMixFunctionValue, WebKitCSSShaderValue and WebKitCSSSVGDocumentValue. In blink case, it removed the *WebKit* prefix. If we can't, I'd like to land a patch of Bug 121776 at the moment.
Gyuyoung Kim
Comment 3 2013-09-23 22:00:02 PDT
Created attachment 212425 [details] Patch
Gyuyoung Kim
Comment 4 2013-09-23 22:03:26 PDT
I'm not sure if this removal is acceptable. If not, I'd like to land Bug 121776 for now.
WebKit Commit Bot
Comment 5 2013-09-23 22:06:28 PDT
Attachment 212425 [details] did not pass style-queue: Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'Source/WebCore/CMakeLists.txt', u'Source/WebCore/ChangeLog', u'Source/WebCore/DerivedSources.cpp', u'Source/WebCore/DerivedSources.make', u'Source/WebCore/DerivedSources.pri', u'Source/WebCore/GNUmakefile.list.am', u'Source/WebCore/Target.pri', u'Source/WebCore/WebCore.order', u'Source/WebCore/WebCore.vcxproj/WebCore.vcxproj', u'Source/WebCore/WebCore.vcxproj/WebCore.vcxproj.filters', u'Source/WebCore/WebCore.xcodeproj/project.pbxproj', u'Source/WebCore/bindings/js/JSCSSValueCustom.cpp', u'Source/WebCore/bindings/objc/DOMCSS.mm', u'Source/WebCore/css/CSSArrayFunctionValue.cpp', u'Source/WebCore/css/CSSArrayFunctionValue.h', u'Source/WebCore/css/CSSComputedStyleDeclaration.cpp', u'Source/WebCore/css/CSSFilterValue.cpp', u'Source/WebCore/css/CSSFilterValue.h', u'Source/WebCore/css/CSSFilterValue.idl', u'Source/WebCore/css/CSSMatFunctionValue.cpp', u'Source/WebCore/css/CSSMatFunctionValue.h', u'Source/WebCore/css/CSSMixFunctionValue.cpp', u'Source/WebCore/css/CSSMixFunctionValue.h', u'Source/WebCore/css/CSSMixFunctionValue.idl', u'Source/WebCore/css/CSSParser.cpp', u'Source/WebCore/css/CSSParser.h', u'Source/WebCore/css/CSSSVGDocumentValue.cpp', u'Source/WebCore/css/CSSSVGDocumentValue.h', u'Source/WebCore/css/CSSShaderValue.cpp', u'Source/WebCore/css/CSSShaderValue.h', u'Source/WebCore/css/CSSTransformValue.cpp', u'Source/WebCore/css/CSSTransformValue.h', u'Source/WebCore/css/CSSTransformValue.idl', u'Source/WebCore/css/CSSValue.cpp', u'Source/WebCore/css/CSSValue.h', u'Source/WebCore/css/StyleResolver.cpp', u'Source/WebCore/css/StyleResolver.h', u'Source/WebCore/css/TransformFunctions.cpp', u'Source/WebCore/css/TransformFunctions.h', u'Source/WebCore/css/WebKitCSSArrayFunctionValue.cpp', u'Source/WebCore/css/WebKitCSSArrayFunctionValue.h', u'Source/WebCore/css/WebKitCSSFilterValue.cpp', u'Source/WebCore/css/WebKitCSSFilterValue.h', u'Source/WebCore/css/WebKitCSSFilterValue.idl', u'Source/WebCore/css/WebKitCSSMatFunctionValue.cpp', u'Source/WebCore/css/WebKitCSSMatFunctionValue.h', u'Source/WebCore/css/WebKitCSSMixFunctionValue.cpp', u'Source/WebCore/css/WebKitCSSMixFunctionValue.h', u'Source/WebCore/css/WebKitCSSMixFunctionValue.idl', u'Source/WebCore/css/WebKitCSSSVGDocumentValue.cpp', u'Source/WebCore/css/WebKitCSSSVGDocumentValue.h', u'Source/WebCore/css/WebKitCSSShaderValue.cpp', u'Source/WebCore/css/WebKitCSSShaderValue.h', u'Source/WebCore/css/WebKitCSSTransformValue.cpp', u'Source/WebCore/css/WebKitCSSTransformValue.h', u'Source/WebCore/css/WebKitCSSTransformValue.idl', u'Source/WebCore/rendering/style/StylePendingShader.h']" exit_code: 1 Source/WebCore/css/StyleResolver.cpp:125: Alphabetical sorting problem. [build/include_order] [4] Source/WebCore/css/StyleResolver.cpp:133: Alphabetical sorting problem. [build/include_order] [4] Source/WebCore/css/StyleResolver.cpp:155: Alphabetical sorting problem. [build/include_order] [4] Source/WebCore/css/StyleResolver.cpp:182: Alphabetical sorting problem. [build/include_order] [4] Source/WebCore/rendering/style/StylePendingShader.h:37: Alphabetical sorting problem. [build/include_order] [4] Source/WebCore/bindings/js/JSCSSValueCustom.cpp:34: Alphabetical sorting problem. [build/include_order] [4] Source/WebCore/bindings/js/JSCSSValueCustom.cpp:34: Alphabetical sorting problem. [build/include_order] [4] Source/WebCore/bindings/js/JSCSSValueCustom.cpp:38: Alphabetical sorting problem. [build/include_order] [4] Source/WebCore/bindings/js/JSCSSValueCustom.cpp:43: Alphabetical sorting problem. [build/include_order] [4] Source/WebCore/css/CSSValue.cpp:60: Alphabetical sorting problem. [build/include_order] [4] Source/WebCore/css/CSSComputedStyleDeclaration.cpp:64: Alphabetical sorting problem. [build/include_order] [4] Source/WebCore/css/CSSComputedStyleDeclaration.cpp:79: Alphabetical sorting problem. [build/include_order] [4] Source/WebCore/css/CSSComputedStyleDeclaration.cpp:86: Alphabetical sorting problem. [build/include_order] [4] Source/WebCore/css/TransformFunctions.cpp:46: Alphabetical sorting problem. [build/include_order] [4] Source/WebCore/DerivedSources.cpp:517: Alphabetical sorting problem. [build/include_order] [4] Source/WebCore/css/CSSParser.cpp:88: Alphabetical sorting problem. [build/include_order] [4] Total errors found: 16 in 40 files If any of these errors are false positives, please file a bug against check-webkit-style.
Build Bot
Comment 6 2013-09-23 23:54:11 PDT
Comment on attachment 212425 [details] Patch Attachment 212425 [details] did not pass mac-wk2-ews (mac-wk2): Output: http://webkit-queues.appspot.com/results/2092193 New failing tests: js/dom/global-constructors-attributes.html transforms/2d/transform-value-types.html css3/filters/custom/custom-filter-property-computed-style.html css3/filters/custom/custom-filter-mix-bindings.html css3/filters/custom-with-at-rule-syntax/parsing-custom-function-valid.html css3/filters/filter-property-computed-style.html css3/filters/custom/custom-filter-property-parsing.html css3/filters/filter-property-parsing.html platform/mac/fast/dom/wrapper-classes-objc.html
Build Bot
Comment 7 2013-09-23 23:54:15 PDT
Created attachment 212433 [details] Archive of layout-test-results from webkit-ews-14 for mac-mountainlion-wk2 The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews. Bot: webkit-ews-14 Port: mac-mountainlion-wk2 Platform: Mac OS X 10.8.5
Build Bot
Comment 8 2013-09-24 01:15:58 PDT
Comment on attachment 212425 [details] Patch Attachment 212425 [details] did not pass mac-ews (mac): Output: http://webkit-queues.appspot.com/results/1808823 New failing tests: js/dom/global-constructors-attributes.html transforms/2d/transform-value-types.html css3/filters/custom/custom-filter-property-computed-style.html css3/filters/custom/custom-filter-mix-bindings.html css3/filters/custom-with-at-rule-syntax/parsing-custom-function-valid.html css3/filters/filter-property-computed-style.html css3/filters/custom/custom-filter-property-parsing.html css3/filters/filter-property-parsing.html platform/mac/fast/dom/wrapper-classes-objc.html
Build Bot
Comment 9 2013-09-24 01:16:02 PDT
Created attachment 212436 [details] Archive of layout-test-results from webkit-ews-03 for mac-mountainlion The attached test failures were seen while running run-webkit-tests on the mac-ews. Bot: webkit-ews-03 Port: mac-mountainlion Platform: Mac OS X 10.8.5
Build Bot
Comment 10 2013-09-24 02:30:34 PDT
Comment on attachment 212425 [details] Patch Attachment 212425 [details] did not pass mac-ews (mac): Output: http://webkit-queues.appspot.com/results/1908397 New failing tests: js/dom/global-constructors-attributes.html transforms/2d/transform-value-types.html css3/filters/custom/custom-filter-property-computed-style.html css3/filters/custom/custom-filter-mix-bindings.html css3/filters/custom-with-at-rule-syntax/parsing-custom-function-valid.html css3/filters/filter-property-computed-style.html css3/filters/custom/custom-filter-property-parsing.html css3/filters/filter-property-parsing.html platform/mac/fast/dom/wrapper-classes-objc.html
Build Bot
Comment 11 2013-09-24 02:30:38 PDT
Created attachment 212442 [details] Archive of layout-test-results from webkit-ews-08 for mac-mountainlion The attached test failures were seen while running run-webkit-tests on the mac-ews. Bot: webkit-ews-08 Port: mac-mountainlion Platform: Mac OS X 10.8.5
Gyuyoung Kim
Comment 12 2013-09-24 17:46:06 PDT
It looks this removal has lots of regression on mac and other ports. So, I'd like to take a look this later.
Note You need to log in before you can comment on or make changes to this bug.