Regression: Browser crash is seen upon clicking on any link in chrome://history/syncedTabs page in emulated view. |
||||||
Issue descriptionVersion:54.0.2817.0 Dev OS: Ubuntu 14.04, Windows What steps will reproduce the problem? (1)Signin to chrome with valid credentials>>Navigate to chrome://history/syncedTabs page>> Open devtools and emulate>> Now click on any link and observe. Expected:Should be able to open link without any browser crash. Actual: Instead browser crash is seen upon clicking on link in chrome://history/syncedTabs page in emulated view. Crash ids: ea7102c200000000, 6008c2fe00000000 This is Regression issue broken in M-54. Will update bisect info soon. Attaching screen-cast for reference.
,
Aug 4 2016
Unable to do tool bisect as issue is seen only after signing into chrome which can't be done in chromium builds. Manual Bisect info: Good build:54.0.2811.0 Dev Bad build:54.0.2812.0 Dev CHANGELOG: https://chromium.googlesource.com/chromium/src/+log/54.0.2811.0..54.0.2812.0?pretty=fuller&n=10000 Suspecting https://codereview.chromium.org/2195593002 from manual changelog. @tsergeant: Please help in re-assigning if it is not related to your change.
,
Aug 4 2016
Users experienced this crash on the following builds: Linux Dev 54.0.2816.0 - 6.94 CPM, 2 reports, 2 clients (signature webui::GetDispositionFromClick) If this update was incorrect, please add "Fracas-Wrong" label to prevent future updates. - Go/Fracas
,
Aug 5 2016
,
Aug 9 2016
,
Aug 9 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/ec398f6d0d29522ae9b7cd07ac611e8e82e05e8e commit ec398f6d0d29522ae9b7cd07ac611e8e82e05e8e Author: tsergeant <tsergeant@chromium.org> Date: Tue Aug 09 06:27:13 2016 MD History: Fix crash when opening Synced Tab links with touch The event handler for synced tab clicks needs to know what modifier keys were pressed during the click, but Polymer tap events do not include this information from touch events. Instead, this changes the handler to listen to a plain 'click' event, which will always include this information. BUG= 634236 CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation Review-Url: https://codereview.chromium.org/2219753002 Cr-Commit-Position: refs/heads/master@{#410597} [modify] https://crrev.com/ec398f6d0d29522ae9b7cd07ac611e8e82e05e8e/chrome/browser/resources/md_history/browser_service.js [modify] https://crrev.com/ec398f6d0d29522ae9b7cd07ac611e8e82e05e8e/chrome/browser/resources/md_history/constants.js [modify] https://crrev.com/ec398f6d0d29522ae9b7cd07ac611e8e82e05e8e/chrome/browser/resources/md_history/synced_device_card.html [modify] https://crrev.com/ec398f6d0d29522ae9b7cd07ac611e8e82e05e8e/chrome/browser/resources/md_history/synced_device_card.js [modify] https://crrev.com/ec398f6d0d29522ae9b7cd07ac611e8e82e05e8e/chrome/test/data/webui/md_history/history_synced_tabs_test.js
,
Aug 9 2016
|
||||||
►
Sign in to add a comment |
||||||
Comment 1 by ajha@chromium.org
, Aug 4 2016Status: Untriaged (was: Unconfirmed)