Summary: | Web Inspector: Remove console.asserts from Worker code in optimized builds | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Joseph Pecoraro <joepeck> | ||||||
Component: | Web Inspector | Assignee: | Joseph Pecoraro <joepeck> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | bburg, commit-queue, graouts, joepeck, mattbaker, nvasilyev, timothy, webkit-bug-importer | ||||||
Priority: | P2 | Keywords: | InRadar | ||||||
Version: | WebKit Nightly Build | ||||||||
Hardware: | All | ||||||||
OS: | All | ||||||||
Attachments: |
|
Description
Joseph Pecoraro
2016-04-01 19:08:17 PDT
Created attachment 275455 [details]
[PATCH] Proposed Fix
Created attachment 275456 [details]
[TEXT] remove-console-asserts.pl
Attached the file on its own because the diff looks horrible but the modifications are very small:
- added a new --input-directory switch
- moved the existing code into a function
- use the functions for the different switches
Comment on attachment 275455 [details] [PATCH] Proposed Fix Clearing flags on attachment: 275455 Committed r198968: <http://trac.webkit.org/changeset/198968> All reviewed patches have been landed. Closing bug. <http://trac.webkit.org/changeset/198969> Follow up to remove the debug print statement. |