Clean up ProfileEntry

RESOLVED FIXED in Firefox 55

Status

()

RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: njn, Assigned: njn)

Tracking

Trunk
mozilla55
Points:
---

Firefox Tracking Flags

(firefox55 fixed)

Details

Attachments

(3 attachments)

Comment hidden (empty)
(Assignee)

Comment 1

2 years ago
Created attachment 8871180 [details] [diff] [review]
(part 1) - Clean up ProfileEntry

This patch:

- renames flags_ as flagsAndCategory_ because it contains both the flags and
  the category;

- uses << for some bitfield definitions, because it's easier to read;

- removes some dead methods from ProfileEntry;

- removes the unnecessary JS_FRIEND_API from setPC().
Attachment #8871180 - Flags: review?(shu)
(Assignee)

Updated

2 years ago
Assignee: nobody → n.nethercote
Status: NEW → ASSIGNED
(Assignee)

Comment 2

2 years ago
Created attachment 8871181 [details] [diff] [review]
(part 2) - In GeckoProfiler.h, rename Sampler* as Profiler*

Because everything else in this file is called ProfilerFoo or PROFILER_FOO.
Attachment #8871181 - Flags: review?(mstange)
(Assignee)

Comment 3

2 years ago
Created attachment 8871182 [details] [diff] [review]
(part 3) - Remove unnecessary BEGIN_PSEUDO_JS frame from AutoGeckoProfilerEntry

BEGIN_PSEUDO_JS frames are necessary before a sequence of JS frames, in order
to provide a stack address. But in AutoGeckoProfilerEntry the BEGIN_PSEUDO_JS
frame is before a C++ frame, so it's useless.
Attachment #8871182 - Flags: review?(shu)

Updated

2 years ago
Attachment #8871180 - Flags: review?(shu) → review+

Updated

2 years ago
Attachment #8871182 - Flags: review?(shu) → review+
Attachment #8871181 - Flags: review?(mstange) → review+

Comment 6

2 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/7b82e5e570d6
https://hg.mozilla.org/mozilla-central/rev/343d001b8dfe
https://hg.mozilla.org/mozilla-central/rev/735a87a03948
Status: ASSIGNED → RESOLVED
Last Resolved: 2 years ago
status-firefox55: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
You need to log in before you can comment on or make changes to this bug.