| OLD | NEW |
| 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. | 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. |
| 2 // Use of this source code is governed by a BSD-style license that can be | 2 // Use of this source code is governed by a BSD-style license that can be |
| 3 // found in the LICENSE file. | 3 // found in the LICENSE file. |
| 4 | 4 |
| 5 #include "chrome/browser/status_icons/status_tray.h" | 5 #include "chrome/browser/status_icons/status_tray.h" |
| 6 | 6 |
| 7 #include <algorithm> | |
| 8 | |
| 9 #include "chrome/browser/status_icons/status_icon.h" | 7 #include "chrome/browser/status_icons/status_icon.h" |
| 10 | 8 |
| 11 StatusTray::~StatusTray() { | 9 StatusTray::~StatusTray() { |
| 12 } | 10 } |
| 13 | 11 |
| 14 StatusIcon* StatusTray::CreateStatusIcon(StatusIconType type, | 12 StatusIcon* StatusTray::CreateStatusIcon(StatusIconType type, |
| 15 const gfx::ImageSkia& image, | 13 const gfx::ImageSkia& image, |
| 16 const base::string16& tool_tip) { | 14 const base::string16& tool_tip) { |
| 17 StatusIcon* icon = CreatePlatformStatusIcon(type, image, tool_tip); | 15 auto icon = CreatePlatformStatusIcon(type, image, tool_tip); |
| 18 if (icon) | 16 if (!icon) |
| 19 status_icons_.push_back(icon); | 17 return nullptr; |
| 20 return icon; | 18 |
| 19 status_icons_.push_back(std::move(icon)); |
| 20 return status_icons_.back().get(); |
| 21 } | 21 } |
| 22 | 22 |
| 23 void StatusTray::RemoveStatusIcon(StatusIcon* icon) { | 23 void StatusTray::RemoveStatusIcon(StatusIcon* icon) { |
| 24 StatusIcons::iterator i( | 24 for (auto iter = status_icons_.begin(); iter != status_icons_.end(); ++iter) { |
| 25 std::find(status_icons_.begin(), status_icons_.end(), icon)); | 25 if (iter->get() == icon) { |
| 26 | 26 status_icons_.erase(iter); |
| 27 if (i == status_icons_.end()) { | 27 return; |
| 28 NOTREACHED(); | 28 } |
| 29 return; | |
| 30 } | 29 } |
| 31 | 30 NOTREACHED(); |
| 32 status_icons_.erase(i); | |
| 33 } | 31 } |
| 34 | 32 |
| 35 StatusTray::StatusTray() { | 33 StatusTray::StatusTray() { |
| 36 } | 34 } |
| OLD | NEW |