Revert of Skip nullptr after copyToVector() from HeapHashSet<WeakMember>

third_party/WebKit/Source/platform/fonts/FontCache.cpp

 /*
  * Copyright (C) 2006, 2008 Apple Inc. All rights reserved.
  * Copyright (C) 2007 Nicholas Shanks <webkit@nickshanks.com>
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  *
  * 1.  Redistributions of source code must retain the above copyright
  *     notice, this list of conditions and the following disclaimer.
@@ skipping 433 matching lines @@
 
   if (gFontPlatformDataCache) {
     delete gFontPlatformDataCache;
     gFontPlatformDataCache = new FontPlatformDataCache;
   }
 
   gGeneration++;
 
   HeapVector<Member<FontCacheClient>> clients;
   copyToVector(fontCacheClients(), clients);
-  for (const auto& client : clients) {
-    // This should not be nullptr, but to see if checking nullptr can suppress
-    // crashes. crbug.com/581698
-    if (client)
-      client->fontCacheInvalidated();
-  }
+  for (const auto& client : clients)
+    client->fontCacheInvalidated();
 
   purge(ForcePurge);
 }
 
 void FontCache::crashWithFontInfo(const FontDescription* fontDescription) {
   FontCache* fontCache = FontCache::fontCache();
   SkFontMgr* fontMgr = nullptr;
   int numFamilies = std::numeric_limits<int>::min();
   if (fontCache) {
     fontMgr = fontCache->m_fontManager.get();
@@ skipping 36 matching lines @@
   for (iter = gFallbackListShaperCache->begin();
        iter != gFallbackListShaperCache->end(); ++iter) {
     shapeResultCacheSize += iter->value->byteSize();
   }
   dump->AddScalar("size", "bytes", shapeResultCacheSize);
   memoryDump->AddSuballocation(dump->guid(),
                                WTF::Partitions::kAllocatedObjectPoolName);
 }
 
 }  // namespace blink