Remove ScopedVector in //components/bubble.

components/bubble/bubble_manager.cc

 // Copyright 2015 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 "components/bubble/bubble_manager.h"
 
 #include <utility>
 #include <vector>
 
 #include "components/bubble/bubble_controller.h"
@@ skipping 49 matching lines @@
   CloseAllMatchingBubbles(nullptr, nullptr, reason);
 }
 
 void BubbleManager::UpdateAllBubbleAnchors() {
   DCHECK(thread_checker_.CalledOnValidThread());
   DCHECK_NE(manager_state_, ITERATING_BUBBLES);
 
   // Guard against bubbles being added or removed while iterating the bubbles.
   ManagerState original_state = manager_state_;
   manager_state_ = ITERATING_BUBBLES;
-  for (auto* controller : controllers_)
+  for (auto& controller : controllers_)
     controller->UpdateAnchorPosition();
   manager_state_ = original_state;
 }
 
 void BubbleManager::AddBubbleManagerObserver(BubbleManagerObserver* observer) {
   observers_.AddObserver(observer);
 }
 
 void BubbleManager::RemoveBubbleManagerObserver(
     BubbleManagerObserver* observer) {
@@ skipping 18 matching lines @@
     const content::RenderFrameHost* owner,
     BubbleCloseReason reason) {
   // Specifying both an owning frame and a particular bubble to close doesn't
   // make sense. If we have a frame, all bubbles owned by that frame need to
   // have the opportunity to close. If we want to close a specific bubble, then
   // it should get the close event regardless of which frame owns it. On the
   // other hand, OR'ing the conditions needs a special case in order to be able
   // to close all bubbles, so we disallow passing both until a need appears.
   DCHECK(!bubble || !owner);
 
-  ScopedVector<BubbleController> close_queue;
+  std::vector<std::unique_ptr<BubbleController>> close_queue;
 
   // Guard against bubbles being added or removed while iterating the bubbles.
   ManagerState original_state = manager_state_;
   manager_state_ = ITERATING_BUBBLES;
   for (auto i = controllers_.begin(); i != controllers_.end();) {
-    if ((!bubble || bubble == *i) && (!owner || (*i)->OwningFrameIs(owner)) &&
-        (*i)->ShouldClose(reason)) {
-      close_queue.push_back(*i);
-      i = controllers_.weak_erase(i);
+    if ((!bubble || bubble == (*i).get()) &&
+        (!owner || (*i)->OwningFrameIs(owner)) && (*i)->ShouldClose(reason)) {
+      close_queue.push_back(std::move(*i));
+      i = controllers_.erase(i);
     } else {
       ++i;
     }
   }
   manager_state_ = original_state;
 
-  for (auto* controller : close_queue) {
+  for (auto& controller : close_queue) {
     controller->DoClose(reason);
 
     for (auto& observer : observers_)
       observer.OnBubbleClosed(controller->AsWeakPtr(), reason);
   }
 
   return !close_queue.empty();
 }