[Webkit-unassigned] [Bug 38846] Merge some pseudo-valid-*.html tests into one
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Mon May 10 22:49:01 PDT 2010
https://bugs.webkit.org/show_bug.cgi?id=38846
Shinichiro Hamaji <hamaji at chromium.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #55556|review? |review+
Flag| |
--- Comment #4 from Shinichiro Hamaji <hamaji at chromium.org> 2010-05-10 22:49:01 PST ---
(From update of attachment 55556)
> +var notValidColor = "rgb(0, 255, 0)";
> +for (var i = 0; i < names.length; i++)
> + shouldBe("getBackgroundColor('" + names[i] + "')", "notValidColor");
I slightly prefer rgb(0, 255, 0) to notValidColor because "foobar is not valid color" sounds like this test is failing.
shouldBe("getBackgroundColor('" + names[i] + "')", "rgb(0, 255, 0)");
Also, I think the color should be not valid nor invalid, right?
--
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
More information about the webkit-unassigned
mailing list