[Chromium] Fix warnings on Windows build
Created attachment 158365 [details] Patch
Comment on attachment 158365 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=158365&action=review > Source/WebKit/chromium/tests/CCRenderSurfaceFiltersTest.cpp:56 > EXPECT_TRUE(isCombined(WebFilterOperation::createGrayscaleFilter(0.5))); Don't you also need to change 0.5 to 0.5f?
(In reply to comment #2) > (From update of attachment 158365 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=158365&action=review > > > Source/WebKit/chromium/tests/CCRenderSurfaceFiltersTest.cpp:56 > > EXPECT_TRUE(isCombined(WebFilterOperation::createGrayscaleFilter(0.5))); > > Don't you also need to change 0.5 to 0.5f? No, 0.5 is exactly representable as a float, so there's no truncation.
Comment on attachment 158365 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=158365&action=review >>> Source/WebKit/chromium/tests/CCRenderSurfaceFiltersTest.cpp:56 >>> EXPECT_TRUE(isCombined(WebFilterOperation::createGrayscaleFilter(0.5))); >> >> Don't you also need to change 0.5 to 0.5f? > > No, 0.5 is exactly representable as a float, so there's no truncation. Makes sense. Then let's add the comment. People will wonder why this test uses 0.3f and 0.5.
Created attachment 158445 [details] Patch
Comment on attachment 158445 [details] Patch Looks OK
Comment on attachment 158445 [details] Patch Clearing flags on attachment: 158445 Committed r125625: <http://trac.webkit.org/changeset/125625>
All reviewed patches have been landed. Closing bug.