Remove #include <utils/fileutils.h> from most header files

It has some platform specific warts, filepath.h is somewhat
cheaper and less ugly overall.

Partially clean up surroundings

Change-Id: Ida1fd470ec020f69c446109427f92a08e879789f
Reviewed-by: Jarek Kobus <jaroslaw.kobus@qt.io>

src/plugins/cmakeprojectmanager/cmakebuildsystem.cpp

@@ -45,6 +45,7 @@
 
 #include <utils/algorithm.h>
 #include <utils/checkablemessagebox.h>
+#include <utils/fileutils.h>
 #include <utils/macroexpander.h>
 #include <utils/mimeconstants.h>
 #include <utils/qtcprocess.h>

src/plugins/cmakeprojectmanager/cmakebuildtarget.h

@@ -9,10 +9,6 @@
 #include <projectexplorer/projectnodes.h>
 #include <projectexplorer/runconfigurationaspects.h>
 
-#include <utils/fileutils.h>
-
-#include <QStringList>
-
 namespace CMakeProjectManager {
 
 enum TargetType {

src/plugins/cmakeprojectmanager/fileapidataextractor.cpp

@@ -16,6 +16,7 @@
 
 #include <utils/algorithm.h>
 #include <utils/icon.h>
+#include <utils/fileutils.h>
 #include <utils/fsengine/fileiconprovider.h>
 #include <utils/mimeconstants.h>
 #include <utils/mimeutils.h>

src/plugins/cmakeprojectmanager/fileapiparser.h

@@ -10,10 +10,6 @@
 #include <projectexplorer/headerpath.h>
 #include <projectexplorer/projectmacro.h>
 
-#include <utils/filesystemwatcher.h>
-#include <utils/fileutils.h>
-
-#include <QString>
 #include <QVector>
 #include <QVersionNumber>
 

src/plugins/cmakeprojectmanager/fileapireader.cpp

@@ -16,6 +16,7 @@
 
 #include <utils/algorithm.h>
 #include <utils/async.h>
+#include <utils/fileutils.h>
 #include <utils/futuresynchronizer.h>
 #include <utils/qtcassert.h>
 #include <utils/temporarydirectory.h>