Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1960)

Unified Diff: Source/core/timing/ResourceTimingInfo.h

Issue 663693005: Use C++11 features in core/timing (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Created 6 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: Source/core/timing/ResourceTimingInfo.h
diff --git a/Source/core/timing/ResourceTimingInfo.h b/Source/core/timing/ResourceTimingInfo.h
index 129f63d4dde503f3cec3286ea5b3f75da1bc3be9..d03244cebe031ceeec863407b7eeb9d1f5c0b7fc 100644
--- a/Source/core/timing/ResourceTimingInfo.h
+++ b/Source/core/timing/ResourceTimingInfo.h
@@ -67,8 +67,8 @@ public:
void clearLoadTimings()
{
m_finalResponse.setResourceLoadTiming(nullptr);
- for (size_t i = 0; i < m_redirectChain.size(); ++i)
- m_redirectChain[i].setResourceLoadTiming(nullptr);
+ for (auto& redirect : m_redirectChain)
Mike West 2014/10/21 19:29:42 Nit: Don't use `auto` here. Using ResourceResponse
+ redirect.setResourceLoadTiming(nullptr);
}
private:
« Source/core/timing/Performance.cpp ('K') | « Source/core/timing/PerformanceUserTiming.cpp ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698