Bug 68323

Summary: [mac] WebKit contains Objective-C classes that are not prefixed with its standard prefixes
Product: WebKit Reporter: mitz
Component: WebKit APIAssignee: mitz
Status: RESOLVED FIXED    
Severity: Normal CC: dglazkov, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Mac   
OS: Unspecified   
Attachments:
Description Flags
WebKit2 part
sam: review+
WebKit1 part
sam: review+
WebCore part
sam: review+, webkit.review.bot: commit-queue-
JavaScriptCore part sam: review+

Description mitz 2011-09-18 11:33:23 PDT
$ nm -jgU JavaScriptCore.framework/JavaScriptCore|egrep '^_OBJC_CLASS_\$_'|egrep -v '\$_(Web|DOM|WK|WTF)'
_OBJC_CLASS_$_ProfilerServer

$ nm -jgU WebCore.framework/WebCore|egrep '^_OBJC_CLASS_\$_'|egrep -v '\$_(Web|DOM|WK|WTF)'
_OBJC_CLASS_$_AccessibilityObjectWrapper
_OBJC_CLASS_$_CascadeList
_OBJC_CLASS_$_CookieStorageObjCAdapter
_OBJC_CLASS_$_LanguageChangeObserver
_OBJC_CLASS_$_ScrollAnimationHelperDelegate
_OBJC_CLASS_$_ScrollbarPainterControllerDelegate
_OBJC_CLASS_$_ScrollbarPainterDelegate
_OBJC_CLASS_$_ScrollbarPartAnimation
_OBJC_CLASS_$_ScrollbarPrefsObserver

$ nm -jgU WebKit.framework/WebKit|egrep '^_OBJC_CLASS_\$_'|egrep -v '\$_(Web|DOM|WK|WTF)'
_OBJC_CLASS_$_NonBlockingPanel
_OBJC_CLASS_$_PDFPrefUpdatingProxy
_OBJC_CLASS_$_ThreadEnabler
_OBJC_CLASS_$__WebSafeForwarder

$ nm -jgU WebKit2.framework/WebKit2|egrep '^_OBJC_CLASS_\$_'|egrep -v '\$_(Web|DOM|WK|WTF)'
_OBJC_CLASS_$_AccessibilityWebPageObject
_OBJC_CLASS_$_FullKeyboardAccessWatcher
Comment 1 mitz 2011-09-18 11:34:27 PDT
If WTF is not to be considered a standard prefix, then there’s also
_OBJC_CLASS_$_WTFMainThreadCaller
in JavaScriptCore.
Comment 2 mitz 2011-09-18 12:45:02 PDT
Created attachment 107790 [details]
WebKit2 part
Comment 3 mitz 2011-09-18 13:02:45 PDT
Created attachment 107792 [details]
WebKit1 part
Comment 4 mitz 2011-09-18 13:43:41 PDT
Created attachment 107795 [details]
WebCore part
Comment 5 WebKit Review Bot 2011-09-18 16:26:55 PDT
Comment on attachment 107795 [details]
WebCore part

Attachment 107795 [details] did not pass chromium-ews (chromium-xvfb):
Output: http://queues.webkit.org/results/9735746

New failing tests:
scrollbars/scrollbar-middleclick-nopaste.html
scrollbars/scrollbar-orientation.html
platform/chromium/fast/text/text-stroke-with-border.html
scrollbars/overflow-scrollbar-combinations.html
svg/as-object/deep-nested-embedded-svg-size-changes-no-layout-triggers-1.html
scrollbars/disabled-scrollbar.html
scrollbars/scrollbar-buttons.html
svg/as-object/nested-embedded-svg-size-changes-no-layout-triggers-1.html
scrollbars/listbox-scrollbar-combinations.html
platform/mac-snowleopard/platform/mac/fast/text/international/Geeza-Pro-vertical-metrics-adjustment.html
svg/as-object/embedded-svg-size-changes-no-layout-triggers.html
scrollbars/scrollbar-click-does-not-blur-content.html
scrollbars/scrollbars-on-positioned-content.html
svg/as-object/deep-nested-embedded-svg-size-changes-no-layout-triggers-2.html
scrollbars/basic-scrollbar.html
Comment 6 mitz 2011-09-18 16:56:58 PDT
Created attachment 107801 [details]
JavaScriptCore part
Comment 7 mitz 2011-09-19 11:35:37 PDT
Fixed in <http://trac.webkit.org/r95448>.