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

Unified Diff: trunk/src/cc/scheduler/texture_uploader_unittest.cc

Issue 13316003: Revert 191364 "cc: Add ‘chromium_code’: 1 to cc.gyp and cc_t..." (Closed) Base URL: svn://svn.chromium.org/chrome/
Patch Set: Created 7 years, 9 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
« no previous file with comments | « trunk/src/cc/scheduler/texture_uploader.cc ('k') | trunk/src/cc/scheduler/vsync_time_source.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: trunk/src/cc/scheduler/texture_uploader_unittest.cc
===================================================================
--- trunk/src/cc/scheduler/texture_uploader_unittest.cc (revision 191370)
+++ trunk/src/cc/scheduler/texture_uploader_unittest.cc (working copy)
@@ -70,7 +70,7 @@
WGC3Denum format,
WGC3Denum type,
const void* pixels) OVERRIDE {
- EXPECT_EQ(static_cast<unsigned>(GL_TEXTURE_2D), target);
+ EXPECT_EQ(GL_TEXTURE_2D, target);
EXPECT_EQ(0, level);
EXPECT_LE(0, width);
EXPECT_LE(0, height);
@@ -83,12 +83,12 @@
unsigned int bytes_per_pixel = 0;
switch (format) {
case GL_ALPHA:
- EXPECT_EQ(static_cast<unsigned>(GL_UNSIGNED_BYTE), type);
+ EXPECT_EQ(GL_UNSIGNED_BYTE, type);
bytes_per_pixel = 1;
break;
case GL_RGB:
- EXPECT_NE(static_cast<unsigned>(GL_UNSIGNED_SHORT_4_4_4_4), type);
- EXPECT_NE(static_cast<unsigned>(GL_UNSIGNED_SHORT_5_5_5_1), type);
+ EXPECT_NE(GL_UNSIGNED_SHORT_4_4_4_4, type);
+ EXPECT_NE(GL_UNSIGNED_SHORT_5_5_5_1, type);
switch (type) {
case GL_UNSIGNED_BYTE:
bytes_per_pixel = 3;
@@ -99,7 +99,7 @@
}
break;
case GL_RGBA:
- EXPECT_NE(static_cast<unsigned>(GL_UNSIGNED_SHORT_5_6_5), type);
+ EXPECT_NE(GL_UNSIGNED_SHORT_5_6_5, type);
switch (type) {
case GL_UNSIGNED_BYTE:
bytes_per_pixel = 4;
@@ -113,11 +113,11 @@
}
break;
case GL_LUMINANCE:
- EXPECT_EQ(static_cast<unsigned>(GL_UNSIGNED_BYTE), type);
+ EXPECT_EQ(GL_UNSIGNED_BYTE, type);
bytes_per_pixel = 1;
break;
case GL_LUMINANCE_ALPHA:
- EXPECT_EQ(static_cast<unsigned>(GL_UNSIGNED_BYTE), type);
+ EXPECT_EQ(GL_UNSIGNED_BYTE, type);
bytes_per_pixel = 2;
break;
}
@@ -144,8 +144,8 @@
}
private:
+ unsigned unpack_alignment_;
unsigned result_available_;
- unsigned unpack_alignment_;
DISALLOW_COPY_AND_ASSIGN(TestWebGraphicsContext3DTextureUpload);
};
@@ -169,19 +169,19 @@
TextureUploader::Create(fake_context.get(), false, false);
fake_context->SetResultAvailable(0);
- EXPECT_EQ(0u, uploader->NumBlockingUploads());
+ EXPECT_EQ(0, uploader->NumBlockingUploads());
UploadTexture(uploader.get(), GL_RGBA, gfx::Size(), NULL);
- EXPECT_EQ(1u, uploader->NumBlockingUploads());
+ EXPECT_EQ(1, uploader->NumBlockingUploads());
UploadTexture(uploader.get(), GL_RGBA, gfx::Size(), NULL);
- EXPECT_EQ(2u, uploader->NumBlockingUploads());
+ EXPECT_EQ(2, uploader->NumBlockingUploads());
fake_context->SetResultAvailable(1);
- EXPECT_EQ(0u, uploader->NumBlockingUploads());
+ EXPECT_EQ(0, uploader->NumBlockingUploads());
UploadTexture(uploader.get(), GL_RGBA, gfx::Size(), NULL);
- EXPECT_EQ(0u, uploader->NumBlockingUploads());
+ EXPECT_EQ(0, uploader->NumBlockingUploads());
UploadTexture(uploader.get(), GL_RGBA, gfx::Size(), NULL);
UploadTexture(uploader.get(), GL_RGBA, gfx::Size(), NULL);
- EXPECT_EQ(0u, uploader->NumBlockingUploads());
+ EXPECT_EQ(0, uploader->NumBlockingUploads());
}
TEST(TextureUploaderTest, MarkPendingUploadsAsNonBlocking) {
@@ -191,21 +191,21 @@
TextureUploader::Create(fake_context.get(), false, false);
fake_context->SetResultAvailable(0);
- EXPECT_EQ(0u, uploader->NumBlockingUploads());
+ EXPECT_EQ(0, uploader->NumBlockingUploads());
UploadTexture(uploader.get(), GL_RGBA, gfx::Size(), NULL);
UploadTexture(uploader.get(), GL_RGBA, gfx::Size(), NULL);
- EXPECT_EQ(2u, uploader->NumBlockingUploads());
+ EXPECT_EQ(2, uploader->NumBlockingUploads());
uploader->MarkPendingUploadsAsNonBlocking();
- EXPECT_EQ(0u, uploader->NumBlockingUploads());
+ EXPECT_EQ(0, uploader->NumBlockingUploads());
UploadTexture(uploader.get(), GL_RGBA, gfx::Size(), NULL);
- EXPECT_EQ(1u, uploader->NumBlockingUploads());
+ EXPECT_EQ(1, uploader->NumBlockingUploads());
fake_context->SetResultAvailable(1);
- EXPECT_EQ(0u, uploader->NumBlockingUploads());
+ EXPECT_EQ(0, uploader->NumBlockingUploads());
UploadTexture(uploader.get(), GL_RGBA, gfx::Size(), NULL);
uploader->MarkPendingUploadsAsNonBlocking();
- EXPECT_EQ(0u, uploader->NumBlockingUploads());
+ EXPECT_EQ(0, uploader->NumBlockingUploads());
}
TEST(TextureUploaderTest, UploadContentsTest) {
« no previous file with comments | « trunk/src/cc/scheduler/texture_uploader.cc ('k') | trunk/src/cc/scheduler/vsync_time_source.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698