Closed Bug 1146380 Opened 9 years ago Closed 9 years ago

Firefox shouldn't be warning that Valence might make it run slowly

Categories

(DevTools :: General, defect)

defect
Not set
normal

Tracking

(firefox39 fixed)

RESOLVED FIXED
Firefox 39
Tracking Status
firefox39 --- fixed

People

(Reporter: past, Assigned: past)

References

Details

Attachments

(1 file)

Bug 1071880 added a warning that is shown every time Firefox spends a lot of time in the compartment of an add-on. Valence should be excluded from this check because it's an official Firefox feature that only happens to be shipped as an add-on.

Furthermore, the code that implements this check treats the devtools loader as an add-on causeing the following error to appear in the browser console:

couldn't look up addon: fx-devtools nsBrowserGlue.js:571:0
This patch makes both the error and the warning go away.
Attachment #8581668 - Flags: review?(blassey.bugs)
Attachment #8581668 - Flags: review?(blassey.bugs) → review+
Backed out for a syntax error that was causing other pref reads to fail:
https://hg.mozilla.org/integration/fx-team/rev/9cee181014eb
Sorry about that! I could swear it was working fine when I tested it locally... *sigh*
https://hg.mozilla.org/mozilla-central/rev/6ea6b9136d58
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 39
I have found the same error about making Firefox slow on :

User Agent 	Mozilla/5.0 (Windows NT 6.3; rv:39.0) Gecko/20100101 Firefox/39.0
Build ID 	20150323030203

Looks like its fixed on today's nightly build :

Build ID 	20150629134017
User Agent 	Mozilla/5.0 (Windows NT 6.3; rv:42.0) Gecko/20100101 Firefox/42.0

Suggesting STATUS -> VERIFIED FIXED

[Bugday-20150701]
Not sure this is fixed. I saw a "would you like to disable Valence" prompt in the current nightly while using the browser toolbox.
(In reply to Gummi from comment #8)
> Not sure this is fixed. I saw a "would you like to disable Valence" prompt
> in the current nightly while using the browser toolbox.

I believe the remaining issue affects all SDK based add-ons, see bug 1191342.
See Also: → 1191342
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.