EndTransaction() takes 25% of time of PaintFrame() when scrolling a gdoc with 200+ pages.

RESOLVED WORKSFORME

Status

()

Core
Layout: Web Painting
RESOLVED WORKSFORME
9 months ago
7 months ago

People

(Reporter: sinker, Assigned: pchang)

Tracking

(Blocks: 1 bug)

Trunk
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox53 affected)

Details

(Whiteboard: [qf:p1][ps-radar], URL)

(Reporter)

Description

9 months ago
In the profile for bug 1269695, I found |EndTransaction()| takes a lot of time when the user scroll the google document in that bug.  It takes 25% of time of PaintFrame().

I found most of its time waiting for the reply of the sync IPC message, PLayerTransaction::SendUpdate().  If it can go to async IPC, it should be improved a lot.
(Reporter)

Comment 1

9 months ago
https://cleopatra.io/#report=03eb4ae8c845a491154a9993d503bf6ec57d3011&search=ForwardTransaction
(Assignee)

Comment 2

9 months ago
I'm going to dump the layer log to see why we are calling EndTransaction with synchronization SendUpdate call for google document.
(Assignee)

Comment 3

9 months ago
Assign to me first to identify the next step.
Assignee: nobody → howareyou322
(Assignee)

Comment 4

9 months ago
Bug 1328797 is going to remove the synchronous transaction, let's see the difference after bug 1328797 is landed.
Depends on: 1328797

Updated

7 months ago
Whiteboard: [qf:p1]
(Assignee)

Comment 5

7 months ago
Thinker, bug 1328797 was landed. Is this still an issue?
Flags: needinfo?(tlee)
Whiteboard: [qf:p1] → [qf:p1][ps-radar]
(Reporter)

Comment 6

7 months ago
I think we can close this bug, now.
Flags: needinfo?(tlee)
Closing based on comment 6.
Status: NEW → RESOLVED
Last Resolved: 7 months ago
Resolution: --- → WORKSFORME
You need to log in before you can comment on or make changes to this bug.