1276093 - Warn on slow format conversions

Status: RESOLVED FIXED

(Core :: Graphics: CanvasWebGL, defect)

Description

[Kelsey Gilbert [:jgilbert]]

It's not always obvious when an upload call incurs a slow format conversion.

We should warn when this happens.


Eventually, we should warn when we miss a GPU->GPU fast path, but we only support that for video right now.

Comment 1

[Kelsey Gilbert [:jgilbert]]

Attachment: MozReview Request: Bug 1276093 - r?jrmuizel - Warn on slow CPU format conversions.

From 7778749a5a7a25e5e51dd70911a755a28e478db5 Mon Sep 17 00:00:00 2001
---
 dom/canvas/TexUnpackBlob.cpp         | 19 +++++++++++++++----
 dom/canvas/TexUnpackBlob.h           |  2 +-
 dom/canvas/WebGLTexelConversions.cpp |  7 ++++++-
 dom/canvas/WebGLTexelConversions.h   |  3 ++-
 4 files changed, 24 insertions(+), 7 deletions(-)

Review commit: https://reviewboard.mozilla.org/r/55594/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/55594/

Comment 2

[Kelsey Gilbert [:jgilbert]]

Attachment: MozReview Request: r?jrmuizel - Warn when we fail to hit the GPU-copy fast-path.

From cbd47e62b513fe53a4e4b82c46c253578c1e9808 Mon Sep 17 00:00:00 2001
---
 dom/canvas/TexUnpackBlob.cpp | 4 ++++
 1 file changed, 4 insertions(+)

Review commit: https://reviewboard.mozilla.org/r/55596/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/55596/

Comment 3

[Jeff Muizelaar [:jrmuizel]]

Comment on attachment 8757077 [details]
MozReview Request: Bug 1276093 - r?jrmuizel - Warn on slow CPU format conversions.

https://reviewboard.mozilla.org/r/55594/#review52544

This code would be much better if it returned a Maybe<ConvertedBuffer> where ConvertedBuffer was a struct that contained all of the appropriate information. Can you do this in a follow up?

Comment 4

[Jeff Muizelaar [:jrmuizel]]

Comment on attachment 8757078 [details]
MozReview Request:  r?jrmuizel - Warn when we fail to hit the GPU-copy fast-path.

https://reviewboard.mozilla.org/r/55596/#review52546

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/c2fc488c389f
Warn on slow CPU format conversions. - r=jrmuizel
https://hg.mozilla.org/integration/mozilla-inbound/rev/d85f0530eb80
Warn when we fail to hit the GPU-copy fast-path. - r=jrmuizel

Comment 6

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/c2fc488c389f
https://hg.mozilla.org/mozilla-central/rev/d85f0530eb80