Bug 90619 – Code refactoring: move mimeTypeFromUTITree to a separate file.

• Status: RESOLVED
• Resolution: FIXED
• Component: Platform

[Yongjun Zhang]

We should UTI handling code to a separate file.

[Yongjun Zhang]

Created attachment 150954 [details]
Patch

[WebKit Review Bot]

Attachment 150954 [details] did not pass style-queue:

Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'Source/WebCore/ChangeLog', u'Source/WebCor..." exit_code: 1
Source/WebCore/platform/network/mac/UTIUtilities.h:35:  Code inside a namespace should not be indented.  [whitespace/indent] [4]
Total errors found: 1 in 5 files


If any of these errors are false positives, please file a bug against check-webkit-style.

[Yongjun Zhang]

Created attachment 150958 [details]
Patch, fix style violation in UTIUtilities.h.

[David Kilzer (:ddkilzer)]

Comment on attachment 150958 [details]
Patch, fix style violation in UTIUtilities.h.

View in context: https://bugs.webkit.org/attachment.cgi?id=150958&action=review

r=me with the clause 3 license fixes.  (The WebCore Xcode project items are just nits I noticed, but can be checked in with this commit, or separately.)

> Source/WebCore/WebCore.xcodeproj/project.pbxproj:7958
> -		333F704F0FB49CA2008E12A6 /* Notification.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; name = Notification.h; path =Modules/notifications/Notification.h; sourceTree = "<group>"; };
> +		333F704F0FB49CA2008E12A6 /* Notification.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; name = Notification.h; path = Modules/notifications/Notification.h; sourceTree = "<group>"; };

NOTE: Drive-by fix that happened when editing the Xcode project file.

> Source/WebCore/WebCore.xcodeproj/project.pbxproj:28176
> -				A8CCBB4C151F833B00AB7CE9 /* WorkerContextWebDatabase.cpp in Sources */, F34742E01343631F00531BC2 /* WorkerDebuggerAgent.cpp in Sources */,
> +				A8CCBB4C151F833B00AB7CE9 /* WorkerContextWebDatabase.cpp in Sources */,
> +				F34742E01343631F00531BC2 /* WorkerDebuggerAgent.cpp in Sources */,

NOTE: Drive-by fix that happened when editing the Xcode project file.

> Source/WebCore/platform/network/mac/UTIUtilities.h:15
> + * 3.  Neither the name of Apple Computer, Inc. ("Apple") nor the names of
> + *     its contributors may be used to endorse or promote products derived
> + *     from this software without specific prior written permission. 

This is the old Apple license.  The one in Source/WebCore/LICENSE-APPLE doesn't include clause 3 anymore.  Please remove clause 3.

> Source/WebCore/platform/network/mac/UTIUtilities.mm:15
> + * 3.  Neither the name of Apple Computer, Inc. ("Apple") nor the names of
> + *     its contributors may be used to endorse or promote products derived
> + *     from this software without specific prior written permission. 

Ditto re: removing clause 3.

[Yongjun Zhang]

Created attachment 150973 [details]
Address review comments on previous patch.

[WebKit Review Bot]

Comment on attachment 150973 [details]
Address review comments on previous patch.

Clearing flags on attachment: 150973

Committed r121923: <http://trac.webkit.org/changeset/121923>

[WebKit Review Bot]

All reviewed patches have been landed.  Closing bug.