Bug 42831 - Web Inspector: test that inspector will log syntax error in console without crashing
Summary: Web Inspector: test that inspector will log syntax error in console without c...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Web Inspector (Deprecated) (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Yury Semikhatsky
URL:
Keywords:
Depends on: 42642
Blocks:
  Show dependency treegraph
 
Reported: 2010-07-22 09:16 PDT by Yury Semikhatsky
Modified: 2010-07-22 23:40 PDT (History)
9 users (show)

See Also:


Attachments
Patch (2.94 KB, patch)
2010-07-22 09:18 PDT, Yury Semikhatsky
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Yury Semikhatsky 2010-07-22 09:16:49 PDT
Web Inspector: test that inspector will log syntax error in console without crashing
Comment 1 Yury Semikhatsky 2010-07-22 09:18:26 PDT
Created attachment 62306 [details]
Patch
Comment 2 Joseph Pecoraro 2010-07-22 11:16:14 PDT
Cool. Are Syntax Error messages likely to be the same between JSC and V8
or could there be platform specific results?
Comment 3 Ojan Vafai 2010-07-22 12:23:14 PDT
Comment on attachment 62306 [details]
Patch

> +resources/syntax-error.js:1SyntaxError: Parse error

The lack of space between "1" and "SyntaxError" looks strange. Is that a bug (obviously not from this patch though)?
Comment 4 WebKit Commit Bot 2010-07-22 17:06:05 PDT
Comment on attachment 62306 [details]
Patch

Clearing flags on attachment: 62306

Committed r63922: <http://trac.webkit.org/changeset/63922>
Comment 5 WebKit Commit Bot 2010-07-22 17:06:10 PDT
All reviewed patches have been landed.  Closing bug.
Comment 6 Yury Semikhatsky 2010-07-22 23:40:40 PDT
(In reply to comment #2)
> Cool. Are Syntax Error messages likely to be the same between JSC and V8
> or could there be platform specific results?

In this patch I'm providing VM-specific results. There is a long discussion of the error message differences between JSC and V8 and it was agreed that they should eventually converge. There is a bug for that problem: https://bugs.webkit.org/show_bug.cgi?id=42181

(In reply to comment #3)
> (From update of attachment 62306 [details])
> > +resources/syntax-error.js:1SyntaxError: Parse error
> 
> The lack of space between "1" and "SyntaxError" looks strange. Is that a bug (obviously not from this patch though)?

It's just a textContent of a DOM element representing the message in Web Inspector UI. Although there is no space in the text dump, there is one in the DOM UI and the message there looks nice. I don't thing we need to fix the text output since it's used only in the tests and it already gives a clear view of the logged message text.