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

Unified Diff: third_party/WebKit/LayoutTests/shapedetection/detection-HTMLCanvasElement.html

Issue 2553343003: ShapeDetection: add support for all CanvasImageSource input types (Closed)
Patch Set: Support case for peekPixels() and isImageBitmap() Created 4 years 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/LayoutTests/shapedetection/detection-HTMLCanvasElement.html
diff --git a/third_party/WebKit/LayoutTests/shapedetection/detection-HTMLCanvasElement.html b/third_party/WebKit/LayoutTests/shapedetection/detection-HTMLCanvasElement.html
new file mode 100644
index 0000000000000000000000000000000000000000..070721c1409d033ef762f9621ca8108a1477fe73
--- /dev/null
+++ b/third_party/WebKit/LayoutTests/shapedetection/detection-HTMLCanvasElement.html
@@ -0,0 +1,95 @@
+<!DOCTYPE html>
+<script src="../resources/testharness.js"></script>
+<script src="../resources/testharnessreport.js"></script>
+<script src="../resources/mojo-helpers.js"></script>
+<script src="resources/mock-barcodedetection.js"></script>
+<script src="resources/mock-facedetection.js"></script>
+<script>
+
+var createTestForCanvasElement = function(detectorName,
+ createCanvas,
+ mockReady,
+ detectionResultTest) {
+ async_test(function(t) {
+
+ var img = new Image();
+ img.onload = function() {
+
+ var canvas = createCanvas(); ////document.createElement('canvas');
Reilly Grant (use Gerrit) 2016/12/07 21:37:06 Leftover comment?
mcasas 2016/12/07 22:02:37 Oops, yes, removed.
+ canvas.getContext("2d").drawImage(img, 0, 0);
+
+ var theMock = null;
+ mockReady()
+ .then(mock => {
+ theMock = mock;
+ var detector = eval("new " + detectorName + "();");
Reilly Grant (use Gerrit) 2016/12/07 21:37:06 Why not pass the constructor function instead of a
mcasas 2016/12/07 22:02:37 Done. Note that you can't pass |bla| and evaluate
+ return detector;
+ })
+ .catch(error => {
+ assert_unreached("Error creating Mock Detector: " + error);
+ })
+ .then(detector => {
+ return detector.detect(canvas);
+ })
+ .then(detectionResult => {
+ detectionResultTest(detectionResult, theMock);
+ t.done();
+ })
+ .catch(error => {
+ assert_unreached("Error during detect(canvas): " + error);
+ });
+ }
+
+ img.src = '../media/content/greenbox.png';
Reilly Grant (use Gerrit) 2016/12/07 21:37:06 Be consistent about ' vs. ".
mcasas 2016/12/07 22:02:37 Done.
+ }, "Detector detect(HTMLCanvasElement)");
+};
+
+function FaceDetectorDetectionResultTest(detectionResult, mock) {
+ const imageReceivedByMock = mock.getFrameData();
+ assert_equals(imageReceivedByMock.byteLength, 180000,"Image length");
+ const GREEN_PIXEL = 0xFF00FF00;
+ assert_equals(imageReceivedByMock[0], GREEN_PIXEL, "Pixel color");
+ assert_equals(detectionResult.length, 3, "Number of faces");
+}
+
+function BarcodeDetectorDetectionResultTest(detectionResult, mock) {
+ assert_equals(detectionResult.length, 2, "Number of barcodes");
+ assert_equals(detectionResult[0].rawValue, "cats", "barcode 1");
+ assert_equals(detectionResult[1].rawValue, "dogs", "barcode 2");
+}
+
+// These tests verify that a Detector's detect() works on an HTMLCanvasElement
+// and on an OffscreenCanvas. Use the mock mojo server implemented in
+// mock-{barcode,face}detection.js.
+generate_tests(createTestForCanvasElement, [
+ [
+ "Face - HTMLCanvasElement",
+ "FaceDetector",
+ () => { return document.createElement('canvas'); },
+ () => { return mockFaceDetectionReady; },
+ FaceDetectorDetectionResultTest
+ ],
+ [
+ "Face - OffscreenCanvas",
+ "FaceDetector",
+ () => { return new OffscreenCanvas(300, 150); },
+ () => { return mockFaceDetectionReady; },
+ FaceDetectorDetectionResultTest
+ ],
+ [
+ "Barcode - HTMLCanvasElement",
+ "BarcodeDetector",
+ () => { return document.createElement('canvas'); },
+ () => { return mockBarcodeDetectionReady; },
+ BarcodeDetectorDetectionResultTest
+ ],
+ [
+ "Barcode - OffscreenCanvas",
+ "BarcodeDetector",
+ () => { return new OffscreenCanvas(300, 150); },
+ () => { return mockBarcodeDetectionReady; },
+ BarcodeDetectorDetectionResultTest
+ ]
+]);
+
+</script>

Powered by Google App Engine
This is Rietveld 408576698