Closed Bug 604419 Opened 9 years ago Closed 9 years ago

Selecting "Don't show this message..." should prevent pop-up blocked info bar from being displayed


(Firefox :: General, defect)

Not set



Firefox 4.0b8
Tracking Status
blocking2.0 --- final+


(Reporter: streetwolf, Assigned: fryn)


(Depends on 1 open bug, )


(Keywords: regression, ux-control)


(1 file)

User-Agent:       Mozilla/5.0 (Windows NT 6.1; WOW64; rv:2.0b8pre) Gecko/20101013 Firefox/4.0b8pre
Build Identifier: 20101013184549

Pop-up block info bar always displays even though it's checked not to. is a good site to test this. 

Reproducible: Always

Today's nightly build introduced this.  Lot's going on with this nightly.
Version: unspecified → Trunk
I can confirm this.
I've been seeing this problem for quite some time in the nightlies.
But, A status bar was removed, and the icon which showed that popup was blocked was also removed, So, I think that some kind of notices are necessary.
this behavior is intentionally by Bug 574688.
And this depend on Bug 594294.
Depends on: 594294
This problem only started with today's nightly.  The Add-on bar has been in place for at least a week or more. I for one don't want to see the info bar everytime there is a blocked popup.
Seems I found a workaround for the info bar.  When the info bar pops up click on the options and allow the site to use popups.  Refresh the page and see the popup.  Close out the popup and while still on the main page right click and choose 'View Page Info'.  Go to Permissions and change he option 'Open pop-up windows' to 'block' or 'default'.
Blocks: 610349
Duplicate of this bug: 610349
Blocking on this regression.
blocking2.0: --- → final+
No longer blocks: 610349
Ever confirmed: true
The bug on which this depends got marked not blocking, but we still want this regression fixed (it's a blocker), so I'm going to go ahead and fix it anyway for now and deal with its dependency later.
Assignee: nobody → fryn
OS: Windows 7 → All
Hardware: x86_64 → All
Depends on: 588317
Summary: Pop-up block info bar always displays. → Selecting "Don't show this message..." should prevent pop-up blocked info bar from being displayed
Attached patch patchSplinter Review
(In reply to bug 594294 comment #4)
Gavin Sharp wrote:
> I think we should remove the dead code in the interim.

This removes it.

This patch fixes the "Don't show this message..." checkbox, which currently does nothing due to its code being commented out. That functionality was already broken for users that had the checkbox checked before we broke it, since we removed the status bar icon, which was the other indicator.

The lack of a less noisy indicator will be fixed asap in bug 588317, which will move both the bar and the icon that used to exist to the location bar. We will also remove the ability to hide the bar, since the bar will no longer exist.
Attachment #492521 - Flags: review?(
Attachment #492521 - Flags: ui-review?(faaborg)
Comment on attachment 492521 [details] [diff] [review]

fine for addressing this blocking bug, eventual UI is still planned to be bug 588317
Attachment #492521 - Flags: ui-review?(faaborg) → ui-review+
Comment on attachment 492521 [details] [diff] [review]

As I understand it, the only thing that's broken is toggling the pref from the menu - if you set it to false manually (or had it set to false before we broke this), it should still be preventing the notification from showing, right? Comment 0 was kind of misleading here.

We could probably go further and remove the blockedPopupDontShowMessage broadcaster (and just set the attributes on the menuitem directly), though I don't know offhand how that meshes with bug 588317 - if it isn't going to be used or removed in that bug, make sure a followup is filed?

This makes bug 311866 INVALID by removing the dialog; nice to finally get rid of it.
Attachment #492521 - Flags: review?( → review+
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 4.0b8
You need to log in before you can comment on or make changes to this bug.