Bug 140898
| Summary: | Web Inspector: DebuggerSidebarPanel: _addBreakpoint function has unnecessary parameter | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Jonathan Wells <jonowells> |
| Component: | Web Inspector | Assignee: | Jonathan Wells <jonowells> |
| Status: | RESOLVED WONTFIX | ||
| Severity: | Normal | CC: | graouts, joepeck, jonowells, mattbaker, nvasilyev, timothy, webkit-bug-importer |
| Priority: | P2 | Keywords: | InRadar |
| Version: | 528+ (Nightly build) | ||
| Hardware: | All | ||
| OS: | All | ||
Jonathan Wells
Near line 310 of the DebuggerSidebarPanel.js file is this call:
this._addBreakpoint(breakpoints[i], sourceCode);
However this function begins:
_addBreakpoint: function(breakpoint, sourceCode)
{
var sourceCode = breakpoint.sourceCodeLocation.displaySourceCode;
Clearly this second parameter is unnecessary.
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Radar WebKit Bug Importer
<rdar://problem/19603573>
Radar WebKit Bug Importer
<rdar://problem/19603574>
Matt Baker
The function in question has changed since this was filed, and this is no longer an issue.