(Concept patch) Simplify WTF::HashTable::add() return value for size and performance

Source/core/dom/custom/CustomElementScheduler.cpp

 /*
  * Copyright (C) 2013 Google Inc. All rights reserved.
  *
  * 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.
  * 2. Redistributions in binary form must reproduce the above copyright
@@ skipping 89 matching lines @@
 
 CustomElementScheduler& CustomElementScheduler::instance()
 {
     DEFINE_STATIC_LOCAL(CustomElementScheduler, instance, ());
     return instance;
 }
 
 CustomElementCallbackQueue* CustomElementScheduler::ensureCallbackQueue(PassRefPtr<Element> element)
 {
     Element* key = element.get();
     ElementCallbackQueueMap::iterator it = m_elementCallbackQueueMap.find(key);

[Erik Corry, 2014/02/04 14:27:15]

Also here with the find that insists on returning an iterator.

     if (it == m_elementCallbackQueueMap.end())
-        it = m_elementCallbackQueueMap.add(key, CustomElementCallbackQueue::create(element)).iterator;
+        return m_elementCallbackQueueMap.add(key, CustomElementCallbackQueue::create(element)).iterator->value.get();
     return it->value.get();
 }
 
 void CustomElementScheduler::clearElementCallbackQueueMap()
 {
     ElementCallbackQueueMap emptyMap;
     instance().m_elementCallbackQueueMap.swap(emptyMap);
 }
 
 // Finds or creates the callback queue for element.
@@ skipping 37 matching lines @@
         return false;
 
     CustomElementMicrotaskStep::Result result = m_baseMicrotaskQueue.dispatch();
     if (m_baseMicrotaskQueue.isEmpty())
         clearElementCallbackQueueMap();
 
     return result & CustomElementMicrotaskStep::DidWork;
 }
 
 } // namespace WebCore