Incognito icon shows up again when opening incognito browser.

chrome/browser/ui/views/frame/browser_non_client_frame_view_ash.cc

Index: chrome/browser/ui/views/frame/browser_non_client_frame_view_ash.cc
diff --git a/chrome/browser/ui/views/frame/browser_non_client_frame_view_ash.cc b/chrome/browser/ui/views/frame/browser_non_client_frame_view_ash.cc
index 4ce5e282dfc6b61e3638a5165901d0a7592218a1..f0c6877a92ff891f58a3e843998a2fac8a5167af 100644
--- a/chrome/browser/ui/views/frame/browser_non_client_frame_view_ash.cc
+++ b/chrome/browser/ui/views/frame/browser_non_client_frame_view_ash.cc
@@ -397,9 +397,11 @@ gfx::ImageSkia BrowserNonClientFrameViewAsh::GetFaviconForTabIconView() {
 // BrowserNonClientFrameView:
 void BrowserNonClientFrameViewAsh::UpdateProfileIcons() {
   Browser* browser = browser_view()->browser();
-  if ((browser->is_type_tabbed() || browser->is_app()) &&
+  const Profile* profile = browser_view()->browser()->profile();
+  bool is_incognito = profile->GetProfileType() == Profile::INCOGNITO_PROFILE;
+  if (((browser->is_type_tabbed() || browser->is_app()) &&
       chrome::MultiUserWindowManager::ShouldShowAvatar(
-          browser_view()->GetNativeWindow())) {
+          browser_view()->GetNativeWindow())) || is_incognito) {

[jdufault, 2016/06/09 15:54:40]

The (((a || b) && c) || d) structure is hard to parse and takes some time to
understand. Can you pull the ((a || b) && c) bit into a helper variable?

[sammiequon, 2016/06/10 04:16:04]

Done.

     UpdateProfileIndicatorIcon();
   }
 }