Use C++11 range-based loop in core/fetch

Source/core/fetch/ResourceLoadPriorityOptimizer.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:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 65 matching lines @@
     renderer->setHasPendingResourceUpdate(false);
 }
 
 void ResourceLoadPriorityOptimizer::updateAllImageResourcePriorities()
 {
     TRACE_EVENT0("blink", "ResourceLoadPriorityOptimizer::updateAllImageResourcePriorities");
 
     m_imageResources.clear();
 
     Vector<RenderObject*> objectsToRemove;
-    for (RenderObjectSet::iterator it = m_objects.begin(); it != m_objects.end(); ++it) {
-        RenderObject* obj = *it;
-        if (!obj->updateImageLoadingPriorities()) {
-            objectsToRemove.append(obj);
+    for (const auto& renderObject : m_objects) {
+        if (!renderObject->updateImageLoadingPriorities()) {

[Mike West, 2014/10/17 10:38:49]

Nit: Drop the {} for one-liners.

[riju_, 2014/10/17 18:36:48]

Done.

+            objectsToRemove.append(renderObject);
         }
     }
     m_objects.removeAll(objectsToRemove);
 
     updateImageResourcesWithLoadPriority();
 }
 
 void ResourceLoadPriorityOptimizer::updateImageResourcesWithLoadPriority()
 {
-    for (ImageResourceMap::iterator it = m_imageResources.begin(); it != m_imageResources.end(); ++it) {
-        ResourceLoadPriority priority = it->value->status == Visible ?
+    for (const auto& resource : m_imageResources) {
+        ResourceLoadPriority priority = resource.value->status == Visible ?
             ResourceLoadPriorityLow : ResourceLoadPriorityVeryLow;
 
-        if (priority != it->value->imageResource->resourceRequest().priority()) {
-            it->value->imageResource->mutableResourceRequest().setPriority(priority, it->value->screenArea);
-            it->value->imageResource->didChangePriority(priority, it->value->screenArea);
+        if (priority != resource.value->imageResource->resourceRequest().priority()) {
+            resource.value->imageResource->mutableResourceRequest().setPriority(priority, resource.value->screenArea);
+            resource.value->imageResource->didChangePriority(priority, resource.value->screenArea);
         }
     }
     m_imageResources.clear();
 }
 
 void ResourceLoadPriorityOptimizer::notifyImageResourceVisibility(ImageResource* img, VisibilityStatus status, const LayoutRect& screenRect)
 {
     if (!img || img->isLoaded())
         return;
 
     int screenArea = 0;
     if (!screenRect.isEmpty() && status == Visible)
         screenArea += static_cast<uint32_t>(screenRect.width() * screenRect.height());
 
     ImageResourceMap::AddResult result = m_imageResources.add(img->identifier(), adoptPtr(new ResourceAndVisibility(img, status, screenArea)));
     if (!result.isNewEntry && status == Visible) {
         result.storedValue->value->status = Visible;
         result.storedValue->value->screenArea += screenArea;
     }
 }
 
 }