[Privet] Do not show notifications from guest and default profiles in ChromeOS

chrome/browser/local_discovery/privet_notifications.cc

Index: chrome/browser/local_discovery/privet_notifications.cc
diff --git a/chrome/browser/local_discovery/privet_notifications.cc b/chrome/browser/local_discovery/privet_notifications.cc
index 5d7ed0aa7de72d3928362c6f72d7d1dfbfc5b815..cf3b5ec11e695efcfad1d4bcebf69efb2efe695c 100644
--- a/chrome/browser/local_discovery/privet_notifications.cc
+++ b/chrome/browser/local_discovery/privet_notifications.cc
@@ -19,6 +19,8 @@
 #include "chrome/browser/notifications/notification.h"
 #include "chrome/browser/notifications/notification_ui_manager.h"
 #include "chrome/browser/profiles/profile.h"
+#include "chrome/browser/signin/signin_manager_base.h"
+#include "chrome/browser/signin/signin_manager_factory.h"
 #include "chrome/browser/ui/browser.h"
 #include "chrome/browser/ui/browser_finder.h"
 #include "chrome/browser/ui/browser_window.h"
@@ -284,6 +286,15 @@ void PrivetNotificationService::PrivetRemoveNotification() {
 }
 
 void PrivetNotificationService::Start() {
+#if defined(CHROMEOS)
+  SigninManagerBase* signin_manager =
+      SigninManagerFactory::GetForProfileIfExists(
+          Profile::FromBrowserContext(profile_));
+
+  if (!signin_manager || signin_manager->GetAuthenticatedUsername().empty())

[Vitaly Buka (NO REVIEWS), 2014/03/04 22:47:33]

I guess that's not enough.
service keeps running after sign off  of user, so it will show notification for
the next guest after that

[Noam Samuel, 2014/03/04 22:58:49]

Have we seen this behavior? I'd suspect notifications should only show on the
desktop of the profile's user, with the default profile being an exception to
that rule.

+    return;
+#endif
+
   enable_privet_notification_member_.Init(
       prefs::kLocalDiscoveryNotificationsEnabled,
       Profile::FromBrowserContext(profile_)->GetPrefs(),