Issue metadata
Sign in to add a comment
|
Launch Shape Detection API |
||||||||||||||||||||||||||||||||||||||||||
Issue descriptionAdd web platform API for detection of faces, text and barcodes using the hardware accelerated capabilities of the client device. Eng owner: reillyg Product owner: reillyg (PM? What PM?) Mocks: N/A (no new UI, only an API) Specification: https://wicg.github.io/shape-detection-api Design doc (send to chrome-design-docs@): http://tinyurl.com/shape-detection-in-chromium Privacy survey: crbug.com/738447 Test survey: See comment #23
,
Jun 28 2017
**Bulk edit** We are now past M-61 FF. Since you still haven't transitioned to a review state, we are assuming that you are not planning to launch in M-61. As such, we're updating the MStone to M-62. If you still intend to launch in M-61, feel free to change this back, but *please begin your xfnl reviews now*! Thanks for your help with ensuring a smooth process.
,
Jun 29 2017
Requesting reviews, aiming to still land in 61
,
Jun 30 2017
,
Jun 30 2017
Adding privacy review bug issue 738447 as a label to the launch bug issue 728474. This is an automated message.
,
Jul 5 2017
Just realized there was an M-62 label added that we didn't update to reflect we're still targeting M-61, so updating that now.
,
Jul 5 2017
Assigning to robert for security review
,
Jul 6 2017
,
Jul 28 2017
Can you please fill out the test survey?
,
Jul 28 2017
I don't know why I'm the owner of this issue.
,
Aug 10 2017
,
Aug 11 2017
Reminder to please fill out the test survey, https://goto.google.com/chrome-test-questions.
,
Aug 17 2017
,
Aug 22 2017
Checking whether the feature us ready for testing? If yes, please fill out the test survey, https://goto.google.com/chrome-test-questions.
,
Aug 31 2017
This feature is targeted for 61-Stable and M61 is going to stable next week. Legal,Security,Test bits (test survey missing) are not yet flipped. owencm@ if this is still on target for M61, please make sure that cross functional bits are flipped latest by Friday (09/01).
,
Sep 5 2017
What's the status of this feature? Test survey is not filled out and also it's shown to be blocking on 756590 which means we're lacking automated test coverage. M-61 stable for Android is today, we'll likely have to postpone this.
,
Sep 5 2017
Note: this is *not* launching in 61 and accidentally didn't have the label removed until now. Removing mstone labels while we work on it from the eng side, will re-add mstone labels and mark as review requested at the appropriate time.
,
Mar 7 2018
This launch bug is marked for milestone "61" and not been modified in the last 90 days. The current stable milestone is "65". Please review and close it if applicable. If this issue should remain open, remove the "Hotlist-LaunchArchive" label. If no action is taken, it will be archived in 30 days. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Apr 9 2018
This bug appears to be inactive and it has been over 30 days since the "Hotlist-LaunchArchive" label was applied. Hence, archiving this issue. Please reopen and remove the "Hotlist-LaunchArchive" label if it is still valid. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Aug 16
I am planning to launch an Origin Trial for this feature in Chrome 70. Re-opening this launch review issue.
,
Aug 16
,
Aug 16
,
Aug 16
1. Does the feature need manual testing? No. 2. Feature implementation status a. What is currently working and not working/known issues? The feature is currently implemented and working on Android, macOS and Windows. These are the platforms on which hardware acceleration is available. The API will be exposed to web content on other platforms but will return an error. b. Is all the code landed in canary and ready for testing? Yes. 3. Link to Design Doc: TODO 4. Link to demos (if any): https://paul.kinlan.me/face-detection/ https://paul.kinlan.me/barcode-detection/ https://paul.kinlan.me/detecting-text-in-an-image/ 5. Link to Test Plan: N/A 6. Are there flags or setup are needed to enable this feature? Is it on by default? It is available behind the --experimental-web-platform-features flag. 7. Are there any special points of interest from a testing perspective? No. 8. Is the feature the same across all platforms? Availability of the feature depends on platform support for hardware accelerated image recognition. a. Is the user flow same across all platforms? (Answer if applicable to both Desktop and Mobile) Yes. b. Is the code flow/logic different on any specific platforms (i.e code branching or logic duplication) Each platform has its own implementation of the integration between the web API and the platform-specific hardware accelerated image recognition API. 9. What's automated (Unit tests, Integration tests, API tests, Browser tests, E2E tests)? Please provide links to automated tests. This feature is tested by unit tests in //services/shape_detection and layout tests in //third_party/WebKit/LayoutTests/shapedetection. 10. What’s not automated that needs manual testing? Nothing. 11. What is the crbug label(s) and/or component(s) for the feature? The crbug component is Blink>ImageCapture. 12. Are there any other team members or mailing lists that should be notified if issues are found? Contacts are reillyg@chromium.org and mcasas@chromium.org. 13. Rollout plan: a. Will this roll out on by default for 100% or will it be rolled out incrementally behind a finch experiment? Is it behind a finch flag even if it’s on by default? This will roll out initially via an Origin Trial, enabled for sites which request a token from us. When shipped as a stable web API it will roll out on by default. b. Is there a mechanism to turn off the feature if it breaks product functionality or performance? There is a Blink feature flag for this API however this cannot be toggled directly via Finch. c. How will you monitor the stability of the feature and/or experiment groups? Crash reports. 14. Confirm that there is no negative meaningful performance impact and how that was checked: There is a small negative performance impact associated with enabling any new web-exposed API due to the overhead of bindings initialization. Other than that this API is dormant unless used by web content.
,
Aug 16
,
Aug 16
No manual testing required ( #23) , flipping the testbit to NA.
,
Aug 22
,
Aug 23
I added a question in the design doc.
,
Sep 5
palmer@, I believe reillyg@ replied to your comments. let us know what else is needed? we would like to OT in M70 if possible.
,
Sep 5
Great, thanks!
,
Sep 5
Double-checking with go/newChromeFeature it looks like an Origin Trial launch is requires Launch-Leadership and not Launch-Exp-Leadership.
,
Sep 6
Does this work on Windows? The OS box is not checked there.
,
Sep 6
By M61 (c3) it did only work on Mac and Android, now it's also supported on Windows.
,
Sep 6
,
Sep 10
This still does not have legal approval. What's the status here?
,
Sep 10
,
Sep 10
Greetings, feature team! Your launch has been approved for the following milestones and channels: - M70+ for beta - M70+ for stable experimentation Note that this approval only indicates you have the proper cross-functional sign-off to launch to the specified milestone and channel; if merges are required, you must still complete the merge request process. Questions? Concerns? Read more (or report bugs) about feature automation at go/chrome-launch-automation or reach out to jojwang@.
,
Sep 24
,
Sep 24
Greetings, feature team! Your launch has been approved for the following milestones and channels: - M70+ for full stable roll-out Note that this approval only indicates you have the proper cross-functional sign-off to launch to the specified milestone and channel; if merges are required, you must still complete the merge request process. Questions? Concerns? Read more (or report bugs) about feature automation at go/chrome-launch-automation or reach out to jojwang@.
,
Sep 24
We want to sit on this for a bit. Thomas, can you come to launch review this week?
,
Sep 24
If you have specific questions to ask us during launch review can you add those here so we can gather information necessary to answer them?
,
Sep 24
I am traveling for most of this week but should be able to come to launch review if needed. Note that I haven't been with this API for its whole life and Reily definitely has more context here. Let me know if there is anything specific we should explore and I can explore with relevant folk.
,
Oct 18
,
Oct 25
,
Oct 26
,
Oct 29
,
Oct 30
What happens on Windows/Mac/Android if a GPU is blacklisted?
,
Nov 6
,
Nov 21
,
Dec 13
,
Dec 13
Per reillyg@, Origin Trial started in M-70 (hence, Target-70) Due to bugs and API feedback that will be rolled in, won't ship earlier than M-75.
,
Dec 18
|
|||||||||||||||||||||||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||||||||||||||||||||||
Comment 1 by emilyschechter@chromium.org
, Jun 20 2017