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

Unified Diff: third_party/WebKit/Source/core/html/canvas/CanvasAsyncBlobCreatorTest.cpp

Issue 2878333004: Use SkJpegEncoder in WebKit platform (Closed)
Patch Set: Create instead of Make Created 3 years, 7 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/html/canvas/CanvasAsyncBlobCreatorTest.cpp
diff --git a/third_party/WebKit/Source/core/html/canvas/CanvasAsyncBlobCreatorTest.cpp b/third_party/WebKit/Source/core/html/canvas/CanvasAsyncBlobCreatorTest.cpp
index ff775f35c89efa1dd5d7e23fa7a3f869f6d0228f..ab50cc725da4b1b5b678e834081ca2223d5330ca 100644
--- a/third_party/WebKit/Source/core/html/canvas/CanvasAsyncBlobCreatorTest.cpp
+++ b/third_party/WebKit/Source/core/html/canvas/CanvasAsyncBlobCreatorTest.cpp
@@ -72,8 +72,8 @@ class MockCanvasAsyncBlobCreatorWithoutStartPng
: MockCanvasAsyncBlobCreator(data, size, kMimeTypePng, document) {}
protected:
- void ScheduleInitiatePngEncoding() override {
- // Deliberately make scheduleInitiatePngEncoding do nothing so that idle
+ void ScheduleInitiateEncoding(double) override {
+ // Deliberately make scheduleInitiateEncoding do nothing so that idle
// task never starts
}
};
@@ -89,16 +89,16 @@ class MockCanvasAsyncBlobCreatorWithoutCompletePng
: MockCanvasAsyncBlobCreator(data, size, kMimeTypePng, document) {}
protected:
- void ScheduleInitiatePngEncoding() override {
+ void ScheduleInitiateEncoding(double quality) override {
Platform::Current()->MainThread()->GetWebTaskRunner()->PostTask(
BLINK_FROM_HERE,
WTF::Bind(
- &MockCanvasAsyncBlobCreatorWithoutCompletePng::InitiatePngEncoding,
- WrapPersistent(this), std::numeric_limits<double>::max()));
+ &MockCanvasAsyncBlobCreatorWithoutCompletePng::InitiateEncoding,
+ WrapPersistent(this), quality, std::numeric_limits<double>::max()));
}
- void IdleEncodeRowsPng(double deadline_seconds) override {
- // Deliberately make idleEncodeRowsPng do nothing so that idle task never
+ void IdleEncodeRows(double deadline_seconds) override {
+ // Deliberately make idleEncodeRows do nothing so that idle task never
// completes
}
};
@@ -116,8 +116,8 @@ class MockCanvasAsyncBlobCreatorWithoutStartJpeg
: MockCanvasAsyncBlobCreator(data, size, kMimeTypeJpeg, document) {}
protected:
- void ScheduleInitiateJpegEncoding(const double&) override {
- // Deliberately make scheduleInitiateJpegEncoding do nothing so that idle
+ void ScheduleInitiateEncoding(double) override {
+ // Deliberately make scheduleInitiateEncoding do nothing so that idle
// task never starts
}
};
@@ -133,17 +133,16 @@ class MockCanvasAsyncBlobCreatorWithoutCompleteJpeg
: MockCanvasAsyncBlobCreator(data, size, kMimeTypeJpeg, document) {}
protected:
- void ScheduleInitiateJpegEncoding(const double& quality) override {
+ void ScheduleInitiateEncoding(double quality) override {
Platform::Current()->MainThread()->GetWebTaskRunner()->PostTask(
BLINK_FROM_HERE,
- WTF::Bind(&MockCanvasAsyncBlobCreatorWithoutCompleteJpeg::
- InitiateJpegEncoding,
- WrapPersistent(this), quality,
- std::numeric_limits<double>::max()));
+ WTF::Bind(
+ &MockCanvasAsyncBlobCreatorWithoutCompleteJpeg::InitiateEncoding,
+ WrapPersistent(this), quality, std::numeric_limits<double>::max()));
}
- void IdleEncodeRowsJpeg(double deadline_seconds) override {
- // Deliberately make idleEncodeRowsJpeg do nothing so that idle task never
+ void IdleEncodeRows(double deadline_seconds) override {
+ // Deliberately make idleEncodeRows do nothing so that idle task never
// completes
}
};
« no previous file with comments | « third_party/WebKit/Source/core/html/canvas/CanvasAsyncBlobCreator.cpp ('k') | third_party/WebKit/Source/platform/BUILD.gn » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698