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.
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].
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.
(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.
Created attachment 46670 [details] Patch 2 for Bug 33710 Renames WebCore/bridge/jni/jsc to WebCore/bridge/jni/js
Created attachment 46682 [details] Patch 3 for Bug 33710 Now with ChangeLog
Comment on attachment 46682 [details] Patch 3 for Bug 33710 r=me, cq-, I assume you want to manually land this one.
jsc is the correct name for this directory See https://bugs.webkit.org/show_bug.cgi?id=33672#c7 Marking as won't fix