Show notification when activating with no connection

ash/system/chromeos/network/tray_sms.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "ash/system/chromeos/network/tray_sms.h"
 
 #include "ash/ash_switches.h"
 #include "ash/shell.h"
 #include "ash/system/tray/system_tray.h"
 #include "ash/system/tray/system_tray_bubble.h"
@@ skipping 284 matching lines @@
   default_ = new SmsDefaultView(this);
   default_->SetVisible(!messages_.empty());
   return default_;
 }
 
 views::View* TraySms::CreateDetailedView(user::LoginStatus status) {
   CHECK(detailed_ == NULL);
   HideNotificationView();
   if (messages_.empty())
     return NULL;
+  HideNotificationView();

[jennyz, 2013/02/14 23:26:45]

Why do you need to HideNotificationView twice? 

[stevenjb, 2013/02/14 23:32:25]

Oops, didn't see that we already did that. Fixed.

   detailed_ = new SmsDetailedView(this);
   return detailed_;
 }
 
 views::View* TraySms::CreateNotificationView(user::LoginStatus status) {
   CHECK(notification_ == NULL);
+  if (detailed_)
+    return NULL;
   size_t index;
   std::string number, text;
   if (GetLatestMessage(&index, &number, &text))
     notification_ = new SmsNotificationView(this, index, number, text);
   return notification_;
 }
 
 void TraySms::DestroyDefaultView() {
   default_ = NULL;
 }
@@ skipping 51 matching lines @@
       if (GetLatestMessage(&index, &number, &text))
         notification_->Update(index, number, text);
     } else if (notify) {
       ShowNotificationView();
     }
   }
 }
 
 }  // namespace internal
 }  // namespace ash