Bug 143691 - Replace JavaScriptCoreOutputConsoleMessagesToSystemConsole default with an SPI
Summary: Replace JavaScriptCoreOutputConsoleMessagesToSystemConsole default with an SPI
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: JavaScriptCore (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Joseph Pecoraro
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2015-04-13 19:35 PDT by Joseph Pecoraro
Modified: 2015-04-14 17:48 PDT (History)
4 users (show)

See Also:


Attachments
[PATCH] Proposed Fix (5.71 KB, patch)
2015-04-13 19:39 PDT, Joseph Pecoraro
timothy: review+
timothy: commit-queue-
Details | Formatted Diff | Diff
[PATCH] Proposed Fix (5.82 KB, patch)
2015-04-14 17:37 PDT, Joseph Pecoraro
ggaren: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Joseph Pecoraro 2015-04-13 19:35:53 PDT
* SUMMARY
Replace JavaScriptCoreOutputConsoleMessagesToSystemConsole default with an SPI.

Accessing the default using CFPreferences comes with a non-trivial amount of memory and overhead. It would be nice to get off of the preferences API here and move to something simpler, like an SPI.

There can be different approaches here:

  - Global log to system console setting - matches the existing default
  - Per context log to system console setting - more fine grained control

Seeing as the existing value was global, lets just make a global setting for this as well.
Comment 1 Joseph Pecoraro 2015-04-13 19:39:07 PDT
Created attachment 250689 [details]
[PATCH] Proposed Fix
Comment 2 Joseph Pecoraro 2015-04-13 19:40:27 PDT
<rdar://problem/20470092>
Comment 3 Timothy Hatcher 2015-04-13 20:31:25 PDT
Comment on attachment 250689 [details]
[PATCH] Proposed Fix

Are there any clients that need to adopt? Safari?
Comment 4 Joseph Pecoraro 2015-04-14 17:37:46 PDT
Created attachment 250765 [details]
[PATCH] Proposed Fix

Better version. This moves the setting to the new JSRemoteInspector.h private header and renames it to JSRemoteInspectorSetLogToSystemConsole.
Comment 5 Geoffrey Garen 2015-04-14 17:45:31 PDT
Comment on attachment 250765 [details]
[PATCH] Proposed Fix

r=me
Comment 6 Joseph Pecoraro 2015-04-14 17:48:48 PDT
http://trac.webkit.org/changeset/182826