Shapedetection module: Blink side implementation

third_party/WebKit/LayoutTests/shapedetection/resources/mock-shapedetection.js

+"use strict";
+
+let mockShapeDetectionReady = define(
+  'mockShapeDetection',
+  ['third_party/WebKit/public/platform/modules/shapedetection/shapedetection.mojom',
+   'mojo/public/js/bindings',
+   'mojo/public/js/connection',
+   'mojo/public/js/core',
+   'content/public/renderer/frame_interfaces',
+  ], (shapeDetection, bindings, connection, mojo, interfaces) => {
+
+  class MockShapeDetection {
+    constructor() {
+      interfaces.addInterfaceOverrideForTesting(
+          shapeDetection.ShapeDetection.name,
+          pipe => this.bindToPipe(pipe));
+    }
+
+    bindToPipe(pipe) {
+      this.stub_ = connection.bindHandleToStub(pipe, shapeDetection.ShapeDetection);
+      bindings.StubBindings(this.stub_).delegate = this;
+    }
+
+    detectFace(frame_data, width, height) {
+      let mojoResult = mojo.mapBuffer(frame_data, 0 /*offset*/, width*height*4/*size*/, 0/*flag*/);

[Ken Rockot(use gerrit already), 2016/09/30 18:05:23]

nit: the inline comments are probably unnecessary

also nit: just result instead of mojoResult?

[xianglu, 2016/10/03 18:12:07]

Done.

+      this.buffer_data_ = new Uint32Array(mojoResult.buffer);
+      return Promise.resolve(
+        { boundingBoxes :

[Ken Rockot(use gerrit already), 2016/09/30 18:05:23]

style nit: I don't think this is correct indenting, though I might be wrong? I'd
expect:

    return Promise.resolve({
      boundingBoxes: [
        { ... },
        { ... },
      ]
    });

[xianglu, 2016/10/03 18:12:07]

Done.

+          [
+            { x : 1.0, y: 1.0, width: 100.0, height: 100.0 },
+            { x : 2.0, y: 2.0, width: 200.0, height: 200.0 },
+            { x : 3.0, y: 3.0, width: 300.0, height: 300.0 },
+          ]
+        });
+      mojo.unmapBuffer(mojoResult.buffer);
+    }
+
+    getFrameData() {
+      return this.buffer_data_;
+    }
+  }
+  return new MockShapeDetection();
+});