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

Side by Side Diff: third_party/WebKit/Source/modules/shapedetection/BarcodeDetector.cpp

Issue 2755393002: Revert of RELAND: ShapeDetection: use mojom::Bitmap for mojo interface. (Closed)
Patch Set: Created 3 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 unified diff | Download patch
OLDNEW
1 // Copyright 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "modules/shapedetection/BarcodeDetector.h" 5 #include "modules/shapedetection/BarcodeDetector.h"
6 6
7 #include "core/dom/DOMException.h" 7 #include "core/dom/DOMException.h"
8 #include "core/dom/DOMRect.h" 8 #include "core/dom/DOMRect.h"
9 #include "core/html/canvas/CanvasImageSource.h" 9 #include "core/html/canvas/CanvasImageSource.h"
10 #include "modules/imagecapture/Point2D.h" 10 #include "modules/imagecapture/Point2D.h"
11 #include "modules/shapedetection/DetectedBarcode.h" 11 #include "modules/shapedetection/DetectedBarcode.h"
12 #include "public/platform/InterfaceProvider.h" 12 #include "public/platform/InterfaceProvider.h"
13 #include "public/platform/Platform.h" 13 #include "public/platform/Platform.h"
14 14
15 namespace blink { 15 namespace blink {
16 16
17 BarcodeDetector* BarcodeDetector::create() { 17 BarcodeDetector* BarcodeDetector::create() {
18 return new BarcodeDetector(); 18 return new BarcodeDetector();
19 } 19 }
20 20
21 BarcodeDetector::BarcodeDetector() : ShapeDetector() { 21 BarcodeDetector::BarcodeDetector() : ShapeDetector() {
22 Platform::current()->interfaceProvider()->getInterface( 22 Platform::current()->interfaceProvider()->getInterface(
23 mojo::MakeRequest(&m_barcodeService)); 23 mojo::MakeRequest(&m_barcodeService));
24 m_barcodeService.set_connection_error_handler(convertToBaseCallback( 24 m_barcodeService.set_connection_error_handler(convertToBaseCallback(
25 WTF::bind(&BarcodeDetector::onBarcodeServiceConnectionError, 25 WTF::bind(&BarcodeDetector::onBarcodeServiceConnectionError,
26 wrapWeakPersistent(this)))); 26 wrapWeakPersistent(this))));
27 } 27 }
28 28
29 ScriptPromise BarcodeDetector::doDetect(ScriptPromiseResolver* resolver, 29 ScriptPromise BarcodeDetector::doDetect(
30 skia::mojom::blink::BitmapPtr bitmap) { 30 ScriptPromiseResolver* resolver,
31 mojo::ScopedSharedBufferHandle sharedBufferHandle,
32 int imageWidth,
33 int imageHeight) {
31 ScriptPromise promise = resolver->promise(); 34 ScriptPromise promise = resolver->promise();
32 if (!m_barcodeService) { 35 if (!m_barcodeService) {
33 resolver->reject(DOMException::create( 36 resolver->reject(DOMException::create(
34 NotSupportedError, "Barcode detection service unavailable.")); 37 NotSupportedError, "Barcode detection service unavailable."));
35 return promise; 38 return promise;
36 } 39 }
37 m_barcodeServiceRequests.insert(resolver); 40 m_barcodeServiceRequests.insert(resolver);
38 m_barcodeService->Detect( 41 m_barcodeService->Detect(
39 std::move(bitmap), convertToBaseCallback(WTF::bind( 42 std::move(sharedBufferHandle), imageWidth, imageHeight,
40 &BarcodeDetector::onDetectBarcodes, 43 convertToBaseCallback(WTF::bind(&BarcodeDetector::onDetectBarcodes,
41 wrapPersistent(this), wrapPersistent(resolver)))); 44 wrapPersistent(this),
45 wrapPersistent(resolver))));
42 return promise; 46 return promise;
43 } 47 }
44 48
45 void BarcodeDetector::onDetectBarcodes( 49 void BarcodeDetector::onDetectBarcodes(
46 ScriptPromiseResolver* resolver, 50 ScriptPromiseResolver* resolver,
47 Vector<shape_detection::mojom::blink::BarcodeDetectionResultPtr> 51 Vector<shape_detection::mojom::blink::BarcodeDetectionResultPtr>
48 barcodeDetectionResults) { 52 barcodeDetectionResults) {
49 DCHECK(m_barcodeServiceRequests.contains(resolver)); 53 DCHECK(m_barcodeServiceRequests.contains(resolver));
50 m_barcodeServiceRequests.erase(resolver); 54 m_barcodeServiceRequests.erase(resolver);
51 55
(...skipping 25 matching lines...) Expand all
77 m_barcodeServiceRequests.clear(); 81 m_barcodeServiceRequests.clear();
78 m_barcodeService.reset(); 82 m_barcodeService.reset();
79 } 83 }
80 84
81 DEFINE_TRACE(BarcodeDetector) { 85 DEFINE_TRACE(BarcodeDetector) {
82 ShapeDetector::trace(visitor); 86 ShapeDetector::trace(visitor);
83 visitor->trace(m_barcodeServiceRequests); 87 visitor->trace(m_barcodeServiceRequests);
84 } 88 }
85 89
86 } // namespace blink 90 } // namespace blink
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698