Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(211)

Unified Diff: src/liveedit-debugger.js

Issue 7080029: Fix Issue 1320: LiveEdit: text differencer fails with out of memory on large files (Closed) Base URL: https://v8.googlecode.com/svn/branches/bleeding_edge
Patch Set: merge Created 9 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« src/liveedit.cc ('K') | « src/liveedit.cc ('k') | src/runtime.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/liveedit-debugger.js
diff --git a/src/liveedit-debugger.js b/src/liveedit-debugger.js
index e05c53ce1841b35c1f3ca19264b857d154a689a3..291bd52f30a27566f434f39a9c4941db8b223358 100644
--- a/src/liveedit-debugger.js
+++ b/src/liveedit-debugger.js
@@ -988,7 +988,11 @@ Debug.LiveEdit = new function() {
this.SetScriptSource = SetScriptSource;
function CompareStrings(s1, s2) {
- return %LiveEditCompareStrings(s1, s2);
+ try {
+ return %LiveEditCompareStrings(s1, s2);
+ } catch (e) {
+ throw new Failure("Failed to calculate text difference: " + String(e));
+ }
}
// Applies the change to the script.
« src/liveedit.cc ('K') | « src/liveedit.cc ('k') | src/runtime.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698