From 090fb7c81d4fe4916babb279dd57c140c25f010a Mon Sep 17 00:00:00 2001 From: Kyle Huey Date: Tue, 7 Jun 2011 10:03:02 -0700 Subject: [PATCH] Bug 662403: Fix reftest-analyzer and leak-gauge to not use removed nsIDOMFile methods. r=bz DONTBUILD --- layout/tools/reftest/reftest-analyzer.xhtml | 19 ++++++++++++------- tools/footprint/leak-gauge.html | 16 ++++++++++++---- 2 files changed, 24 insertions(+), 11 deletions(-) diff --git a/layout/tools/reftest/reftest-analyzer.xhtml b/layout/tools/reftest/reftest-analyzer.xhtml index 4e7c621c939..c59f5b89921 100644 --- a/layout/tools/reftest/reftest-analyzer.xhtml +++ b/layout/tools/reftest/reftest-analyzer.xhtml @@ -191,23 +191,28 @@ function fileentry_changed() { show_phase("loading"); var input = ID("fileentry"); var files = input.files; - var log = null; if (files.length > 0) { // Only handle the first file; don't handle multiple selection. // The parts of the log we care about are ASCII-only. Since we // can ignore lines we don't care about, best to read in as // iso-8859-1, which guarantees we don't get decoding errors. - log = files[0].getAsText("iso-8859-1"); + var fileReader = new FileReader(); + fileReader.onload = function(e) { + var log = null; + + log = e.target.result; + + if (log) + process_log(log); + else + show_phase("entry"); + } + fileReader.readAsText(files[0], "iso-8859-1"); } // So the user can process the same filename again (after // overwriting the log), clear the value on the form input so we // will always get an onchange event. input.value = ""; - - if (log) - process_log(log); - else - show_phase("entry"); } function log_pasted() { diff --git a/tools/footprint/leak-gauge.html b/tools/footprint/leak-gauge.html index 34f6afe2105..1a7296daf02 100644 --- a/tools/footprint/leak-gauge.html +++ b/tools/footprint/leak-gauge.html @@ -48,7 +48,18 @@ pre.output { border: medium solid; padding: 1em; margin: 1em; }