Send notification to users upon receiving sms messages

ash/common/system/chromeos/screen_security/screen_tray_item.cc

Index: ash/common/system/chromeos/screen_security/screen_tray_item.cc
diff --git a/ash/common/system/chromeos/screen_security/screen_tray_item.cc b/ash/common/system/chromeos/screen_security/screen_tray_item.cc
index 4f5f73cb9ce12d2738199a24c485b359ad45ebef..393b446d2176aa228f832ec923425030e1926188 100644
--- a/ash/common/system/chromeos/screen_security/screen_tray_item.cc
+++ b/ash/common/system/chromeos/screen_security/screen_tray_item.cc
@@ -106,8 +106,6 @@ void ScreenStatusView::CreateItems() {
 }
 
 void ScreenStatusView::UpdateFromScreenTrayItem() {
-  // Hide the notification bubble when the ash tray bubble opens.
-  screen_tray_item_->HideNotificationView();

[tdanderson, 2017/01/26 22:19:07]

I'm a bit confused about what this line of code was doing here in the first
place. Does removing it actually result in any user-visible change? As far as I
can tell, screen tray items don't have corresponding notification views.

[yiyix, 2017/02/02 20:43:56]

I am not sure what does this line do here. Hide NotificationView if there is
one? The only class that uses the NotificationView is SmsTray, as we stop
displaying sms messages in the NotificationView, NotificationView should not be
used anymore. Therefore, i think i can safely remove this line.

[tdanderson, 2017/02/07 00:15:45]

Agreed.

   SetVisible(screen_tray_item_->is_started());
 }