Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(171)

Unified Diff: sky/engine/tonic/dart_library_loader.h

Issue 1241583003: Remove all remaining HashMap and HashSet usages except dart_string_cache (Closed) Base URL: git@github.com:domokit/mojo.git@master
Patch Set: Created 5 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « sky/engine/tonic/dart_dependency_catcher.cc ('k') | sky/engine/tonic/dart_library_loader.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: sky/engine/tonic/dart_library_loader.h
diff --git a/sky/engine/tonic/dart_library_loader.h b/sky/engine/tonic/dart_library_loader.h
index 609734070226dbd89bdfce39436ece9c8eae70be..3b50529a1c0533622de717387900d5cbb45da77d 100644
--- a/sky/engine/tonic/dart_library_loader.h
+++ b/sky/engine/tonic/dart_library_loader.h
@@ -5,15 +5,16 @@
#ifndef SKY_ENGINE_TONIC_DART_LIBRARY_LOADER_H_
#define SKY_ENGINE_TONIC_DART_LIBRARY_LOADER_H_
+#include <memory>
#include <string>
#include <unordered_map>
+#include <unordered_set>
#include <vector>
#include "base/callback_forward.h"
#include "base/macros.h"
#include "base/memory/weak_ptr.h"
#include "dart/runtime/include/dart_api.h"
-#include "sky/engine/wtf/HashSet.h"
#include "sky/engine/wtf/OwnPtr.h"
namespace blink {
@@ -38,8 +39,9 @@ class DartLibraryLoader {
void LoadLibrary(const std::string& name);
- void WaitForDependencies(const HashSet<DartDependency*>& dependencies,
- const base::Closure& callback);
+ void WaitForDependencies(
+ const std::unordered_set<DartDependency*>& dependencies,
+ const base::Closure& callback);
void set_dependency_catcher(DartDependencyCatcher* dependency_catcher) {
DCHECK(!dependency_catcher_ || !dependency_catcher);
@@ -72,8 +74,8 @@ class DartLibraryLoader {
DartState* dart_state_;
DartLibraryProvider* library_provider_;
std::unordered_map<std::string, Job*> pending_libraries_;
- HashSet<OwnPtr<Job>> jobs_;
- HashSet<OwnPtr<DependencyWatcher>> dependency_watchers_;
+ std::unordered_set<std::unique_ptr<Job>> jobs_;
+ std::unordered_set<std::unique_ptr<DependencyWatcher>> dependency_watchers_;
DartDependencyCatcher* dependency_catcher_;
DISALLOW_COPY_AND_ASSIGN(DartLibraryLoader);
« no previous file with comments | « sky/engine/tonic/dart_dependency_catcher.cc ('k') | sky/engine/tonic/dart_library_loader.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698