This one just popped up on Windows, and a few times on Linux too.
SUMMARY-UNEXPECTED-FAIL | e:\buildbot\comm-central-win32-opt-unittest-mozmill\build\mozmill\content-tabs\test-lwthemes.js | test-lwthemes.js::test_lightweight_themes
EXCEPTION: No lightweight theme installed after selecting undo
at: test-lwthemes.js line 127
install_theme(2,1) test-lwthemes.js 127
test_lightweight_themes() test-lwthemes.js 171
Straight porting bug 592338 didn't work. When I try manually, the test seems to pass, but running in automatic mode it fails.
Going to run a bisect to see if I can find out what caused this.
Sorry - I forgot to update this bug - I've bisected this one, and found it was an issue for both Firefox and Thunderbird.
(See bug 727702)
Unfocused has bug 727702 on his "TODO" list.
Alright, I think I may have found a solution to this one - but I have to clear it with Mossop or Dao before I can be certain it's the right approach (posting my patch in bug 727702 shortly).
In the meantime, we might want to shut off this test temporarily, in order to make the oranges in our tree more meaningful.
Is that OK with you, Mark?
Yep, and I've already disabled the test:
(I split it into two test functions, so that we could still test the apply -> undo -> apply path without the returning to a previous persona part).
Regarding the solution I had for this - it looks like it makes our tests pass, but seems to gloss over the fact that Personas are still kinda broken.
There are still instances when installed Personas will not apply from the Add-ons Manager, for example.
I'm pretty sure the patch for bug 714841 broke Personas horribly. :/
This was fixed by bug 714841 backing out bug 727702, I don't think we need to do anything about it at the moment.