The position of opaque black rect is incorrect when alpha is false.

The position of opaque black rect is incorrect when alpha is false.

It should be drawn using the layout rect instead of the canvas size rect.

BUG=382884
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=176024

[zino, 2014-06-10 12:00:38]

Please take a look.

It was be already covered by fast/canvas/alpha.html.
But the expected image is incorrect.
So, we need just rebaseline.

Thank you :)

[fs, 2014-06-10 12:29:59]

Fix LGTM. But maybe you could also add a test where the difference between the
intrinsic size and the computed size is more pronounced? (I.e. something like
http://jsfiddle.net/PDfsP/ - which should be possible to turn into a ref-test
quite easily.)

[zino, 2014-06-10 16:19:45]

On 2014/06/10 12:29:59, fs wrote:
> Fix LGTM. But maybe you could also add a test where the difference between the
> intrinsic size and the computed size is more pronounced? (I.e. something like
> http://jsfiddle.net/PDfsP/ - which should be possible to turn into a ref-test
> quite easily.)

I added the test.

Thank you.

[zino, 2014-06-12 07:52:33]

On 2014/06/10 16:19:45, zino wrote:
> On 2014/06/10 12:29:59, fs wrote:
> > Fix LGTM. But maybe you could also add a test where the difference between
the
> > intrinsic size and the computed size is more pronounced? (I.e. something
like
> > http://jsfiddle.net/PDfsP/ - which should be possible to turn into a
ref-test
> > quite easily.)
> 
> I added the test.
> 
> Thank you.

Would you mind if I commit this?

[fs, 2014-06-12 08:26:03]

On 2014/06/12 07:52:33, zino wrote:
> On 2014/06/10 16:19:45, zino wrote:
> > On 2014/06/10 12:29:59, fs wrote:
> > > Fix LGTM. But maybe you could also add a test where the difference between
> the
> > > intrinsic size and the computed size is more pronounced? (I.e. something
> like
> > > http://jsfiddle.net/PDfsP/ - which should be possible to turn into a
> ref-test
> > > quite easily.)
> > 
> > I added the test.
> > 
> > Thank you.
> 
> Would you mind if I commit this?

Go ahead.

[zino, 2014-06-12 08:30:13]

The CQ bit was checked by jinho.bang@samsung.com

[zino, 2014-06-12 08:30:15]

The CQ bit was unchecked by jinho.bang@samsung.com

[zino, 2014-06-12 08:58:49]

The CQ bit was checked by jinho.bang@samsung.com

[commit-bot: I haz the power, 2014-06-12 08:59:52]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/jinho.bang@samsung.com/327703005/60001

[commit-bot: I haz the power, 2014-06-12 10:11:38]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  linux_blink_dbg on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/linux_blink_dbg/builds/1...)
  linux_blink_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/linux_blink_rel/builds/1...)
  mac_blink_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/mac_blink_rel/builds/11191)
  win_blink_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/win_blink_rel/builds/11719)

[commit-bot: I haz the power, 2014-06-12 10:38:27]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-06-12 10:38:28]

Try jobs failed on following builders:
  linux_blink_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/linux_blink_rel/builds/1...)

[zino, 2014-06-12 11:04:41]

The CQ bit was checked by jinho.bang@samsung.com

[commit-bot: I haz the power, 2014-06-12 11:07:38]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/jinho.bang@samsung.com/327703005/80001

[commit-bot: I haz the power, 2014-06-12 12:15:41]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  linux_blink_dbg on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/linux_blink_dbg/builds/1...)
  win_blink_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/win_blink_rel/builds/11738)

[commit-bot: I haz the power, 2014-06-12 13:23:38]

Change committed as 176024