Closed Bug 1492397 Opened 3 years ago Closed 2 years ago

Make WebRenderBridgeParent creation to async in content process

Categories

(Core :: Graphics: WebRender, enhancement, P2)

enhancement

Tracking

()

RESOLVED INVALID

People

(Reporter: sotaro, Assigned: sotaro)

References

(Blocks 1 open bug)

Details

Attachments

(2 files, 5 obsolete files)

SendPWebRenderBridgeConstructor() is sync ipc, it is better if the ipc is async.
(In reply to Sotaro Ikeda [:sotaro] from comment #0)
> SendPWebRenderBridgeConstructor() is sync ipc, it is better if the ipc is
> async.

It is possible when SendPWebRenderBridgeConstructor() is in content process.
Summary: Make SendPWebRenderBridgeConstructor() to async → Make SendPWebRenderBridgeConstructor() to async in content process
Assignee: nobody → sotaro.ikeda.g
Priority: -- → P2
Priority: P2 → P3
Depends on: 1492398
Depends on: 1492745
Depends on: 1493327
No longer depends on: 1493327
Depends on: 1493327
Priority: P3 → P4
With Bug 1493327 patch, latency became better. With it, I did not see more than 4ms latency.
Priority: P4 → P2
Summary: Make SendPWebRenderBridgeConstructor() to async in content process → Make WebRenderBridgeParent creation to async in content process
Depends on: 1496003
Attached patch wipSplinter Review
Rebased.
Attachment #9013930 - Attachment is obsolete: true
Depends on: 1413558
Depends on: 1496825
Sotaro does this need to block the MVP?
Flags: needinfo?(sotaro.ikeda.g)
I do not thinks this needs to block the MVP. In the past, WebRenderBridgeParent creation took longer time(more than 10ms). But since Bug 1493327, the creation time is normally less than 1ms.
Flags: needinfo?(sotaro.ikeda.g)
I rechecked the latency with attachment 9013195 [details] [diff] [review]. The majority of cases, the latency was less than 0.1 ms. And the longest latency was 1.5ms. Then, the latency is not a problem now.
The latency problem was addressed by Bug 1493327.

Re-open it, when the latency becomes problem.
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → INVALID
You need to log in before you can comment on or make changes to this bug.