Title: [151861] trunk/Source/WebInspectorUI
- Revision
- 151861
- Author
- [email protected]
- Date
- 2013-06-21 15:13:07 -0700 (Fri, 21 Jun 2013)
Log Message
Web Inspector: Uncaught exception when pretty printing WebInspector.Script
https://bugs.webkit.org/show_bug.cgi?id=117893
This was an issue with an earlier refactoring. The sourceCode is a Script,
so replace the non-existent local variable "script" with "this._sourceCode".
Patch by Joseph Pecoraro <[email protected]> on 2013-06-21
Reviewed by Timothy Hatcher.
* UserInterface/SourceCodeTextEditor.js:
(WebInspector.SourceCodeTextEditor.prototype.textEditorUpdatedFormatting):
Modified Paths
Diff
Modified: trunk/Source/WebInspectorUI/ChangeLog (151860 => 151861)
--- trunk/Source/WebInspectorUI/ChangeLog 2013-06-21 22:13:04 UTC (rev 151860)
+++ trunk/Source/WebInspectorUI/ChangeLog 2013-06-21 22:13:07 UTC (rev 151861)
@@ -1,3 +1,16 @@
+2013-06-21 Joseph Pecoraro <[email protected]>
+
+ Web Inspector: Uncaught exception when pretty printing WebInspector.Script
+ https://bugs.webkit.org/show_bug.cgi?id=117893
+
+ This was an issue with an earlier refactoring. The sourceCode is a Script,
+ so replace the non-existent local variable "script" with "this._sourceCode".
+
+ Reviewed by Timothy Hatcher.
+
+ * UserInterface/SourceCodeTextEditor.js:
+ (WebInspector.SourceCodeTextEditor.prototype.textEditorUpdatedFormatting):
+
2013-06-21 Timothy Hatcher <[email protected]>
Stop removing and re-adding breakpoints from the backend on reload.
Modified: trunk/Source/WebInspectorUI/UserInterface/SourceCodeTextEditor.js (151860 => 151861)
--- trunk/Source/WebInspectorUI/UserInterface/SourceCodeTextEditor.js 2013-06-21 22:13:04 UTC (rev 151860)
+++ trunk/Source/WebInspectorUI/UserInterface/SourceCodeTextEditor.js 2013-06-21 22:13:07 UTC (rev 151861)
@@ -779,8 +779,8 @@
for (var i = 0; i < scripts.length; ++i)
scripts[i].formatterSourceMap = this.formatterSourceMap;
} else if (this._sourceCode instanceof WebInspector.Script) {
- if (script.resource)
- script.resource.formatterSourceMap = this.formatterSourceMap;
+ if (this._sourceCode.resource)
+ this._sourceCode.resource.formatterSourceMap = this.formatterSourceMap;
}
// Some breakpoints may have moved, some might not have. Just go through
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes
