-
Brett Wilson authored
This is roughly half of the uses in //components. Some of the uses require deque and TODOs have been added with the reason why. #include <algorithm> is added to circular_deque.h for the calls to std::max. Previous uses apparently included this some other way. Some nontrivial updates were made to dependency_graph.cc to simplify the code. In trying to study what this code's requirements were to see if the usage could be replaced, I realized it could be simplified by using the (relatively new) base::Erase function. The constructors that take a range are also used, instead of the existing std::copy with a back_inserter iterator. Bug: 757232 TBR: erg@chromium.org Reland of https://chromium-review.googlesource.com/622887 which should have been fixed by https://chromium-review.googlesource.com/627017 Change-Id: Iab83d5ec264f65786f39985e8bdfcdaf1e4a2762 Reviewed-on: https://chromium-review.googlesource.com/627187 Reviewed-by: Brett Wilson <brettw@chromium.org> Reviewed-by: Elliot Glaysher <erg@chromium.org> Commit-Queue: Brett Wilson <brettw@chromium.org> Cr-Commit-Position: refs/heads/master@{#496716}
45d6f2d8