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

Unified Diff: chrome/browser/image_decoder_browsertest.cc

Issue 2715153006: Set desired_image_size when decoding images for NTP Tile icons. (Closed)
Patch Set: comment formatting Created 3 years, 10 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 | « chrome/browser/image_decoder.cc ('k') | chrome/browser/search/suggestions/image_decoder_impl.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/image_decoder_browsertest.cc
diff --git a/chrome/browser/image_decoder_browsertest.cc b/chrome/browser/image_decoder_browsertest.cc
index 602e3a4f6f06650a667a7f3ca21597b4113a78f0..a66d5bde6be7e2e7148c93bce7664c64a590f298 100644
--- a/chrome/browser/image_decoder_browsertest.cc
+++ b/chrome/browser/image_decoder_browsertest.cc
@@ -178,7 +178,7 @@ IN_PROC_BROWSER_TEST_F(ImageDecoderBrowserTest, BasicDecodeWithOptionsString) {
ImageDecoder::StartWithOptions(&test_request,
std::string(data.begin(), data.end()),
ImageDecoder::ROBUST_PNG_CODEC,
- false /* shrink_to_fit */);
+ /*shrink_to_fit=*/false);
runner->Run();
EXPECT_TRUE(test_request.decode_succeeded());
}
@@ -187,9 +187,9 @@ IN_PROC_BROWSER_TEST_F(ImageDecoderBrowserTest, RobustJpegCodecWithJpegData) {
scoped_refptr<content::MessageLoopRunner> runner =
new content::MessageLoopRunner;
TestImageRequest test_request(runner->QuitClosure());
- ImageDecoder::StartWithOptions(&test_request, GetValidJpgData(),
- ImageDecoder::ROBUST_JPEG_CODEC,
- false /* shrink_to_fit */);
+ ImageDecoder::StartWithOptions(
+ &test_request, GetValidJpgData(), ImageDecoder::ROBUST_JPEG_CODEC,
+ /*shrink_to_fit=*/false, /*desired_image_frame_size=*/gfx::Size());
runner->Run();
EXPECT_TRUE(test_request.decode_succeeded());
}
@@ -198,9 +198,9 @@ IN_PROC_BROWSER_TEST_F(ImageDecoderBrowserTest, RobustJpegCodecWithPngData) {
scoped_refptr<content::MessageLoopRunner> runner =
new content::MessageLoopRunner;
TestImageRequest test_request(runner->QuitClosure());
- ImageDecoder::StartWithOptions(&test_request, GetValidPngData(),
- ImageDecoder::ROBUST_JPEG_CODEC,
- false /* shrink_to_fit */);
+ ImageDecoder::StartWithOptions(
+ &test_request, GetValidPngData(), ImageDecoder::ROBUST_JPEG_CODEC,
+ /*shrink_to_fit=*/false, /*desired_image_frame_size=*/gfx::Size());
runner->Run();
// Should fail with PNG data because only JPEG data is allowed.
EXPECT_FALSE(test_request.decode_succeeded());
@@ -210,9 +210,9 @@ IN_PROC_BROWSER_TEST_F(ImageDecoderBrowserTest, RobustPngCodecWithPngData) {
scoped_refptr<content::MessageLoopRunner> runner =
new content::MessageLoopRunner;
TestImageRequest test_request(runner->QuitClosure());
- ImageDecoder::StartWithOptions(&test_request, GetValidPngData(),
- ImageDecoder::ROBUST_PNG_CODEC,
- false /* shrink_to_fit */);
+ ImageDecoder::StartWithOptions(
+ &test_request, GetValidPngData(), ImageDecoder::ROBUST_PNG_CODEC,
+ /*shrink_to_fit=*/false, /*desired_image_frame_size=*/gfx::Size());
runner->Run();
EXPECT_TRUE(test_request.decode_succeeded());
}
@@ -221,9 +221,9 @@ IN_PROC_BROWSER_TEST_F(ImageDecoderBrowserTest, RobustPngCodecWithJpegData) {
scoped_refptr<content::MessageLoopRunner> runner =
new content::MessageLoopRunner;
TestImageRequest test_request(runner->QuitClosure());
- ImageDecoder::StartWithOptions(&test_request, GetValidJpgData(),
- ImageDecoder::ROBUST_PNG_CODEC,
- false /* shrink_to_fit */);
+ ImageDecoder::StartWithOptions(
+ &test_request, GetValidJpgData(), ImageDecoder::ROBUST_PNG_CODEC,
+ /*shrink_to_fit=*/false, /*desired_image_frame_size=*/gfx::Size());
runner->Run();
// Should fail with JPEG data because only PNG data is allowed.
EXPECT_FALSE(test_request.decode_succeeded());
« no previous file with comments | « chrome/browser/image_decoder.cc ('k') | chrome/browser/search/suggestions/image_decoder_impl.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698