Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(647)

Unified Diff: third_party/WebKit/Source/platform/graphics/DeferredImageDecoderTest.cpp

Issue 1571233003: Fix errors caused by unsafe conversions to/from size_t (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: improved ALLOW_NUMERIC_ARG_TYPES_PROMOTABLE_TO Created 4 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Source/platform/graphics/DeferredImageDecoderTest.cpp
diff --git a/third_party/WebKit/Source/platform/graphics/DeferredImageDecoderTest.cpp b/third_party/WebKit/Source/platform/graphics/DeferredImageDecoderTest.cpp
index 417b2c83380b653c727818f3283cce6931b85ed0..bb60190cba85be49ea519702927a2614fd2d2284 100644
--- a/third_party/WebKit/Source/platform/graphics/DeferredImageDecoderTest.cpp
+++ b/third_party/WebKit/Source/platform/graphics/DeferredImageDecoderTest.cpp
@@ -248,7 +248,7 @@ TEST_F(DeferredImageDecoderTest, singleFrameImageLoading)
EXPECT_TRUE(m_actualDecoder);
m_status = ImageFrame::FrameComplete;
- m_data->append(" ", 1);
+ m_data->append(" ", 1u);
m_lazyDecoder->setData(*m_data, true);
EXPECT_FALSE(m_actualDecoder);
EXPECT_TRUE(m_lazyDecoder->frameIsCompleteAtIndex(0));
@@ -277,7 +277,7 @@ TEST_F(DeferredImageDecoderTest, multiFrameImageLoading)
m_frameCount = 2;
m_frameDuration = 20;
m_status = ImageFrame::FrameComplete;
- m_data->append(" ", 1);
+ m_data->append(" ", 1u);
m_lazyDecoder->setData(*m_data, false);
image = m_lazyDecoder->createFrameAtIndex(0);

Powered by Google App Engine
This is Rietveld 408576698