Dartium, setFillColor, setStrokeColor does not work on 2d canvas

[DartBot]

This issue was originally filed by Burak...@gmail.com

---

What steps will reproduce the problem?

1. Try to call setFillColor, then fillRect on a 2d canvas.

What is the expected output? What do you see instead?

I would expect to see a filled rect. However, nothing happens.
When I change ctx.setFillColor(color) to ctx.fillStyle = color, things work again.

What version of the product are you using? On what operating system?

Chromium 21.0.1160.0 (Developer Build 139982)
OS Mac OS X
WebKit 537.1 (Source/WebCore/Configurations@568)
JavaScript V8 3.11.8
DartVM r8224
Flash 11.2.202.235
User Agent Mozilla/5.0 (Macintosh; Intel Mac OS X 10_6_8) AppleWebKit/537.1 (KHTML, like Gecko) Chrome/21.0.1160.0 (Dart) Safari/537.1
Command Line /Users/bqe/Downloads/dart/dart-sdk/Chromium.app/Contents/MacOS/Chromium --remote-debugging-port=9322 --user-data-dir=/Users/bqe/.dartChromeSettings --bwsi --no-first-run --no-default-browser-check --no-process-singleton-dialog --flag-switches-begin --flag-switches-end about:blank
Executable Path /Users/bqe/Downloads/dart/dart-sdk/Chromium.app/Contents/MacOS/Chromium
Profile Path /Users/bqe/.dartChromeSettings/Default

Please provide any additional information below.

Things seem to work fine when doing frogc.

[madsager]

Added Area-Dartium, Triaged labels.

[DartBot]

This comment was originally written by antonm@google.com

---

Burak.Emir,

please, provide the problematic snippet. I would be esp. curious what are the values you pass into .setFillColor.

---

Set owner to antonm@google.com.
Added this to the M1 milestone.
Added NeedsInfo label.

[DartBot]

This comment was originally written by Burak....@gmail.com

---

.setFillColor("white")

[DartBot]

This comment was originally written by antonm@google.com

---

Thanks!

That's the known issue then.

Thanks a lot for reporting!

---

Added Duplicate label.
Marked as being merged into #3353.

[DartBot]

This comment was originally written by antonm@google.com

---

Just some more comments.

Currently we generate the following dispatch:

463 if ((c_OR_color_OR_grayLevel_OR_r === null || c_OR_color_OR_grayLevel_OR_r is String) &&
464 b_OR_y === null &&
465 a_OR_k === null &&
466 a === null) {
467 _setFillColor(c_OR_color_OR_grayLevel_OR_r, alpha_OR_g_OR_m);
468 return;
469 }

And in C++ code we cannot tolerate null passed as an alpha value.