Update SVGException to use the description in toString() Part of https://bugs.webkit.org/show_bug.cgi?id=159822
Created attachment 283844 [details] Patch
Comment on attachment 283844 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=283844&action=review > LayoutTests/svg/filters/feBlend-invalid-mode-expected.txt:1 > +CONSOLE MESSAGE: line 14: An invalid value was passed to an operation or assigned to an attribute.: An invalid value was passed to an operation or assigned to an attribute. This looks bad. Is this really a progression?
(In reply to comment #2) > Comment on attachment 283844 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=283844&action=review > > > LayoutTests/svg/filters/feBlend-invalid-mode-expected.txt:1 > > +CONSOLE MESSAGE: line 14: An invalid value was passed to an operation or assigned to an attribute.: An invalid value was passed to an operation or assigned to an attribute. > > This looks bad. Is this really a progression? That's really weird. Will double check before landing.
(In reply to comment #3) > (In reply to comment #2) > > Comment on attachment 283844 [details] > > Patch > > > > View in context: > > https://bugs.webkit.org/attachment.cgi?id=283844&action=review > > > > > LayoutTests/svg/filters/feBlend-invalid-mode-expected.txt:1 > > > +CONSOLE MESSAGE: line 14: An invalid value was passed to an operation or assigned to an attribute.: An invalid value was passed to an operation or assigned to an attribute. > > > > This looks bad. Is this really a progression? > > That's really weird. Will double check before landing. The reason for this is because of how the inspector reports exceptions in the console. I'll add a stop-gap utility method for now to provide a reasonable string. I filed https://bugs.webkit.org/show_bug.cgi?id=159855 to track moving console messages over to "toString()" once we're all done with bug 159822
Created attachment 283852 [details] Patch for landing (with console.log change)
Letting EWS run on the patch for landing, then I'll cq+
Whoops, bad changelog here, just a sec.
Created attachment 283853 [details] Patch for landing with console.log change
Comment on attachment 283853 [details] Patch for landing with console.log change Clearing flags on attachment: 283853 Committed r203328: <http://trac.webkit.org/changeset/203328>