CustomizableUI.createWidget should invalidate widget wrappers

RESOLVED FIXED in Firefox 28

Status

()

RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: Gijs, Assigned: Gijs)

Tracking

(Blocks: 1 bug)

unspecified
Firefox 28
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [Australis:P1])

Attachments

(1 attachment)

(Assignee)

Description

5 years ago
This is blocking correct fixes for bug 940820.
(Assignee)

Comment 1

5 years ago
Created attachment 8335391 [details] [diff] [review]
invalidate widget wrappers on createWidget calls,

Added a test, too.
Attachment #8335391 - Flags: review?(mconley)
Comment on attachment 8335391 [details] [diff] [review]
invalidate widget wrappers on createWidget calls,

Review of attachment 8335391 [details] [diff] [review]:
-----------------------------------------------------------------

This looks fine - I just think we need to include this bug number in the test filename.

::: browser/components/customizableui/test/browser.ini
@@ +35,5 @@
>  # Because this test is about the menubar, it can't be run on mac
>  skip-if = os == "mac"
>  
>  [browser_panel_toggle.js]
> +[browser_invalidate_wrapper_cache_createWidget.js]

This test filename should include the bug number, I think.
Attachment #8335391 - Flags: review?(mconley) → review+
(Assignee)

Comment 3

5 years ago
remote:   https://hg.mozilla.org/integration/fx-team/rev/97ae1ad323b7

with a bug number in the filename.
Status: NEW → ASSIGNED
Whiteboard: [Australis:P1] → [Australis:P1][fixed-in-fx-team]
https://hg.mozilla.org/mozilla-central/rev/97ae1ad323b7
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Whiteboard: [Australis:P1][fixed-in-fx-team] → [Australis:P1]
Target Milestone: --- → Firefox 28
You need to log in before you can comment on or make changes to this bug.