Open Bug 597572 Opened 9 years ago Updated 9 years ago

allow reftest conditions to contain whitespace

Categories

(Testing :: Reftest, defect)

defect
Not set

Tracking

(Not tracked)

People

(Reporter: sgreenlay, Unassigned)

Details

Inserting the following into /modules/plugin/test/reftest/reftest.list and running the reftest:

skip-if(!prefs.getBoolPref("dom.ipc.plugins.enabled") && !prefs.getBoolPref("dom.ipc.plugins.enabled.i386"))

leads to:

REFTEST TEST-UNEXPECTED-FAIL | | EXCEPTION: SyntaxError: missing ) in parenthetical

however, by removing the white space around &&:

skip-if(!prefs.getBoolPref("dom.ipc.plugins.enabled")&&!prefs.getBoolPref("dom.ipc.plugins.enabled.i386"))

this error does not occur.
Did you have the linebreak in that first version too, or just in bugzilla?
Sorry, that is only in Bugzilla.
layout/tools/reftest/README.txt documents that the conditions can't have whitespace in them.  I've been meaning to change that sometime, but I haven't.
Component: Layout → Reftest
Product: Core → Testing
QA Contact: layout → reftest
Hardware: x86 → All
Summary: "EXCEPTION: SyntaxError: missing ) in parenthetical" with whitespace around && in reftest.list → allow reftest conditions to contain whitespace
You need to log in before you can comment on or make changes to this bug.