Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(2960)

Unified Diff: chrome/browser/resources/settings/settings_menu/settings_menu.js

Issue 2469393004: MD Settings: Preserve search URL param in subpages. (Closed)
Patch Set: Fix test. Created 4 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/resources/settings/settings_menu/settings_menu.js
diff --git a/chrome/browser/resources/settings/settings_menu/settings_menu.js b/chrome/browser/resources/settings/settings_menu/settings_menu.js
index fb672628d46bb92741caafc2dc76d93df4dba12f..d95c4f004deca2e46f2add77a046bb25f509af8c 100644
--- a/chrome/browser/resources/settings/settings_menu/settings_menu.js
+++ b/chrome/browser/resources/settings/settings_menu/settings_menu.js
@@ -75,7 +75,7 @@ Polymer({
openPage_: function(event) {
var route = settings.getRouteForPath(event.currentTarget.dataset.path);
assert(route, 'settings-menu has an an entry with an invalid path');
- settings.navigateTo(route);
+ settings.navigateTo(route, null, false);
dschuyler 2016/11/09 23:10:14 I find null, false unclear. Could we do settings.
dpapad 2016/11/09 23:19:16 Done.
},
/**

Powered by Google App Engine
This is Rietveld 408576698