Regression:[NTP]'Can't Create shortcut' confirmation message is not displayed on creating duplicate shortcut
Reported by
vineetha...@etouch.net,
Sep 27
|
|||||
Issue descriptionChrome Version: 70.0.3538.37 (Official Build) Revision 53f287561c37d0074842843cca3b4ac3560a4459-refs/branch-heads/3538@{#693}(32/64-Bit) OS: Windows(7,8,8.1,10), Mac (10.12.6, 10.13.1, 10.13.6, 10.14) and Linux (14.04 LTS) Steps to reproduce: 1.Launch chrome and navigate to chrome://restart and relaunch browser. 2.Navigate to NTP and add a shortcut by clicking on 'Add Shortcut' button. 3.Now add another shortcut with the same URL used for the previously added shortcut and observe the confirmation message. Actual Result : 'Can't Create shortcut' confirmation message is not displayed. Expected Result: 'Can't Create shortcut' confirmation message should be displayed on creating a new shortcut using existing URL. This is a regression issue, broken in 'M-70', and below is the bisect info: Good Build:70.0.3537.0 (Revision: 587303) Bad Build :70.0.3538.0 (Revision: 587811) Change-Log URL: https://chromium.googlesource.com/chromium/src/+log/70.0.3537.0..70.0.3538.0?pretty=fuller&n=10000 Suspecting:r587313? @kristipark: Could you please check whether this is caused with respect to your change, if not please help us in assigning it to the right owner. Note: 1. Providing suspect through 'Change-Log' because unable to perform bisect using 'per-revision' and 'chromium bisect' script. 2. Tried performing 'per revision' bisect on multiple Windows, Mac and Linux machines but unable to perform the same since getting following error: (a) Error message on Windows and Linux OS : Unable to find locale data files (b) Error message on Mac OS:[Errno 2] No such file or directory error message 3. Unable to perform 'chromium bisect' script as issue is not reproducible on chromium builds. Thank You!
,
Sep 27
Tested with 70.0.3538.35, and the add shortcut error message is working as expected in local NTP. Support for remote NTP is still rolling out, however. Is the Actual screencast using remote NTP?
,
Sep 28
Update w.r.t comment #2, Yes, the attached Actual screencast is using remote NTP.
,
Oct 1
Friendly ping! Could you please provide any update on this issue as it has been marked as a stable blocker. Thank You!
,
Oct 1
[bulk edit] - This issue is marked as a stable blocker for M70. We are two weeks away from M70 Stable. Please take a look urgently!
,
Oct 1
Apologies for the delay, the fix has completed rolling out. Please go ahead and verify. Thanks!
,
Oct 1
[Auto-generated comment by a script] We noticed that this issue is targeted for M-70; it appears the fix may have landed after branch point, meaning a merge might be required. Please confirm if a merge is required here - if so add Merge-Request-70 label, otherwise remove Merge-TBD label. Thanks.
,
Oct 1
,
Oct 16
|
|||||
►
Sign in to add a comment |
|||||
Comment 1 by pnangunoori@chromium.org
, Sep 27