Bug 156867

Summary:

Web Inspector: Debugger statement gets a space after it when pretty printed

Product:

WebKit

Reporter:

Timothy Hatcher <timothy>

Component:

Web Inspector

Assignee:

Joseph Pecoraro <joepeck>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

bburg, commit-queue, graouts, joepeck, mattbaker, nvasilyev, timothy, webkit-bug-importer

Priority:

Keywords:

InRadar

Version:

WebKit Local Build

Hardware:

All

OS:

All

Attachments:

Description	Flags
[PATCH] Proposed Fix	none

Timothy Hatcher

Reported 2016-04-21 14:12:34 PDT

Pretty print: navigator.geolocation.getCurrentPosition(function(position){ console.log(position); debugger;}, function(error){ console.log(error); debugger; }); It will format as: navigator.geolocation.getCurrentPosition(function(position) { console.log(position); debugger ; }, function(error) { console.log(error); debugger ; });

Attachments
[PATCH] Proposed Fix (6.67 KB, patch) 2016-04-21 14:25 PDT, Joseph Pecoraro	no flags	Details Formatted Diff Diff
View All Add attachment proposed patch, testcase, etc.

Radar WebKit Bug Importer

Comment 1 2016-04-21 14:12:56 PDT

<rdar://problem/25862308>

Joseph Pecoraro

Comment 2 2016-04-21 14:25:46 PDT

Created attachment 276961 [details] [PATCH] Proposed Fix

Geoffrey Garen

Comment 3 2016-04-21 14:33:36 PDT

Comment on attachment 276961 [details] [PATCH] Proposed Fix r=me

WebKit Commit Bot

Comment 4 2016-04-21 15:23:57 PDT

Comment on attachment 276961 [details] [PATCH] Proposed Fix Clearing flags on attachment: 276961 Committed r199838: <http://trac.webkit.org/changeset/199838>

WebKit Commit Bot

Comment 5 2016-04-21 15:24:02 PDT

All reviewed patches have been landed. Closing bug.

Note You need to log in before you can comment on or make changes to this bug.