Commit 5fa4b36e authored by James Cook's avatar James Cook Committed by Chromium LUCI CQ

Change 2 NOTIMPLEMENTED to comments in BrowserNonClientFrameViewChromeOS

They print to the logs during normal usage. Changing to comments makes
the logs cleaner.

Bug: none
Change-Id: I7f75ccce2ae60a0abfb54e081d056ab2eda2fdfa
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2587255
Commit-Queue: James Cook <jamescook@chromium.org>
Commit-Queue: Antonio Gomes <tonikitoo@igalia.com>
Auto-Submit: James Cook <jamescook@chromium.org>
Reviewed-by: default avatarAntonio Gomes <tonikitoo@igalia.com>
Cr-Commit-Position: refs/heads/master@{#836182}
parent e28e6bd4
......@@ -694,12 +694,13 @@ bool BrowserNonClientFrameViewChromeOS::ShouldShowProfileIndicatorIcon() const {
return MultiUserWindowManagerHelper::ShouldShowAvatar(
browser_view()->GetNativeWindow());
#else
NOTIMPLEMENTED() << "Multi-signin support is deprecated in Lacros.";
// Multi-signin support is deprecated in Lacros.
return false;
#endif
}
void BrowserNonClientFrameViewChromeOS::UpdateProfileIcons() {
// Multi-signin support is deprecated in Lacros, so only do this for ash.
#if BUILDFLAG(IS_CHROMEOS_ASH)
View* root_view = frame()->GetRootView();
if (ShouldShowProfileIndicatorIcon()) {
......@@ -725,8 +726,6 @@ void BrowserNonClientFrameViewChromeOS::UpdateProfileIcons() {
if (root_view)
root_view->Layout();
}
#else
NOTIMPLEMENTED() << "Multi-signin support is deprecated in Lacros.";
#endif
}
......
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