Bug 169387 – Add plumbing to WebProcess to enable JavaScriptCore configuration and logging

• Component: WebKit2
• Status: RESOLVED FIXED

[Michael Saboff]

This is to add the ability to configure JavaScriptCore debug and logging from within the Web process through the presence of a configuration file.

[Michael Saboff]

<rdar://problem/30930797>

[Michael Saboff]

Created attachment 303852 [details]
Patch

[WebKit Commit Bot]

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


ERROR: Source/WebCore/ForwardingHeaders/runtime/ConfigFile.h:0:  No copyright message found.  You should have a line: "Copyright [year] <Copyright Owner>"  [legal/copyright] [5]
ERROR: Source/JavaScriptCore/runtime/ConfigFile.cpp:469:  Missing space before {  [whitespace/braces] [5]
Total errors found: 2 in 20 files


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

[Geoffrey Garen]

Comment on attachment 303852 [details]
Patch

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

> Source/JavaScriptCore/runtime/ConfigFile.cpp:39
> +#include <mutex>

This is the Windows build failure.

> Source/WebKit2/UIProcess/Cocoa/WebProcessPoolCocoa.mm:482
> +    path = path + "/Library/WebKit/JavaScriptConfig";

I would call this JavaScriptCoreDebug since it will include debug output in addition to configuration files.

We might need to make sure not to put anything in the Framework or app bundle, otherwise we might invalidate the bundle signature.

[Michael Saboff]

Created attachment 303972 [details]
Patch for landing - Speculative Windows build fix

[Michael Saboff]

(In reply to comment #4)
> Comment on attachment 303852 [details]
> Patch
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=303852&action=review
> 
> > Source/JavaScriptCore/runtime/ConfigFile.cpp:39
> > +#include <mutex>
> 
> This is the Windows build failure.
> 
> > Source/WebKit2/UIProcess/Cocoa/WebProcessPoolCocoa.mm:482
> > +    path = path + "/Library/WebKit/JavaScriptConfig";
> 
> I would call this JavaScriptCoreDebug since it will include debug output in
> addition to configuration files.
> 
> We might need to make sure not to put anything in the Framework or app
> bundle, otherwise we might invalidate the bundle signature.

I made these changes.  I also changed the name of the config file to JSC.config.

[WebKit Commit Bot]

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


ERROR: Source/WebCore/ForwardingHeaders/runtime/ConfigFile.h:0:  No copyright message found.  You should have a line: "Copyright [year] <Copyright Owner>"  [legal/copyright] [5]
ERROR: Source/JavaScriptCore/runtime/ConfigFile.cpp:469:  Missing space before {  [whitespace/braces] [5]
Total errors found: 2 in 20 files


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

[Michael Saboff]

Created attachment 304000 [details]
Patch for landing - Speculative gtk build fix

[WebKit Commit Bot]

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


ERROR: Source/WebCore/ForwardingHeaders/runtime/ConfigFile.h:0:  No copyright message found.  You should have a line: "Copyright [year] <Copyright Owner>"  [legal/copyright] [5]
ERROR: Source/JavaScriptCore/runtime/ConfigFile.cpp:469:  Missing space before {  [whitespace/braces] [5]
Total errors found: 2 in 21 files


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

[Michael Saboff]

Committed r213690: <http://trac.webkit.org/changeset/213690>