Closed Bug 777890 Opened 9 years ago Closed 9 years ago

Tremendous amount of time spent in GetCurrentScreenConfiguration on cnn.com

Categories

(Firefox OS Graveyard :: General, defect)

x86_64
Linux
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: jrmuizel, Assigned: cjones)

References

Details

(Whiteboard: [sps])

Attachments

(1 file)

We seem to be spending a huge amount of time inside this function on cnn.com.

See the following profile for more info:
http://people.mozilla.com/~bgirard/cleopatra/?report=6ab3936ae5c9546bd567ad4b65b50331b279908e
That is ... what in the f ...
Assignee: nobody → jones.chris.g
Attachment #646301 - Flags: review?(justin.lebar+bug)
Whiteboard: [sps]
(In reply to Chris Jones [:cjones] [:warhammer] from comment #2)
> Created attachment 646301 [details] [diff] [review]
> The cached value is valid after we synchronously Get() it, as far as we know
Comment on attachment 646301 [details] [diff] [review]
The cached value is valid after we synchronously Get() it, as far as we know

Indeed.
Attachment #646301 - Flags: review?(justin.lebar+bug) → review+
https://hg.mozilla.org/mozilla-central/rev/cc7c443a6aa8
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.