Avoid copy-prone idiom "for (auto item : collection)"
[WebKit-https.git] / Source / WebCore / platform / FileChooser.cpp
index 4a10f16..095f8de 100644 (file)
@@ -70,12 +70,13 @@ void FileChooser::chooseFiles(const Vector<String>& filenames)
         return;
 
     Vector<FileChooserFileInfo> files;
-    for (unsigned i = 0; i < filenames.size(); ++i)
-        files.append(FileChooserFileInfo(filenames[i]));
+    for (auto& filename : filenames)
+        files.append(FileChooserFileInfo(filename));
     m_client->filesChosen(files);
 }
 
 #if PLATFORM(IOS)
+
 // FIXME: This function is almost identical to FileChooser::chooseFiles(). We should merge this function
 // with FileChooser::chooseFiles() and hence remove the PLATFORM(IOS)-guard.
 void FileChooser::chooseMediaFiles(const Vector<String>& filenames, const String& displayString, Icon* icon)
@@ -88,19 +89,20 @@ void FileChooser::chooseMediaFiles(const Vector<String>& filenames, const String
         return;
 
     Vector<FileChooserFileInfo> files;
-    for (auto filename : filenames)
+    for (auto& filename : filenames)
         files.append(FileChooserFileInfo(filename));
     m_client->filesChosen(files, displayString, icon);
 }
+
 #endif
 
 void FileChooser::chooseFiles(const Vector<FileChooserFileInfo>& files)
 {
-    // FIXME: This is inelegant. We should not be looking at settings here.
     Vector<String> paths;
-    for (unsigned i = 0; i < files.size(); ++i)
-        paths.append(files[i].path);
+    for (auto& file : files)
+        paths.append(file.path);
 
+    // FIXME: This is inelegant. We should not be looking at settings here.
     if (m_settings.selectedFiles == paths)
         return;