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

Unified Diff: tools/cc-frame-viewer/app/viewer.js

Issue 12260030: [cc-frame-viewer] Panoply of fixes (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: more fixes Created 7 years, 10 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
« no previous file with comments | « no previous file | tools/cc-frame-viewer/src/lthi_view.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tools/cc-frame-viewer/app/viewer.js
diff --git a/tools/cc-frame-viewer/app/viewer.js b/tools/cc-frame-viewer/app/viewer.js
index 68fe61852baf1c72300fde06534fd91255790588..e6e1e87badaf2bf9b4c8754c877f2981a71a7636 100644
--- a/tools/cc-frame-viewer/app/viewer.js
+++ b/tools/cc-frame-viewer/app/viewer.js
@@ -29,7 +29,12 @@ base.exportTo('ccfv', function() {
window.g_lastResult = trace;
var model = new ccfv.Model();
- model.initFromTraceEvents(trace);
+ try {
+ model.initFromTraceEvents(trace);
+ } catch(e) {
+ onLoadError(e);
+ return;
+ }
window.g_model = model;
var modelViewEl = new ccfv.ModelView();
« no previous file with comments | « no previous file | tools/cc-frame-viewer/src/lthi_view.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698