| Summary: | Make WTF Logging runtime switchable, starting with WebCore's Network channel | ||||||
|---|---|---|---|---|---|---|---|
| Product: | WebKit | Reporter: | Gavin Barraclough <barraclough> | ||||
| Component: | Web Template Framework | Assignee: | Gavin Barraclough <barraclough> | ||||
| Status: | NEW --- | ||||||
| Severity: | Normal | CC: | aestes, bburg, commit-queue, ossy | ||||
| Priority: | P2 | ||||||
| Version: | 528+ (Nightly build) | ||||||
| Hardware: | Unspecified | ||||||
| OS: | Unspecified | ||||||
| Attachments: |
|
||||||
|
Description
Gavin Barraclough
2015-04-19 15:52:50 PDT
Created attachment 251137 [details]
Fix
With the attached patch, Network logging can be enabled in a release build using:
defaults write com.apple.WebKit.Networking.Development WebCoreLogging 'Network'
Attachment 251137 [details] did not pass style-queue:
ERROR: Source/WebKit2/Platform/Logging.cpp:35: JOIN_LOG_CHANNEL_WITH_PREFIX is incorrectly named. Don't use underscores in your identifier names. [readability/naming/underscores] [4]
ERROR: Source/WebCore/platform/Logging.cpp:36: JOIN_LOG_CHANNEL_WITH_PREFIX is incorrectly named. Don't use underscores in your identifier names. [readability/naming/underscores] [4]
Total errors found: 2 in 27 files
If any of these errors are false positives, please file a bug against check-webkit-style.
Per conversation with Geoff, Geoff likes log level names "Always", "Customer", "Debug". If you rebase this, I'll review it! Comment on attachment 251137 [details]
Fix
Clearing r?, this patch is rotted.
|