Use C++11 range-based loop for core/clipboard, core/dom and core/testing

Source/core/testing/Internals.cpp

 /*
  * Copyright (C) 2012 Google Inc. All rights reserved.
  * Copyright (C) 2013 Apple 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.
@@ skipping 706 matching lines @@
 
 unsigned Internals::activeMarkerCountForNode(Node* node)
 {
     ASSERT(node);
 
     // Only TextMatch markers can be active.
     DocumentMarker::MarkerType markerType = DocumentMarker::TextMatch;
     DocumentMarkerVector markers = node->document().markers().markersFor(node, markerType);
 
     unsigned activeMarkerCount = 0;
-    for (DocumentMarkerVector::iterator iter = markers.begin(); iter != markers.end(); ++iter) {
-        if ((*iter)->activeMatch())
+    for (auto& marker : markers) {

[Julien - ping for review, 2015/01/02 09:42:02]

const auto&

[zhaoze.zhou, 2015/01/05 15:17:57]

Done.

+        if (marker->activeMatch())
             activeMarkerCount++;
     }
 
     return activeMarkerCount;
 }
 
 DocumentMarker* Internals::markerAt(Node* node, const String& markerType, unsigned index, ExceptionState& exceptionState)
 {
     ASSERT(node);
     DocumentMarker::MarkerTypes markerTypes = 0;
@@ skipping 406 matching lines @@
 }
 
 static unsigned eventHandlerCount(Document& document, EventHandlerRegistry::EventHandlerClass handlerClass)
 {
     if (!document.frameHost())
         return 0;
     EventHandlerRegistry* registry = &document.frameHost()->eventHandlerRegistry();
     unsigned count = 0;
     const EventTargetSet* targets = registry->eventHandlerTargets(handlerClass);
     if (targets) {
-        for (EventTargetSet::const_iterator iter = targets->begin(); iter != targets->end(); ++iter)
-            count += iter->value;
+        for (const auto& target : *targets)
+            count += target.value;
     }
     return count;
 }
 
 unsigned Internals::wheelEventHandlerCount(Document* document)
 {
     ASSERT(document);
     return eventHandlerCount(*document, EventHandlerRegistry::WheelEvent);
 }
 
@@ skipping 508 matching lines @@
         return 0;
 
     return PrintContext::pageNumberForElement(element, FloatSize(pageWidth, pageHeight));
 }
 
 Vector<String> Internals::iconURLs(Document* document, int iconTypesMask) const
 {
     Vector<IconURL> iconURLs = document->iconURLs(iconTypesMask);
     Vector<String> array;
 
-    Vector<IconURL>::const_iterator iter(iconURLs.begin());
-    for (; iter != iconURLs.end(); ++iter)
-        array.append(iter->m_iconURL.string());
+    for (auto& iconURL : iconURLs)
+        array.append(iconURL.m_iconURL.string());
 
     return array;
 }
 
 Vector<String> Internals::shortcutIconURLs(Document* document) const
 {
     return iconURLs(document, Favicon);
 }
 
 Vector<String> Internals::allIconURLs(Document* document) const
@@ skipping 577 matching lines @@
     ASSERT(element);
     return ClientRect::create(element->boundsInRootViewSpace());
 }
 
 String Internals::serializeNavigationMarkup()
 {
     Vector<Document::TransitionElementData> elementData;
     frame()->document()->getTransitionElementData(elementData);
 
     StringBuilder markup;
-    Vector<Document::TransitionElementData>::iterator iter = elementData.begin();
-    for (; iter != elementData.end(); ++iter)
-        markup.append(iter->markup);
+    for (const auto& element : elementData)
+        markup.append(element.markup);
 
     return markup.toString();
 }
 
 Vector<String> Internals::getTransitionElementIds()
 {
     Vector<Document::TransitionElementData> elementData;
     frame()->document()->getTransitionElementData(elementData);
 
     Vector<String> ids;
@@ skipping 20 matching lines @@
     for (size_t i = 0; i < rects.size(); ++i)
         quads[i] = FloatRect(rects[i]);
     return ClientRectList::create(quads);
 }
 
 void Internals::hideAllTransitionElements()
 {
     Vector<Document::TransitionElementData> elementData;
     frame()->document()->getTransitionElementData(elementData);
 
-    Vector<Document::TransitionElementData>::iterator iter = elementData.begin();
-    for (; iter != elementData.end(); ++iter)
-        frame()->document()->hideTransitionElements(AtomicString(iter->selector));
+    for (const auto& element : elementData)
+        frame()->document()->hideTransitionElements(AtomicString(element.selector));
 }
 
 void Internals::showAllTransitionElements()
 {
     Vector<Document::TransitionElementData> elementData;
     frame()->document()->getTransitionElementData(elementData);
 
-    Vector<Document::TransitionElementData>::iterator iter = elementData.begin();
-    for (; iter != elementData.end(); ++iter)
-        frame()->document()->showTransitionElements(AtomicString(iter->selector));
+    for (const auto& element : elementData)
+        frame()->document()->showTransitionElements(AtomicString(element.selector));
 }
 
 void Internals::setExitTransitionStylesheetsEnabled(bool enabled)
 {
     frame()->document()->styleEngine()->setExitTransitionStylesheetsEnabled(enabled);
 }
 
 void Internals::forcePluginPlaceholder(HTMLElement* element, PassRefPtrWillBeRawPtr<DocumentFragment> fragment, ExceptionState& exceptionState)
 {
     if (!element->isPluginElement()) {
@@ skipping 16 matching lines @@
 {
     return new InternalsIterator;
 }
 
 void Internals::forceBlinkGCWithoutV8GC()
 {
     ThreadState::current()->scheduleGC(ThreadState::ForcedGC);
 }
 
 } // namespace blink