Closed Bug 863099 Opened 7 years ago Closed 7 years ago

[SeaMonkey] TEST-UNEXPECTED-FAIL | browser_ApplicationPrefs.js | Check existing boolean property for existence

Categories

(SeaMonkey :: General, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
seamonkey2.20

People

(Reporter: mcsmurf, Assigned: mcsmurf)

Details

Attachments

(1 file)

Bug 857820 removed the browser.blink_allowed pref, we need to fix the browser_ApplicationPrefs.js test because of that:
TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/suite/smile/test/browser_ApplicationPrefs.js | Check existing boolean property for existence
Stack trace:
    JS frame :: chrome://mochitests/content/browser/suite/smile/test/browser_ApplicationPrefs.js :: test :: line 97
    JS frame :: chrome://mochikit/content/browser-test.js :: Tester_execTest :: line 402
    JS frame :: chrome://mochikit/content/browser-test.js :: Tester_nextTest/< :: line 343
    native frame :: <unknown filename> :: <TOP_LEVEL> :: line 0
Attached file Patch
Pushed the patch to comm-central without review as it's a really trivial test fix: https://hg.mozilla.org/comm-central/rev/09c9f84de3e1
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
You need to fix the comment as well.

>   // PREF: boolean browser.blink_allowed == true
> 
>   // test to see if an existing boolean property exists
>   ok(Application.prefs.has(testdata.boolean), "Check existing boolean property for existence");
Flags: needinfo?(bugzilla)
Yeah...will be fixed when I'll land another patch.
Flags: needinfo?(bugzilla)
Target Milestone: --- → seamonkey2.20
Pushed the comment fix to comm-central: https://hg.mozilla.org/comm-central/rev/27503b22e289
You need to log in before you can comment on or make changes to this bug.