Commit 9630e67a authored by Kush Sinha's avatar Kush Sinha Committed by Commit Bot

Add kAccountManagerSubPage to Chrome OS settings pages

Account Manager is an "OS level" setting, not a "browser level" setting
and was somehow not in the list of OS level settings pages, causing the
crash linked in the attached bug.

Fix this by adding it to the list of OS level settings pages.

Bug: 962438
Change-Id: I46424cb8636b7600ad5d144561626fb6d83b378b
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1609778Reviewed-by: default avatarColin Blundell <blundell@chromium.org>
Commit-Queue: Kush Sinha <sinhak@chromium.org>
Cr-Commit-Position: refs/heads/master@{#659014}
parent 67f59be5
......@@ -371,6 +371,7 @@ const char kTetherSettingsSubPage[] = "networks?type=Tether";
bool IsOSSettingsSubPage(const std::string& sub_page) {
static const char* const kSubPages[] = {kAccessibilitySubPage,
kAccountManagerSubPage,
kAndroidAppsDetailsSubPage,
kAssistantSubPage,
kBluetoothSubPage,
......
......@@ -9,6 +9,7 @@
#define CHROME_COMMON_WEBUI_URL_CONSTANTS_H_
#include <stddef.h>
#include <string>
#include "base/strings/string_piece_forward.h"
#include "build/build_config.h"
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment