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

Unified Diff: third_party/WebKit/Source/core/layout/ImageQualityControllerTest.cpp

Issue 2395683002: Revert of Reformat comments in core/layout up until LayoutBox (Closed)
Patch Set: Created 4 years, 2 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/core/layout/ImageQualityControllerTest.cpp
diff --git a/third_party/WebKit/Source/core/layout/ImageQualityControllerTest.cpp b/third_party/WebKit/Source/core/layout/ImageQualityControllerTest.cpp
index b630595a7df251476157c6d123165ed64beb5b53..dd38bcd5aa5fa637232979000cebc22b1ec6c264 100644
--- a/third_party/WebKit/Source/core/layout/ImageQualityControllerTest.cpp
+++ b/third_party/WebKit/Source/core/layout/ImageQualityControllerTest.cpp
@@ -183,8 +183,7 @@
std::unique_ptr<PaintController> paintController = PaintController::create();
GraphicsContext context(*paintController);
- // Paint once. This will kick off a timer to see if we resize it during that
- // timer's execution.
+ // Paint once. This will kick off a timer to see if we resize it during that timer's execution.
EXPECT_EQ(InterpolationMedium,
controller()->chooseInterpolationQuality(
*img, testImage.get(), testImage.get(), LayoutSize(2, 2)));
@@ -200,8 +199,7 @@
*img, testImage.get(), testImage.get(), LayoutSize(4, 4)));
mockTimer->fire();
- // The timer fired before painting at another size, so this doesn't count as
- // animation. Therefore not painting at low quality.
+ // The timer fired before painting at another size, so this doesn't count as animation. Therefore not painting at low quality.
EXPECT_EQ(InterpolationMedium,
controller()->chooseInterpolationQuality(
*img, testImage.get(), testImage.get(), LayoutSize(4, 4)));
@@ -226,8 +224,7 @@
std::unique_ptr<PaintController> paintController = PaintController::create();
GraphicsContext context(*paintController);
- // Paint once. This will kick off a timer to see if we resize it during that
- // timer's execution.
+ // Paint once. This will kick off a timer to see if we resize it during that timer's execution.
EXPECT_EQ(InterpolationMedium, controller()->chooseInterpolationQuality(
*animatingImage, testImage.get(),
testImage.get(), LayoutSize(2, 2)));
@@ -237,21 +234,18 @@
*animatingImage, testImage.get(),
testImage.get(), LayoutSize(3, 3)));
- // The non-animating image receives a medium-quality filter, even though the
- // other one is animating.
+ // The non-animating image receives a medium-quality filter, even though the other one is animating.
EXPECT_EQ(InterpolationMedium, controller()->chooseInterpolationQuality(
*nonAnimatingImage, testImage.get(),
testImage.get(), LayoutSize(4, 4)));
- // Now the second image has animated, so it also gets painted with a
- // low-quality filter.
+ // Now the second image has animated, so it also gets painted with a low-quality filter.
EXPECT_EQ(InterpolationLow, controller()->chooseInterpolationQuality(
*nonAnimatingImage, testImage.get(),
testImage.get(), LayoutSize(3, 3)));
mockTimer->fire();
- // The timer fired before painting at another size, so this doesn't count as
- // animation. Therefore not painting at low quality for any image.
+ // The timer fired before painting at another size, so this doesn't count as animation. Therefore not painting at low quality for any image.
EXPECT_EQ(InterpolationMedium, controller()->chooseInterpolationQuality(
*animatingImage, testImage.get(),
testImage.get(), LayoutSize(4, 4)));
@@ -273,8 +267,7 @@
RefPtr<TestImageLowQuality> testImage = adoptRef(new TestImageLowQuality);
- // Paint once. This will kick off a timer to see if we resize it during that
- // timer's execution.
+ // Paint once. This will kick off a timer to see if we resize it during that timer's execution.
EXPECT_EQ(InterpolationMedium,
controller()->chooseInterpolationQuality(
*img, testImage.get(), testImage.get(), LayoutSize(2, 2)));
@@ -291,13 +284,11 @@
mockTimer->stop();
EXPECT_FALSE(mockTimer->isActive());
- // Painted at the same size, so even though timer is still executing, don't go
- // to low quality.
+ // Painted at the same size, so even though timer is still executing, don't go to low quality.
EXPECT_EQ(InterpolationLow,
controller()->chooseInterpolationQuality(
*img, testImage.get(), testImage.get(), LayoutSize(4, 4)));
- // Check that the timer was not kicked. It should not have been, since the
- // image was painted at the same size as last time.
+ // Check that the timer was not kicked. It should not have been, since the image was painted at the same size as last time.
EXPECT_FALSE(mockTimer->isActive());
}
@@ -313,8 +304,7 @@
RefPtr<TestImageLowQuality> testImage = adoptRef(new TestImageLowQuality);
- // Paint once. This will kick off a timer to see if we resize it during that
- // timer's execution.
+ // Paint once. This will kick off a timer to see if we resize it during that timer's execution.
mockTimer->setTime(0.1);
EXPECT_FALSE(controller()->shouldPaintAtLowQuality(
*img, testImage.get(), testImage.get(), LayoutSize(2, 2), 0.1));
@@ -332,8 +322,7 @@
ImageQualityController::cTimerRestartThreshold / 2.0,
mockTimer->nextFireInterval());
- // This animation is far enough in the future to make the timer restart, since
- // it is half over.
+ // This animation is far enough in the future to make the timer restart, since it is half over.
nextTime = 0.1 + ImageQualityController::cTimerRestartThreshold + 0.01;
EXPECT_EQ(true, controller()->shouldPaintAtLowQuality(
*img, testImage.get(), testImage.get(), LayoutSize(4, 4),
« no previous file with comments | « third_party/WebKit/Source/core/layout/ImageQualityController.cpp ('k') | third_party/WebKit/Source/core/layout/LayoutBR.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698