[ES6 modules] Introduce ScriptModuleHash

third_party/WebKit/Source/bindings/core/v8/ScriptModule.h

 // Copyright 2017 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.
 
 #ifndef ScriptModule_h
 #define ScriptModule_h
 
 #include "bindings/core/v8/ScriptState.h"
 #include "bindings/core/v8/SharedPersistent.h"
 #include "core/CoreExport.h"
 #include "v8/include/v8.h"
 #include "wtf/Allocator.h"
 #include "wtf/text/WTFString.h"
 
 namespace blink {
 
 class CORE_EXPORT ScriptModule final {
   DISALLOW_NEW_EXCEPT_PLACEMENT_NEW();
 
  public:
   static ScriptModule compile(v8::Isolate*,
                               const String& source,
                               const String& fileName);
 
+  // TODO(kouhei): Remove copy ctor
   ScriptModule() {}
-  ScriptModule(const ScriptModule& module) : m_module(module.m_module) {}
+  ScriptModule(WTF::HashTableDeletedValueType)
+      : m_module(WTF::HashTableDeletedValue) {}
   ~ScriptModule();
 
   bool instantiate(ScriptState*);
   void evaluate(ScriptState*);
 
+  bool isHashTableDeletedValue() const {
+    return m_module.isHashTableDeletedValue();
+  }
+
+  bool operator==(const blink::ScriptModule& rhs) const {
+    if (m_module.isHashTableDeletedValue() &&

[yhirano, 2017/03/30 07:35:09]

isHashTableDeletedValue() && rhs.isHashTableDeletedValue() is easier to
understand.

[kouhei (in TOK), 2017/04/04 05:19:26]

Done.

+        rhs.m_module.isHashTableDeletedValue())
+      return true;
+

[yhirano, 2017/03/30 07:35:09]

if (isHashTableDeltedValue() || rhs.isHashTableDeletedValue())
  return false;

[kouhei (in TOK), 2017/04/04 05:19:26]

Done.

+    blink::SharedPersistent<v8::Module>* lhsp = m_module.get();
+    blink::SharedPersistent<v8::Module>* rhsp = rhs.m_module.get();
+    if (lhsp == rhsp)

[haraken, 2017/03/30 13:20:14]

Nit: Blink prefers a fully qualified name.

[kouhei (in TOK), 2017/04/04 05:19:26]

Done.

+      return true;
+    if (!lhsp || !rhsp)
+      return false;
+    return *lhsp == *rhsp;
+  }
+
   bool isNull() const { return m_module->isEmpty(); }
 
  private:
   ScriptModule(v8::Isolate*, v8::Local<v8::Module>);
 
   RefPtr<SharedPersistent<v8::Module>> m_module;
+  unsigned m_identityHash = 0;
+
+  friend struct ScriptModuleHash;
+};
+
+struct ScriptModuleHash {
+  STATIC_ONLY(ScriptModuleHash);
+
+ public:
+  static unsigned hash(const blink::ScriptModule& key) {
+    return static_cast<unsigned>(key.m_identityHash);

[yhirano, 2017/03/30 07:35:09]

This cast is not needed.

[kouhei (in TOK), 2017/04/04 05:19:26]

Done.

+  }
+
+  static bool equal(const blink::ScriptModule& a,
+                    const blink::ScriptModule& b) {
+    return a == b;
+  }
+
+  static constexpr bool safeToCompareToEmptyOrDeleted = true;
 };
 
 }  // namespace blink
 
+namespace WTF {
+
+template <>
+struct DefaultHash<blink::ScriptModule> {
+  using Hash = blink::ScriptModuleHash;
+};
+
+template <>
+struct HashTraits<blink::ScriptModule>
+    : public SimpleClassHashTraits<blink::ScriptModule> {};
+
+}  // namespace WTF
+
 #endif  // ScriptModule_h