Remove unused includes from core/inspector, core/loader and core/page

Source/core/inspector/InjectedScript.h

 /*
  * Copyright (C) 2012 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 18 matching lines @@
  */
 
 #ifndef InjectedScript_h
 #define InjectedScript_h
 
 #include "InspectorTypeBuilder.h"
 #include "bindings/v8/ScriptObject.h"
 #include "core/inspector/InjectedScriptBase.h"
 #include "core/inspector/InjectedScriptManager.h"
 #include "core/inspector/ScriptArguments.h"
-#include <wtf/Forward.h>
-#include <wtf/Noncopyable.h>
-#include <wtf/PassRefPtr.h>
-#include <wtf/RefPtr.h>
+#include "wtf/Forward.h"
+#include "wtf/PassRefPtr.h"

[yurys, 2013/06/24 15:00:44]

Why not remove PassRefPtr as well? Forward declaration should be enough to
declare return types as PassRefPtr.

 
 namespace WebCore {
 
 class InjectedScriptModule;
 class Node;
 class SerializedScriptValue;
 
 
 class InjectedScript : public InjectedScriptBase {
 public:
@@ skipping 49 matching lines @@
     friend InjectedScript InjectedScriptManager::injectedScriptFor(ScriptState*);
     InjectedScript(ScriptObject, InspectedStateAccessCheck);
 
     ScriptValue nodeAsScriptValue(Node*);
 };
 
 
 } // namespace WebCore
 
 #endif