Use target.getFront to instantiate PerformanceFront

RESOLVED FIXED in Firefox 64

Status

enhancement
P3
normal
RESOLVED FIXED
9 months ago
27 days ago

People

(Reporter: yulia, Assigned: yulia)

Tracking

(Blocks 1 bug)

unspecified
Firefox 64
Dependency tree / graph

Firefox Tracking Flags

(firefox64 fixed)

Details

(Whiteboard: dt-fission)

Attachments

(1 attachment, 1 obsolete attachment)

Assignee

Description

9 months ago
Example refactoring from front instantiation to getFront --> https://phabricator.services.mozilla.com/D4579

* PerformanceFront:
  https://searchfox.org/mozilla-central/search?q=PerformanceFront(&case=false&regexp=false&path=
  Only tests as bug 1222047 already refactored production callsites. It may be useful to migrate initPerformance/destroyPerformance pieces to fronts.
  https://searchfox.org/mozilla-central/rev/3fa761ade83ed0b8ab463acb057c2cf0b104689e/devtools/client/framework/toolbox.js#3087-3116
Assignee

Comment 2

9 months ago
Depends on D7060
Comment on attachment 9012537 [details]
Bug 1494605 - Use target.getFront for PerformanceFront in tests; r=ochameau

Alexandre Poirot [:ochameau] has approved the revision.
Attachment #9012537 - Flags: review+
Assignee: nobody → ystartsev
Component: WebGL Shader Editor → Performance Tools (Profiler/Timeline)
Attachment #9012538 - Attachment is obsolete: true

Comment 4

9 months ago
Pushed by ystartsev@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/cb8b8676ee77
Use target.getFront for PerformanceFront in tests; r=ochameau

Comment 5

9 months ago
Backout by nbeleuzu@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/f394dec25673
Backed out changeset cb8b8676ee77 as per Yulia`s request.

Comment 6

9 months ago
Pushed by ystartsev@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/0a653b330b03
Use target.getFront for PerformanceFront in tests; r=ochameau

Comment 7

9 months ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/0a653b330b03
Status: NEW → RESOLVED
Closed: 9 months ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 64
Whiteboard: dt-fission
You need to log in before you can comment on or make changes to this bug.