New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 668343 link

Starred by 3 users

Issue metadata

Status: Fixed
Owner:
Closed: Dec 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Mac
Pri: 2
Type: Bug



Sign in to add a comment

Shape Detection API causes perpetually-pending promise

Reported by sime.vi...@gmail.com, Nov 24 2016

Issue description

UserAgent: Mozilla/5.0 (Windows NT 10.0; WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/54.0.2840.99 Safari/537.36

Steps to reproduce the problem:
1. Go to https://output.jsbin.com/gegudoc/quiet (this is Paul Kinlan’s demo) in Chrome Canary with Web Platform flag enabled on Windows 10
2. Pressing the “Detect faces” button will invoke the Shape Detection API detect method

What is the expected behavior?
The detect method should result in a resolved or rejected promise.

What went wrong?
The promise remains in a pending state. You can open DevTools and set breakpoints in the then() and catch() handlers to confirm that neither one is invoked.

Did this work before? N/A 

Does this work in other browsers? Yes

Chrome version: 57.0.2929.3  Channel: canary
OS Version: 10.0
Flash Version: Shockwave Flash 24.0 r0
 

Comment 1 by ajha@chromium.org, Nov 24 2016

Labels: Needs-Bisect M-57
Cc: sureshkumari@chromium.org
Components: Blink>JavaScript>API
Labels: -Needs-Bisect OS-Linux OS-Mac
Status: Untriaged (was: Unconfirmed)
Able to reproduce the issue on windows-7, Mac-10.11.6 and Linux Ubuntu-14.04 using 
chrome stable version 54.0.2840.99 and latest canary 57.0.2931.0 with the steps mentioned above.

This is non-regression issue observed from M-47#47.0.2491.0. Hence marking it as Untriaged to get it addressed.
Please find the attached screencast.

Note:From M30 to M46 the image was not loaded even when the plugins are enabled.

Thanks.
issue-668343.mp4
874 KB View Download
Cc: mcasas@chromium.org xianglu@chromium.org
Components: -Blink>JavaScript>API Blink
Components: -Blink Blink>ImageCapture

Comment 5 by mcasas@chromium.org, Nov 28 2016

Labels: ShapeDetection

Comment 6 by mcasas@chromium.org, Nov 28 2016

Owner: mcasas@chromium.org
Status: Assigned (was: Untriaged)
There is no Face Detection implementation for Windows (yet?), so probably
what happens is that the Promise is waiting to be resolved by an
inexistent mojo interface implementation. Will make sure the Promise
is correctly rejected, in the meantime please try to use Android, where
there is a working implementation (Mac being worked on too).
Project Member

Comment 7 by bugdroid1@chromium.org, Dec 1 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/62beea5d8166bee43361033475448a0a08cd5bfa

commit 62beea5d8166bee43361033475448a0a08cd5bfa
Author: mcasas <mcasas@chromium.org>
Date: Wed Nov 30 23:59:43 2016

Barcode/Face detection: reject requests in platforms where not implemented

This CL adds on_connection_error listeners to the mojo interface
pointers used for face/barcode detection. The connection error
handler is used to reject all Promises related to the given interface,
otherwise requests from Blink are neither resolved nor rejected (see bug).

BUG= 668343 
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation

Review-Url: https://codereview.chromium.org/2538053002
Cr-Commit-Position: refs/heads/master@{#435484}

[modify] https://crrev.com/62beea5d8166bee43361033475448a0a08cd5bfa/third_party/WebKit/Source/modules/shapedetection/ShapeDetector.cpp
[modify] https://crrev.com/62beea5d8166bee43361033475448a0a08cd5bfa/third_party/WebKit/Source/modules/shapedetection/ShapeDetector.h

Status: Fixed (was: Assigned)
#7 should solve this bug, can anyone please verify
when it hits the Canary?

Sign in to add a comment