Status

()

enhancement
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)

Assignee

Description

2 years ago
No description provided.
Assignee

Comment 1

2 years ago
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
Because everything else in this file is called ProfilerFoo or PROFILER_FOO.
Attachment #8871181 - Flags: review?(mstange)
Assignee

Comment 3

2 years ago
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
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
You need to log in before you can comment on or make changes to this bug.