|Summary:||console.assert should prefix messages with "Assertion failed: "|
|Product:||WebKit||Reporter:||Adam Roben (:aroben) <aroben>|
|Component:||Web Inspector (Deprecated)||Assignee:||Nobody <webkit-unassigned>|
|Version:||528+ (Nightly build)|
|Bug Depends on:||19134|
Description Adam Roben (:aroben) 2008-05-19 21:56:03 PDT
console.assert should prefix messages with "Assertion failed: ". This would make it clearer why the message was logged.
Comment 1 Keishi Hattori 2009-10-21 20:07:05 PDT
Created attachment 41632 [details] proposed patch Adds an AlertMessageType. "Assertion failed: " is added in the front-end.
Comment 2 Timothy Hatcher 2009-10-21 20:25:26 PDT
Comment on attachment 41632 [details] proposed patch > + messageTextElement.appendChild(document.createTextNode("Assertion failed: ")); This sting needs to be localizable. WebInspector.UIString and added LocalizableStrings.js.
Comment 3 Keishi Hattori 2009-10-21 20:37:37 PDT
Created attachment 41634 [details] proposed patch 2 Fixed :)
Comment 4 Keishi Hattori 2009-10-21 21:52:23 PDT
*** Bug 19136 has been marked as a duplicate of this bug. ***
Comment 5 Eric Seidel (no email) 2009-10-26 12:08:29 PDT
Comment on attachment 41634 [details] proposed patch 2 AFAIK Keishi is not a committer, so adding this to the queue for the bot.
Comment 6 WebKit Commit Bot 2009-10-26 12:31:06 PDT
Comment on attachment 41634 [details] proposed patch 2 Clearing flags on attachment: 41634 Committed r50084: <http://trac.webkit.org/changeset/50084>
Comment 7 WebKit Commit Bot 2009-10-26 12:31:10 PDT
All reviewed patches have been landed. Closing bug.