This is to add the ability to configure JavaScriptCore debug and logging from within the Web process through the presence of a configuration file.
<rdar://problem/30930797>
Created attachment 303852 [details] Patch
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.
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.
Created attachment 303972 [details] Patch for landing - Speculative Windows build fix
(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.
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.
Created attachment 304000 [details] Patch for landing - Speculative gtk build fix
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.
Committed r213690: <http://trac.webkit.org/changeset/213690>