Regression : In md-settings, unnecessarily background page changes on clicking ‘Reset’ button.
Reported by
yfulgaon...@etouch.net,
Nov 9 2016
|
||
Issue descriptionChrome Version : 56.0.2914.0 (Official Build)a081fcfbc7471a6681142d49491b82f7b8402851-refs/heads/master@{#430837} 32/64-bit OS : Mac(10.10.5, 10.11.4), Windows (7,8,8.1,10), Linux (14.04 LTS) What steps will reproduce the problem? 1. Launch chrome and navigate to chrome://md-settings page. 2. Type “reset” in search box (search result is seen) and click on ‘Reset’ 3. Observe the background. Actual : Unnecessarily background page changes on clicking ‘Reset’ button. Expected : Background page should not change after clicking on ‘Reset’ button. This is a regression issue broken in ‘M-56’, below is the Manual Regression range and will soon update bisect info. Good build : 56.0.2907.0 Bad build : 56.0.2908.0
,
Nov 9 2016
This will be addressed by https://codereview.chromium.org/2469393004.
,
Nov 10 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/6da08c5ddf401d99c58878e00726875ce72691ec commit 6da08c5ddf401d99c58878e00726875ce72691ec Author: dpapad <dpapad@chromium.org> Date: Thu Nov 10 02:53:48 2016 MD Settings: Preserve search URL param in subpages. BUG= 661835 , 663708 CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation Review-Url: https://codereview.chromium.org/2469393004 Cr-Commit-Position: refs/heads/master@{#431152} [modify] https://crrev.com/6da08c5ddf401d99c58878e00726875ce72691ec/chrome/browser/resources/settings/route.js [modify] https://crrev.com/6da08c5ddf401d99c58878e00726875ce72691ec/chrome/browser/resources/settings/settings_menu/settings_menu.html [modify] https://crrev.com/6da08c5ddf401d99c58878e00726875ce72691ec/chrome/browser/resources/settings/settings_menu/settings_menu.js [modify] https://crrev.com/6da08c5ddf401d99c58878e00726875ce72691ec/chrome/browser/resources/settings/settings_ui/settings_ui.js [modify] https://crrev.com/6da08c5ddf401d99c58878e00726875ce72691ec/chrome/test/data/webui/settings/route_tests.js [modify] https://crrev.com/6da08c5ddf401d99c58878e00726875ce72691ec/chrome/test/data/webui/settings/settings_menu_test.js [modify] https://crrev.com/6da08c5ddf401d99c58878e00726875ce72691ec/chrome/test/data/webui/settings/settings_ui_browsertest.js
,
Nov 10 2016
|
||
►
Sign in to add a comment |
||
Comment 1 by msrchandra@chromium.org
, Nov 9 2016Owner: dpa...@chromium.org
Status: Assigned (was: Unconfirmed)