Regression :Unwanted loading message is seen while refreshing the chrome://settings/siteData page
Reported by
pranjali...@etouch.net,
Oct 30 2017
|
|||||||||||||
Issue descriptionChrome Version:64.0.3253.0 (Official Build) 3163a676b23fd74bf3d1c5c13a1141d26d8dd6aa-refs/heads/master@{#512377}(32/64 bit) OS:Windows (7,8,10),Linux (14.04 LTS),Mac(10.12.6,10.13.1). Steps to reproduce: 1.Launch chrome and navigate to 'chrome://settings/siteData'. 2.click on 'Remove all' button to clear all the cookies , press F5 and observe. Actual Result:Unwanted loading message is seen while refreshing 'chrome://settings/siteData' page Expected Result:Unwanted Loading message should not be seen while refreshing 'chrome://settings/siteData' page This is Regression issue broken in 'M-64' and Using the per-revision bisect providing the bisect results, Good Build:64.0.3244.0 Bad Build:64.0.3245.0 You are probably looking for a change made after 509956 (known good), but no later than 509957 (first known bad). CHANGELOG URL: The script might not always return single CL as suspect as some perf builds might get missing due to failure. https://chromium.googlesource.com/chromium/src/+log/ff5b6944f36e0e4fcc62d8c65ce07b2b17809291..34eeae65c51a2e8c545daef878a74e1efc6c81ef Suspect:https://chromium.googlesource.com/chromium/src/+/34eeae65c51a2e8c545daef878a74e1efc6c81ef
,
Oct 30 2017
,
Oct 31 2017
I've emailed bettes@ to ask what should happen here. I'll change it when I hear what he'd like.
,
Nov 14 2017
bettes@ mentioned in the UX meeting today that a spinner is preferred over 'loading' text.
,
Nov 14 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/eca759013e8ff178096d8032ece30127d68fa84e commit eca759013e8ff178096d8032ece30127d68fa84e Author: Dave Schuyler <dschuyler@chromium.org> Date: Tue Nov 14 19:31:00 2017 [MD settings] replace 'loading' text with spinner This CL replaces the "Loading, please wait..." text in the cookies and local data page with a spinner. Bug: 779467 Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation Change-Id: Ibdb817adf6c623a520096016e6dd44cef13d8519 Reviewed-on: https://chromium-review.googlesource.com/767571 Reviewed-by: Demetrios Papadopoulos <dpapad@chromium.org> Commit-Queue: Dave Schuyler <dschuyler@chromium.org> Cr-Commit-Position: refs/heads/master@{#516386} [modify] https://crrev.com/eca759013e8ff178096d8032ece30127d68fa84e/chrome/app/settings_strings.grdp [modify] https://crrev.com/eca759013e8ff178096d8032ece30127d68fa84e/chrome/browser/resources/settings/site_settings/site_data.html [modify] https://crrev.com/eca759013e8ff178096d8032ece30127d68fa84e/chrome/browser/ui/webui/settings/md_settings_localized_strings_provider.cc
,
Nov 14 2017
,
Nov 15 2017
Rechecked the above issue on Windows (7,8,10),Linux (14.04LTS),Mac(10.12.6)OS with latest canary chrome version :64.0.3269.0 and the issue is not reproducible.Kindly refer the attached screen cast for reference.
|
|||||||||||||
►
Sign in to add a comment |
|||||||||||||
Comment 1 by pranjali...@etouch.net
, Oct 30 20171.3 MB
1.3 MB Download