[serializer] small fixes for blink snapshot.

src/snapshot/serializer-common.cc

 // Copyright 2016 the V8 project 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 "src/snapshot/serializer-common.h"
 
 #include "src/external-reference-table.h"
 #include "src/ic/stub-cache.h"
 #include "src/list-inl.h"
 
 #if defined(DEBUG) && defined(V8_OS_LINUX) && !defined(V8_OS_ANDROID)
 #define SYMBOLIZE_FUNCTION
 #include <execinfo.h>
 #endif  // DEBUG && V8_OS_LINUX && !V8_OS_ANDROID
 
 namespace v8 {
 namespace internal {
 
 ExternalReferenceEncoder::ExternalReferenceEncoder(Isolate* isolate) {
   map_ = isolate->external_reference_map();
-  if (map_ != NULL) return;
-  map_ = new base::HashMap();
+  if (map_ != nullptr) return;
+  map_ = new AddressToIndexHashMap();
   ExternalReferenceTable* table = ExternalReferenceTable::instance(isolate);
-  for (int i = 0; i < table->size(); ++i) {
+  for (uint32_t i = 0; i < table->size(); ++i) {
     Address addr = table->address(i);
-    if (addr == ExternalReferenceTable::NotAvailable()) continue;
     // We expect no duplicate external references entries in the table.
     // AccessorRefTable getter may have duplicates, indicated by an empty string
     // as name.
-    DCHECK(table->name(i)[0] == '\0' ||
-           map_->Lookup(addr, Hash(addr)) == nullptr);
-    map_->LookupOrInsert(addr, Hash(addr))->value = reinterpret_cast<void*>(i);
+    DCHECK(table->name(i)[0] == '\0' || !map_->Has(addr));
+    map_->Set(addr, i);
+    DCHECK(map_->Has(addr));
   }
   isolate->set_external_reference_map(map_);
 }
 
 uint32_t ExternalReferenceEncoder::Encode(Address address) const {
-  DCHECK_NOT_NULL(address);
-  base::HashMap::Entry* entry =
-      const_cast<base::HashMap*>(map_)->Lookup(address, Hash(address));
-  if (entry == nullptr) {
+  if (!map_->Has(address)) {
     void* function_addr = address;
     v8::base::OS::PrintError("Unknown external reference %p.\n", function_addr);
 #ifdef SYMBOLIZE_FUNCTION
     v8::base::OS::PrintError("%s\n", backtrace_symbols(&function_addr, 1)[0]);
 #endif  // SYMBOLIZE_FUNCTION
     v8::base::OS::Abort();
   }
-  return static_cast<uint32_t>(reinterpret_cast<intptr_t>(entry->value));
+  return map_->Get(address);
 }
 
 const char* ExternalReferenceEncoder::NameOfAddress(Isolate* isolate,
                                                     Address address) const {
-  base::HashMap::Entry* entry =
-      const_cast<base::HashMap*>(map_)->Lookup(address, Hash(address));
-  if (entry == NULL) return "<unknown>";
-  uint32_t i = static_cast<uint32_t>(reinterpret_cast<intptr_t>(entry->value));
-  return ExternalReferenceTable::instance(isolate)->name(i);
+  if (!map_->Has(address)) return "<unknown>";
+  return ExternalReferenceTable::instance(isolate)->name(map_->Get(address));
 }
 
 void SerializedData::AllocateData(int size) {
   DCHECK(!owns_data_);
   data_ = NewArray<byte>(size);
   size_ = size;
   owns_data_ = true;
   DCHECK(IsAligned(reinterpret_cast<intptr_t>(data_), kPointerAlignment));
 }
 
@@ skipping 14 matching lines @@
     if (cache->at(i)->IsUndefined(isolate)) break;
   }
 }
 
 bool SerializerDeserializer::CanBeDeferred(HeapObject* o) {
   return !o->IsString() && !o->IsScript();
 }
 
 }  // namespace internal
 }  // namespace v8