Closed
Bug 575597
Opened 14 years ago
Closed 14 years ago
Feedback button only appears when first feedback notification occurs
Categories
(Firefox :: General, defect)
Tracking
()
RESOLVED
FIXED
Firefox 4.0b1
Tracking | Status | |
---|---|---|
blocking2.0 | --- | beta1+ |
People
(Reporter: beltzner, Assigned: jono)
References
Details
Attachments
(1 file, 1 obsolete file)
4.82 KB,
patch
|
mossop
:
review+
|
Details | Diff | Splinter Review |
STR:
1. Download a beta from ftp://ftp.mozilla.org/pub/firefox/nightly/4.0b1-candidates/build1/
2. Start it up
Expected: feedback button to be in upper right
Actual: it's not there until the first time a survey appears
(note: this is different from bug 575666)
cc'ing marco in case this is related to the recent move of the bookmark toolbar button
Reporter | ||
Comment 1•14 years ago
|
||
Marking this as a b1 blocker at this time, though since the first notification comes almost instantly, we might let it go. Need to think about it a bit.
(Also: wasn't that first notification supposed to come after a delay of a few minutes?)
blocking2.0: --- → beta1+
Comment 2•14 years ago
|
||
I filed this earlier than you, Mike :). Since this bug is already marked as a blocker, I'm duping my bug.
Reporter | ||
Comment 4•14 years ago
|
||
(In reply to comment #2)
> I filed this earlier than you, Mike :). Since this bug is already marked as a
> blocker, I'm duping my bug.
Ooops - I looked for dupes and everything. Alas!
I missed this because I was testing with the same profile each time; the bug only happens on a new profile. (I have now verified this).
It's happening because the feedback button is not overlaid into the toolbar in XUL but is rather added to the Customize palette and then dynamically added to the toolbar on first run. Since we pushed TestPilotSetup.globalStartup() to a ten-second delay in order to not interfere with Firefox startup, the code that adds it on first run isn't happening until ten seconds after startup. Need to make sure that happens immediately; hopefully it won't do much to startup time. I will attach a patch.
Will investigate why the notification is showing up so soon, too. It is not supposed to show up until 10 minutes after startup (based on the pref extensions.testpilot.popup.delayAfterStartup).
This patch moves the feedback button installation to the startup path, which ensures that it appears immediately on a new profile. It should be tested for its effects on startup time. I think it also fixed the bug that causes the notification popup to appear too early, but I will investigate further on that one.
> I think it also fixed
Nope. Still happening. Will fix and update the patch.
OK, I have tested this change pretty thoroughly, and it fixes the notification problem (no notifications will appear until 10mins after startup, as designed) and the feedback button problem (appears immediately). The new patch contains both fixes so it obsoletes the previous patch.
Attachment #454912 -
Attachment is obsolete: true
Comment 10•14 years ago
|
||
flagging in-litmus? to to add testcase coverage when creating new firefox profiles. probably should automate this too.
Flags: in-litmus?
Comment 11•14 years ago
|
||
Comment on attachment 454973 [details] [diff] [review]
Fixes both the feedback button not appearing and the notification appearing too early.
r=me, landing now
Attachment #454973 -
Flags: review+
Reporter | ||
Comment 12•14 years ago
|
||
Tony: would need to be beta-specific, there, as we don't ship this in final.
Comment 13•14 years ago
|
||
correct, i understood that, but forgot to note that here. it'll be called out during beta testing cycles going forward.
Comment 14•14 years ago
|
||
Landed on trunk: http://hg.mozilla.org/mozilla-central/rev/fc03205a35e7
Landed on relbranch: http://hg.mozilla.org/mozilla-central/rev/69b71ac32781
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 3.7b1
You need to log in
before you can comment on or make changes to this bug.
Description
•