Closed Bug 723583 Opened 13 years ago Closed 13 years ago

test pilot thunderbird 'all your user studies' shows each more than once


(Thunderbird :: General, defect)

Not set


(thunderbird11 fixed, thunderbird12 fixed)

Thunderbird 13.0
Tracking Status
thunderbird11 --- fixed
thunderbird12 --- fixed


(Reporter: glind, Assigned: glind)



(2 files)

To reproduce:

1.  studies in ````
2.  tbird with test pilot 1.2:  Preferences > advanced > config editor:  

    extensions.testpilot.indexFileName -> index-tb-dev.json

3.  (clear the cached download - OSX)... 

    $ rm ~/Library/Thunderbird/Profiles/*.default//TestPilotExperimentFiles/index.json

4.  Restart Tbird.
5.  GOOD - doorhanger appears about new study
6.  BAD - ``tools > test pilot > all your user studies > [current]`` 
    (studies show up multiple times)

To investigate...

Starting with ````, going to look for issues in ``setup.js``

after uninstalling and reinstalling the latest version of the extension (1.3.4), this problem goes away.  @rebron, what version comes with TBird?
1.3.4 currently, but 1.3.5 soon.

I believe I've been seeing this in 1.3.4 though.
Uses the ``observer`` correctly, rather than the kludgey "assume only one window" hacks that were there before.  

As per Mark Banner:

09:24 <Standard8> updated patch:
09:25 <Standard8> the magic is in chrome.manifest
09:25 <Standard8> it sets up TestPilotComponent as a component, and registers it in the profile-after-change category
09:26 <Standard8> when that is going to be called out to the observers, the component gets created as a service, and 
                  then the observer function gets called with profile-after-change
09:26 <Standard8> at which point it registers iteself for mail-startup-done, which is the thing you probably really want 
                  for  Thundebird
Attachment #594237 - Flags: review?(squibblyflabbetydoo)
Comment on attachment 594237 [details] [diff] [review]
patch for bug.  Neeeds review.

Review of attachment 594237 [details] [diff] [review]:

I wasn't able to reproduce the specific issue in comment 0, but this works for me. We should remove the dump() statements though.
Attachment #594237 - Flags: review?(squibblyflabbetydoo) → review+
Checked in for Thunderbird:

I also bumped the version that comes with Thunderbird so I can get this up on AMO:

Moving this to the Thunderbird product so that we can track it is fixed there, if we unify the extensions, then we'll probably want a different method.
Closed: 13 years ago
Component: Test Pilot → General
Product: Mozilla Labs → Thunderbird
QA Contact: test-pilot → general
Resolution: --- → FIXED
Attachment #594237 - Flags: approval-comm-beta?
Attachment #594237 - Flags: approval-comm-aurora?
Target Milestone: --- → Thunderbird 13.0
Attachment #594237 - Flags: approval-comm-beta?
Attachment #594237 - Flags: approval-comm-beta+
Attachment #594237 - Flags: approval-comm-aurora?
Attachment #594237 - Flags: approval-comm-aurora+
(In reply to Mark Banner (:standard8) from comment #5)

This still has the debug dump() statements in it. We should remove those.
You need to log in before you can comment on or make changes to this bug.