[Extensions] Convert APIs to use movable types [4]

chrome/browser/extensions/api/downloads/downloads_api.cc

Index: chrome/browser/extensions/api/downloads/downloads_api.cc
diff --git a/chrome/browser/extensions/api/downloads/downloads_api.cc b/chrome/browser/extensions/api/downloads/downloads_api.cc
index 0a297d2116bd4a67bd2bed82ea1b9fbfa9a1556f..a1c6fff5816dc505140bdac407d6eb83e97ecab0 100644
--- a/chrome/browser/extensions/api/downloads/downloads_api.cc
+++ b/chrome/browser/extensions/api/downloads/downloads_api.cc
@@ -989,11 +989,7 @@ bool DownloadsDownloadFunction::RunAsync() {
 
   if (options.headers.get()) {
     typedef downloads::HeaderNameValuePair HeaderNameValuePair;

[lazyboy, 2016/03/23 21:26:58]

optional nit: can get rid of this typedef since it is used only once now.

[Devlin, 2016/03/24 20:56:26]

Done.

-    for (std::vector<linked_ptr<HeaderNameValuePair> >::const_iterator iter =
-         options.headers->begin();
-         iter != options.headers->end();
-         ++iter) {
-      const HeaderNameValuePair& name_value = **iter;
+    for (const HeaderNameValuePair& name_value : *options.headers) {
       if (!net::HttpUtil::IsValidHeaderName(name_value.name)) {
         error_ = errors::kInvalidHeaderName;
         return false;