Bug 138988

Summary: Web Inspector: DOMExceptions do not show the error message string in Pause Reason section
Product: WebKit Reporter: Joseph Pecoraro <joepeck>
Component: Web InspectorAssignee: Joseph Pecoraro <joepeck>
Status: RESOLVED FIXED    
Severity: Normal CC: commit-queue, graouts, joepeck, timothy, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Attachments:
Description Flags
[PATCH] Proposed Fix timothy: review+, commit-queue: commit-queue-

Description Joseph Pecoraro 2014-11-21 15:29:02 PST
* SUMMARY
DOMExceptions do not show the error message string in Pause Reason section.

* STEPS TO REPRODUCE
1. Inspect nytimes.com
2. Enable pause on all exceptions
3. Reload
  => When pausing due to a DOMException the message is just "DOMException"
  => Expected something better, like: "Error: SyntaxError: DOM Exception 12"
Comment 1 Radar WebKit Bug Importer 2014-11-21 15:29:34 PST
<rdar://problem/19064068>
Comment 2 Joseph Pecoraro 2014-11-21 15:31:57 PST
Created attachment 242084 [details]
[PATCH] Proposed Fix
Comment 3 WebKit Commit Bot 2014-11-26 08:56:48 PST
Comment on attachment 242084 [details]
[PATCH] Proposed Fix

Rejecting attachment 242084 [details] from commit-queue.

Failed to run "['/Volumes/Data/EWS/WebKit/Tools/Scripts/webkit-patch', '--status-host=webkit-queues.appspot.com', '--bot-id=webkit-cq-01', 'land-attachment', '--force-clean', '--non-interactive', '--parent-command=commit-queue', 242084, '--port=mac']" exit_code: 2 cwd: /Volumes/Data/EWS/WebKit

Last 500 characters of output:
t.org/repository/webkit/trunk ...
Authentication realm: <http://svn.webkit.org:80> Mac OS Forge
Password for 'commit-queue@webkit.org': 
Authentication realm: <http://svn.webkit.org:80> Mac OS Forge
Username: Use of uninitialized value $username in chomp at /usr/share/git-core/perl/Git/SVN/Prompt.pm line 114.
error: git-svn died of signal 11

Failed to run "['git', 'svn', 'dcommit', '--rmdir']" exit_code: 139 cwd: /Volumes/Data/EWS/WebKit
Updating OpenSource
Current branch master is up to date.

Full output: http://webkit-queues.appspot.com/results/6545376169426944
Comment 4 Joseph Pecoraro 2014-12-01 11:11:09 PST
http://trac.webkit.org/changeset/176595