Open
Bug 597572
Opened 15 years ago
Updated 3 years ago
allow reftest conditions to contain whitespace
Categories
(Testing :: Reftest, defect)
Testing
Reftest
Tracking
(Not tracked)
NEW
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.
|
||
Comment 1•15 years ago
|
||
Did you have the linebreak in that first version too, or just in bugzilla?
|
|
Reporter | |
Comment 2•15 years ago
|
||
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
|
||
Updated•3 years ago
|
Severity: normal → S3
You need to log in
before you can comment on or make changes to this bug.
Description
•