JavaScript error: Graphs.jsm, line 754: InvalidStateError after bug 1122639 when devtools.performance.ui.enable-memory is false

RESOLVED FIXED in Firefox 38

Status

RESOLVED FIXED
4 years ago
6 months ago

People

(Reporter: vporof, Assigned: vporof)

Tracking

unspecified
Firefox 38
Dependency tree / graph

Firefox Tracking Flags

(firefox38 fixed)

Details

Attachments

(1 attachment, 2 obsolete attachments)

Comment hidden (empty)
(Assignee)

Updated

4 years ago
Blocks: 1122639, 1075567
(Assignee)

Updated

4 years ago
Summary: JavaScript error: Graphs.jsm, line 754: InvalidStateError after bug 1122639 → JavaScript error: Graphs.jsm, line 754: InvalidStateError after bug 1122639 when devtools.performance.ui.enable-memory is false
(Assignee)

Updated

4 years ago
Blocks: 1123815
(Assignee)

Comment 1

4 years ago
Created attachment 8560144 [details] [diff] [review]
v1

Apparently doing canvas stuff inside a hidden node is sad. This patch makes everything load lazily when the containers become visible.
Assignee: nobody → vporof
Status: NEW → ASSIGNED
Attachment #8560144 - Flags: review?(jsantell)
(Assignee)

Comment 2

4 years ago
Created attachment 8560145 [details] [diff] [review]
v1

Removed some accidental changes.
Attachment #8560144 - Attachment is obsolete: true
Attachment #8560144 - Flags: review?(jsantell)
(Assignee)

Updated

4 years ago
Attachment #8560145 - Flags: review?(jsantell)
Comment on attachment 8560145 [details] [diff] [review]
v1

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

::: browser/devtools/performance/test/browser_perf-overview-render-01.js
@@ +1,5 @@
>  /* Any copyright is dedicated to the Public Domain.
>     http://creativecommons.org/publicdomain/zero/1.0/ */
>  
> +const FRAMERATE_PREF = "devtools.performance.ui.enable-framerate";
> +const MEMORY_PREF = "devtools.performance.ui.enable-memory";

These are defined in head.js once bug 1128859 lands

::: browser/devtools/performance/test/browser_perf-overview-selection-02.js
@@ +2,5 @@
>     http://creativecommons.org/publicdomain/zero/1.0/ */
>  
> +const FRAMERATE_PREF = "devtools.performance.ui.enable-framerate";
> +const MEMORY_PREF = "devtools.performance.ui.enable-memory";
> +

ditto

::: browser/devtools/performance/test/browser_perf-overview-selection-03.js
@@ +2,5 @@
>     http://creativecommons.org/publicdomain/zero/1.0/ */
>  
> +const FRAMERATE_PREF = "devtools.performance.ui.enable-framerate";
> +const MEMORY_PREF = "devtools.performance.ui.enable-memory";
> +

blap

::: browser/devtools/performance/test/browser_perf-recording-selected-04.js
@@ +3,5 @@
>     http://creativecommons.org/publicdomain/zero/1.0/ */
>  
> +const FRAMERATE_PREF = "devtools.performance.ui.enable-framerate";
> +const MEMORY_PREF = "devtools.performance.ui.enable-memory";
> +

!

::: browser/devtools/performance/test/browser_perf-states.js
@@ +1,5 @@
> +/* Any copyright is dedicated to the Public Domain.
> +   http://creativecommons.org/publicdomain/zero/1.0/ */
> +
> +const FRAMERATE_PREF = "devtools.performance.ui.enable-framerate";
> +const MEMORY_PREF = "devtools.performance.ui.enable-memory";

!
Attachment #8560145 - Flags: review?(jsantell) → review+
(Assignee)

Comment 4

4 years ago
(In reply to Jordan Santell [:jsantell] [@jsantell] from comment #3)
> These are defined in head.js once bug 1128859 lands

Ok, making bug 1128859 block this one. Land away!
Depends on: 1128859
(Assignee)

Comment 5

4 years ago
Created attachment 8560586 [details] [diff] [review]
v2

Rebased and fixed some tests.
Attachment #8560145 - Attachment is obsolete: true
Attachment #8560586 - Flags: review+
https://hg.mozilla.org/mozilla-central/rev/1de14614f44e
Status: ASSIGNED → RESOLVED
Last Resolved: 4 years ago
status-firefox38: --- → fixed
Resolution: --- → FIXED
Whiteboard: [fixed-in-fx-team]
Target Milestone: --- → Firefox 38

Updated

6 months ago
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.