Chromium Code Reviews| Index: Source/core/fetch/ResourceLoadPriorityOptimizer.cpp |
| diff --git a/Source/core/fetch/ResourceLoadPriorityOptimizer.cpp b/Source/core/fetch/ResourceLoadPriorityOptimizer.cpp |
| index 06bf4db45e6d619512dd6d3fa606540fa384557c..03b31dab8853188a25ba1a3b9ae19e2b42d3cc13 100644 |
| --- a/Source/core/fetch/ResourceLoadPriorityOptimizer.cpp |
| +++ b/Source/core/fetch/ResourceLoadPriorityOptimizer.cpp |
| @@ -83,10 +83,9 @@ void 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); |
| @@ -96,13 +95,13 @@ void ResourceLoadPriorityOptimizer::updateAllImageResourcePriorities() |
| 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(); |