Issue metadata
Sign in to add a comment
|
Regression: Window crash is seen on Hangout extension on clicking upgrade.
Reported by
dmascare...@etouch.net,
Sep 15 2016
|
||||||||||||||||||||||
Issue descriptionChrome Version: 55.0.2860.1 (Official Build) b10f394de7dc3578fa7982cb73b67b7d59609307-refs/branch-heads/2860@{#1} OS: Windows (7,8,10) What steps will reproduce the problem? 1. Launch chrome and Sign-in the browser with valid credential. 2. Click on extension icon such that window gets open and then click on 'Update' button. 3. Observe hangout window. Actual:Hangout extension window crashes after step 2.. Expected: Window should not crash. This is regression issue, broken in 'M 55' and will soon update the info.
,
Sep 15 2016
Could have this be caused by r401390? It's in the narrow bissect range. - // Without --enable-panels, only support Hangouts. +// Panels are an Eraser target. See http://crbug.com/571511 for details. +// TODO(dimich): remove the rest of Panels code (except ChromeOS) once +// erasure is stable. +#if defined(OS_CHROMEOS) + // Without --enable-panels, only support Hangouts on ChromeOS. for (const char* id : extension_misc::kHangoutsExtensionIds) { if (extension_id == id) return true; } +#endif // OS_CHROMEOS
,
Sep 15 2016
I believe the bisect is incorrect. The range shows the CL I did that disables Panels (windows glued to the bottom of the screen for Hangouts). It converts them into regular floating windows. The captured video shows the hangout window glued to the bottom of the screen. It can't be with the specified bisect range. I would suggest getting a better reproducible steps as well. The original bug says it's visible in M55, the bisect points to M53. Also, it would be good to know how to get a version of the Hangouts extension that shows that "Upgrade" button because mine doesn't. I can't reproduce the issue locally on M55 build on Windows or Linux. Is there crash reports associated with this perhaps?
,
Sep 15 2016
See bug 571511 for more info on Panels removal and Hangout extension update.
,
Oct 12 2016
dmascarenhas@, can you please update per c#3? Thank you!
,
Oct 13 2016
With response to comment #3: Above is not reproducible as 'Update' button is not visible on the app window using Latest Canary Version:56.0.2889.0 (Official Build) 1e457676004f8af16e0086280b73d87c1e5efba7-refs/heads/master@{#424926}
,
Oct 24 2016
Resolving Won't Fix since several things happened since then: - the Upgrade button apparently disappeared form the extension - the Panels were completely removed from Chrome in M54 - we don't have reproducible case anymore. Please reactivate with reproducible case attached if needed. |
|||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||
Comment 1 by dmascare...@etouch.net
, Sep 15 2016Labels: HasTestcase hasbisect
Owner: fdoray@chromium.org
Status: Assigned (was: Unconfirmed)
597 KB
597 KB View Download
1.4 MB
1.4 MB View Download