Regression: Browser crash is seen after blocking 'MIDI' and reloading the page
Reported by
jshan...@etouch.net,
Feb 16 2017
|
||||||||
Issue descriptionChrome Version : 58.0.3013.3 (Official Build) f3229ac11997e2b5fe740be8738c2ac80ab9f4c3-refs/branch-heads/3013@{#6} 32/64 bit OS : Windows (7,8,10), Mac(10.11.6, 10.12.1, 10.12), Linux(14.04 LTS) Steps: 1. Launch Chrome and navigate to https://permission.site 2. Click on 'MIDI', click on 'Block' of permission bubble and reload the page 3. Observe Actual: Browser crash is seen after blocking 'MIDI' and reloading the page Expected: Browser should not crash This is regression issue, broken in ‘M 58’ and will soon update other info : Good build: 58.0.3008.0 Bad build: 58.0.3009.0 Crash ID 9213e928-76f6-4c1f-a118-0c178f0b8bb8 (Server ID: 62d2bd3580000000)
,
Feb 16 2017
Issue 693023 has been merged into this issue.
,
Feb 16 2017
,
Feb 16 2017
My change is on test runners, nothing that gets compiled into Chrome, so pretty sure it can't be related to crashes reproduced manually.
,
Feb 16 2017
Not exactly sure how this works, but the link above about the crash info points to: r449574 Web MIDI: add dynamic MidiManager instantiation support for Linux Which sounds like a lot more related to the stack above.
,
Feb 16 2017
Just to update this seems to be specific to MIDI permission and Issue doesn't repro with other type of notifications on https://permission.site page. Link to the list of the OS where the crashes are seen: ======================================================= https://crash.corp.google.com/browse?q=custom_data.ChromeCrashProto.magic_signature_1.name%3D%27midi%3A%3AMidiService%3A%3AEndSession%27&ignore_case=false&enable_rewrite=true&omit_field_name=&omit_field_value=&omit_field_opt=%3D Single digits of crashes instances are seen on Windows,Mac and Linux and Android latest M-58(58.0.3014.0) shows 17 instances from 6 clients.
,
Feb 16 2017
Not a Dev blocker since this is not a P0 functional use case. Thank you!
,
Feb 16 2017
Users experienced this crash on the following builds: Mac Canary 58.0.3014.0 - 2.25 CPM, 3 reports, 3 clients (signature midi::MidiService::EndSession) If this update was incorrect, please add "Fracas-Wrong" label to prevent future updates. - Go/Fracas
,
Feb 17 2017
A fix is already under code review. Note: this is not a regression, but one newly added assertion finds a potential problem that does not matter actually now, but will be a problem for the new design. |
||||||||
►
Sign in to add a comment |
||||||||
Comment 1 by brajkumar@chromium.org
, Feb 16 2017Labels: hasbisect-per-revision ReleaseBlock-Dev
Owner: perezju@chromium.org
Status: Assigned (was: Unconfirmed)