1162395 - [LayerScope]: No need to read and send the texture when the same texture id contains in a frame.

Status: RESOLVED FIXED

(Core :: Graphics, defect)

Description

[Vincent Liu[:vliu]]

In Bug 1150381, SendTextureSource doesn't send the same texture twice in a frame. But even so, it still read this texture. Since SendTextureSource knows all texture id before reading it, SendTextureSource can decide whether it need to read it or not based on checking id.

Comment 1

[Vincent Liu[:vliu]]

Attachment: bug-1162395-fix.patch

Hi cku,

Can you please have a review for me? Thanks

Comment 2

[u459114]

Comment on attachment 8602546 [details] [diff] [review]
bug-1162395-fix.patch

Review of attachment 8602546 [details] [diff] [review]:
-----------------------------------------------------------------

LGTM

Comment 3

[Vincent Liu[:vliu]]

(In reply to C.J. Ku[:cjku] from comment #2)
> Comment on attachment 8602546 [details] [diff] [review]
> bug-1162395-fix.patch
> 
> Review of attachment 8602546 [details] [diff] [review]:
> -----------------------------------------------------------------
> 
> LGTM

Thanks

Comment 4

[Vincent Liu[:vliu]]

Try server result:

https://treeherder.mozilla.org/#/jobs?repo=try&revision=658d8c7fc345

Comment 5

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/a9a7bb0afb7b

Comment 6

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/a9a7bb0afb7b