[Webkit-unassigned] [Bug 130166] Web Inspector: Fix multiple console.assert stripping issues

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed Mar 12 16:50:29 PDT 2014


https://bugs.webkit.org/show_bug.cgi?id=130166





--- Comment #1 from Joseph Pecoraro <joepeck at webkit.org>  2014-03-12 16:50:52 PST ---
With the new warning, and a script to process each JS file of ours, this is the output I see:

    shell> ./Scripts/remove-console-asserts-dryrun.rb 
    DOMTreeManager.js:
    WARNING: console.assert inside control flow statement without braces on line: 556:             console.assert(contentFlow.contentNodes[i].id === flowPayload.content[i]);

    DataGrid.js:
    WARNING: Multi-line console.assert on line 775:         console.assert(!child.isPlaceholderNode, "Shouldn't delete the placeholder node.")

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.



More information about the webkit-unassigned mailing list