Bug 33710 – WebCore/bridge/jni/jsc should be renamed WebCore/bridge/jni/js

• Status: RESOLVED
• Resolution: WONTFIX
• Component: WebCore Misc.

[Steve Block]

WebCore/bridge/jni/jsc should be renamed WebCore/bridge/jni/js. When referring to WebKit’s built-in JavaScript engine, the precedent is to use js, rather than jsc for the directory name. See Bug 33672.

[Steve Block]

Created attachment 46664 [details]
Patch 1 for Bug 33710

Renames WebCore/bridge/jni/jsc to WebCore/bridge/jni/js. Also renames jni_utility_private.[cpp|h] to JniUtilityPrivate.[cpp|h].

[Mark Rowe (bdash)]

Comment on attachment 46664 [details]
Patch 1 for Bug 33710

JNI is an acronym and therefore should be expressed in all caps rather than in title case.

Is there some reason that you’re renaming only two of the 13 files that are named in this manner?  It would be more logical to me if you did the directory rename in one step, and then renamed all of the misnamed files at once in a follow-up change.

[Steve Block]

(In reply to comment #2)
> (From update of attachment 46664 [details])
> JNI is an acronym and therefore should be expressed in all caps rather than in
> title case.
> 
> Is there some reason that you’re renaming only two of the 13 files that are
> named in this manner?  It would be more logical to me if you did the directory
> rename in one step, and then renamed all of the misnamed files at once in a
> follow-up change.

OK, I've opened Bug 33712 to rename all of these files and will update the patch to just rename the directory.

[Steve Block]

Created attachment 46670 [details]
Patch 2 for Bug 33710

Renames WebCore/bridge/jni/jsc to WebCore/bridge/jni/js

[Steve Block]

Created attachment 46682 [details]
Patch 3 for Bug 33710

Now with ChangeLog

[Dmitry Titov]

Comment on attachment 46682 [details]
Patch 3 for Bug 33710

r=me, cq-, I assume you want to manually land this one.

[Steve Block]

jsc is the correct name for this directory
See https://bugs.webkit.org/show_bug.cgi?id=33672#c7

Marking as won't fix