Bug 914985. Adjust browser_tabopen_reflows test to deal with a shorter stack to the reflow and log more information when it fails. r=ttaubert

This commit is contained in:
Boris Zbarsky 2013-09-11 23:19:07 -04:00
parent 9714f6a991
commit 183898489c

View file

@ -38,8 +38,7 @@ const EXPECTED_REFLOWS = [
"TabItem__reconnect@chrome://browser/content/tabview.js|" + "TabItem__reconnect@chrome://browser/content/tabview.js|" +
"TabItem@chrome://browser/content/tabview.js|" + "TabItem@chrome://browser/content/tabview.js|" +
"TabItems_link@chrome://browser/content/tabview.js|" + "TabItems_link@chrome://browser/content/tabview.js|" +
"@chrome://browser/content/tabview.js|" + "@chrome://browser/content/tabview.js|",
"addTab@chrome://browser/content/tabbrowser.xml|",
// SessionStore.getWindowDimensions() // SessionStore.getWindowDimensions()
"ssi_getWindowDimension@resource:///modules/sessionstore/SessionStore.jsm|" + "ssi_getWindowDimension@resource:///modules/sessionstore/SessionStore.jsm|" +
@ -88,6 +87,7 @@ let observer = {
let path = (new Error().stack).split("\n").slice(1).map(line => { let path = (new Error().stack).split("\n").slice(1).map(line => {
return line.replace(/:\d+$/, ""); return line.replace(/:\d+$/, "");
}).join("|"); }).join("|");
let pathWithLineNumbers = (new Error().stack).split("\n").slice(1).join("|");
// Stack trace is empty. Reflow was triggered by native code. // Stack trace is empty. Reflow was triggered by native code.
if (path === "") { if (path === "") {
@ -102,7 +102,7 @@ let observer = {
} }
} }
ok(false, "unexpected uninterruptible reflow '" + path + "'"); ok(false, "unexpected uninterruptible reflow '" + pathWithLineNumbers + "'");
}, },
reflowInterruptible: function (start, end) { reflowInterruptible: function (start, end) {