Bug 68635 - <canvas> in SVG doesn't scale properly
Summary: <canvas> in SVG doesn't scale properly
Status: RESOLVED DUPLICATE of bug 103117
Alias: None
Product: WebKit
Classification: Unclassified
Component: Canvas (show other bugs)
Version: 528+ (Nightly build)
Hardware: Mac (Intel) OS X 10.8
: P2 Normal
Assignee: Nobody
URL: http://www.peepo.com
Keywords: HasReduction, Regression
Depends on:
Blocks:
 
Reported: 2011-09-22 10:39 PDT by jay
Modified: 2012-11-25 00:50 PST (History)
4 users (show)

See Also:


Attachments
svg with canvas (985 bytes, image/svg+xml)
2011-09-28 05:55 PDT, jay
no flags Details
html using 'svg with canvas' (511 bytes, text/html)
2011-09-28 05:58 PDT, jay
no flags Details
now updates canvas (1.01 KB, image/svg+xml)
2012-11-23 02:52 PST, jay
no flags Details
updates canvas (1.03 KB, image/svg+xml)
2012-11-23 03:09 PST, jay
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description jay 2011-09-22 10:39:49 PDT
regression

open the url in new macbook pro,

the dark orange square is meant to cover the whole yellow area.

check using iPad2 ,10.6 and ppc 10.5.8, firefox etc...

reduced testcase:

peepo.com/index.svgz contains xhtml canvas,

when the svgz file is rescaled in the html5 page index.html,

the canvas is not rescaled.
Comment 1 jay 2011-09-28 05:55:54 PDT
Created attachment 109012 [details]
svg with canvas
Comment 2 jay 2011-09-28 05:58:55 PDT
Created attachment 109013 [details]
html using 'svg with canvas'
Comment 3 jay 2011-09-28 06:04:51 PDT
open html attachment, 
foreignObject scaled, 
SVG red rectangle scaled, but
canvas black border not scaled

wfm latest 10.6 Safari,  and recent Mozilla or Opera

broken Google Chrome 16.0.891.0 dev

and believed broken latest Safari 10.7
Comment 4 jay 2011-09-29 04:23:48 PDT
wfm  chrome 16.0.891.1 dev is this transient, coincidence or fix?

need to confirm for 10.7.1
Comment 5 Jon Gibbins 2011-10-04 06:03:07 PDT
Confirmed broken in Safari 5.1 (7534.48.3) running on OS X Lion (10.7.1).

SVG red rectangle is scaled up, but canvas black border is not.

Jon
Comment 6 Nikolas Zimmermann 2012-01-03 07:56:19 PST
You mean scaling in "svg with canvas" (your first testcase) is broken? I can confirm that.
For "html using 'svg with canvas'" there seems to be a similar bug, but not necesarily the same, it behaves differently, but also broken.
Comment 7 Bob Petrie 2012-03-22 15:36:56 PDT
I was curious to hear whether there has been any movement on this one.  We have a usability issue whose resolution is held up by this.
Comment 8 Nikolas Zimmermann 2012-03-24 03:03:55 PDT
(In reply to comment #7)
> I was curious to hear whether there has been any movement on this one.  We have a usability issue whose resolution is held up by this.

Thanks for the reminder, I checked this again, and it got better. The <canvas> is now resizing properly when resizing the Safari window. Only zooming is left broken. When zooming this file, the size of the black rectangle changes, where it doesn't in other browsers (only stroke-width changes).

Can anyone else confirm that as well please using a Safari/Chrome nightly?
Comment 9 jay 2012-03-25 00:23:32 PDT
please read recent comments bug3781
re slow rendering
Comment 10 Nikolas Zimmermann 2012-03-25 05:10:29 PDT
(In reply to comment #9)
> please read recent comments bug3781
> re slow rendering
This is unrelated to this bug.  I've changed this bug title, to reflect the current state of <canvas> in SVG: zooming problems, no more sizing problems.
Comment 11 jay 2012-03-25 07:22:49 PDT
#9 please do not morph my bugs.

file another bug for zoom
Comment 12 jay 2012-03-25 07:25:18 PDT
#9 please advise: what evidence do you have that scaling is not causing slow rendering?

ie if you compare the two testcase given, pretty much the only difference is the scaling.
in one case the animation runs fine, in the other it does not.
Comment 13 Nikolas Zimmermann 2012-03-25 08:39:36 PDT
(In reply to comment #12)
> #9 please advise: what evidence do you have that scaling is not causing slow rendering?

I'll summarize again for you:
- your original bug report has been fixed in trunk. The rectangle is correctly resized!
- zooming in/out is still broken, as the black rectangle doesn't behave as expected, if you compare to Opera/FF

This has NOTHING to do with "slow rendering". It's just that canvas in SVG + zooming is broken.
Comment 14 Nikolas Zimmermann 2012-03-25 08:48:27 PDT
(In reply to comment #12)
> #9 please advise: what evidence do you have that scaling is not causing slow rendering?
Your original bug report said that something is broken, now you're claiming its rendering slow, please decide.

"please read recent comments bug3781
re slow rendering"
???

(In reply to comment #11)
> #9 please do not morph my bugs.
"morph? The bug you filed is gone, and I'm reusing this to track zooming + SVG <canvas> which is broken.
Comment 15 jay 2012-03-25 09:51:29 PDT
DO NOT MORPH

I politely wrote you off this bug, you ignored me.

I will not file bugs if this continues!
Comment 16 jay 2012-03-25 09:56:53 PDT
#12

it wasn't possible to tell that the scaling might be slow, until there was scaling.
as the slow renfering appears to be related to scaling, I am content to morph this bug to current
scaling causing slow rendering, if that doesn't suite then
either the slow rendering bug could morph
or I could enter a new bug.

however it needs someone else to try out the two tests, one of which has scale the other doesnt,
and advise how you would like to proceed.

else frankly I am content to watch webkit hang,
its your product not mine that has the bug...
Comment 17 jay 2012-03-25 10:00:45 PDT
Please note the URL for this bug is given as peepo.com

that is the test case, the attachments are reduced test cases,

which necessarily cannot cover all possible eventualities!
Comment 18 jay 2012-03-25 10:27:25 PDT
filed as bug82147
Comment 19 Nikolas Zimmermann 2012-03-25 10:54:00 PDT
(In reply to comment #15)
> DO NOT MORPH
> 
> I politely wrote you off this bug, you ignored me.
For the record: I replied Jay in private and told him, that it's not respectful to revert changes to bug reports that a WebKit reviewer changes.

As he thinks I'm not in the position to "morph" "his" bug reports, I am not sure how to go on here.

Jonathan, it would help if you would actually answer questions I posted here:

Example: (In reply to comment #8)
> Can anyone else confirm that as well please using a Safari/Chrome nightly?

Your answer:
please read recent comments bug3781
re slow rendering

Honestly, do you think anyone will understand you posting fragments of sentences, ignoring my question? You've a track record of bug reports like this. It makes me sad.

Example 2:
> Your original bug report said that something is broken, now you're claiming its rendering slow, please decide.

Your answer:
> DO NOT MORPH

--
Sorry this is going nowhere, if you just don't answer questions, I can't help you.
Comment 20 jay 2012-03-31 12:54:56 PDT
see bug 82835 for aspect related to canvas and SVG layers
Comment 21 jay 2012-11-22 07:46:27 PST
#19

Nikolas, apologies not having access to Mountain Lion until very recently it was difficult for me to update this bug.

it appears the issue relates to updating canvas content,
(the attachments for this bug render as expected)

however you will readily discern that that the canvas on the url reverts after resizing, to the original size set. first, click anywhere on the board.

this may be a regression as safari 5 does not exhibit this behaviour,
neither does Firefox...
else it may be one of those oddities around.

I shall attempt to create a working testcase, now I understand the issue a little better

regards
Comment 22 jay 2012-11-22 07:49:40 PST
is iOS6 a webkit platform?
 in any case it is another case for this bug...
Comment 23 jay 2012-11-23 02:52:31 PST
Created attachment 175759 [details]
now updates canvas

now updates canvas
Comment 24 jay 2012-11-23 03:09:09 PST
Created attachment 175765 [details]
updates canvas

updates canvas
Comment 25 jay 2012-11-23 03:40:51 PST
bug#103117 replaces part of this bug
Comment 26 jay 2012-11-25 00:49:40 PST
dupe
Comment 27 jay 2012-11-25 00:50:35 PST
please change status of Bug 103117

*** This bug has been marked as a duplicate of bug 103117 ***