Clean up forward declarations in Source/modules

Source/modules/crypto/CryptoKey.h

 /*
  * Copyright (C) 2013 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 15 matching lines @@
  * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
  * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
 #ifndef CryptoKey_h
 #define CryptoKey_h
 
 #include "bindings/core/v8/ScriptWrappable.h"
 #include "modules/crypto/NormalizeAlgorithm.h"
-#include "platform/heap/Handle.h"

[Mike West, 2014/10/11 10:53:05]

Same here.

 #include "public/platform/WebCryptoKey.h"
-#include "wtf/Forward.h"

[Mike West, 2014/10/11 10:53:05]

Don't we need this to use Vector<> below?

-#include "wtf/text/WTFString.h"

[Mike West, 2014/10/11 10:53:05]

This file uses String. Why remove the header?

 
 namespace blink {
 
 class CryptoResult;
-class KeyAlgorithm;
 
 class CryptoKey : public GarbageCollectedFinalized<CryptoKey>, public ScriptWrappable {
     DEFINE_WRAPPERTYPEINFO();
 public:
     static CryptoKey* create(const WebCryptoKey& key)
     {
         return new CryptoKey(key);
     }
 
     ~CryptoKey();
@@ skipping 16 matching lines @@
 
 protected:
     explicit CryptoKey(const WebCryptoKey&);
 
     const WebCryptoKey m_key;
 };
 
 } // namespace blink
 
 #endif // CryptoKey_h