Use Forward.h instead of forward-declaring WTF::String
authorjfbastien@apple.com <jfbastien@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 30 Mar 2018 05:05:15 +0000 (05:05 +0000)
committerjfbastien@apple.com <jfbastien@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 30 Mar 2018 05:05:15 +0000 (05:05 +0000)
https://bugs.webkit.org/show_bug.cgi?id=184172
<rdar://problem/39026146>

Reviewed by Yusuke Suzuki.

As part of #184164 I'm changing WTF::String, and the forward
declarations are just wrong because I'm making it templated. We
should use Forward.h anyways, so do that instead.

Source/JavaScriptCore:

* runtime/DateConversion.h:

Source/WebCore:

* css/makeprop.pl:
* platform/cocoa/PlaybackSessionInterface.h:
* platform/ios/Device.h:
* platform/ios/PlaybackSessionInterfaceAVKit.h:
* platform/ios/VideoFullscreenInterfaceAVKit.h:
* platform/win/PathWalker.h:
* testing/js/WebCoreTestSupport.h:

Source/WebKit:

* Scripts/webkit/LegacyMessages-expected.h:
* Scripts/webkit/Messages-expected.h:
* Scripts/webkit/MessagesSuperclass-expected.h:
* UIProcess/WebOpenPanelResultListenerProxy.h:

Source/WebKitLegacy/mac:

* WebView/WebScriptDebugger.h:

Source/WTF:

* wtf/Forward.h:
* wtf/HashTraits.h:
* wtf/PrintStream.h:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@230097 268f45cc-cd09-0410-ab3c-d52691b4dbfc

21 files changed:
Source/JavaScriptCore/ChangeLog
Source/JavaScriptCore/runtime/DateConversion.h
Source/WTF/ChangeLog
Source/WTF/wtf/Forward.h
Source/WTF/wtf/HashTraits.h
Source/WTF/wtf/PrintStream.h
Source/WebCore/ChangeLog
Source/WebCore/css/makeprop.pl
Source/WebCore/platform/cocoa/PlaybackSessionInterface.h
Source/WebCore/platform/ios/Device.h
Source/WebCore/platform/ios/PlaybackSessionInterfaceAVKit.h
Source/WebCore/platform/ios/VideoFullscreenInterfaceAVKit.h
Source/WebCore/platform/win/PathWalker.h
Source/WebCore/testing/js/WebCoreTestSupport.h
Source/WebKit/ChangeLog
Source/WebKit/Scripts/webkit/LegacyMessages-expected.h
Source/WebKit/Scripts/webkit/Messages-expected.h
Source/WebKit/Scripts/webkit/MessagesSuperclass-expected.h
Source/WebKit/UIProcess/WebOpenPanelResultListenerProxy.h
Source/WebKitLegacy/mac/ChangeLog
Source/WebKitLegacy/mac/WebView/WebScriptDebugger.h

index 39b6858..78b3449 100644 (file)
@@ -1,3 +1,17 @@
+2018-03-29  JF Bastien  <jfbastien@apple.com>
+
+        Use Forward.h instead of forward-declaring WTF::String
+        https://bugs.webkit.org/show_bug.cgi?id=184172
+        <rdar://problem/39026146>
+
+        Reviewed by Yusuke Suzuki.
+
+        As part of #184164 I'm changing WTF::String, and the forward
+        declarations are just wrong because I'm making it templated. We
+        should use Forward.h anyways, so do that instead.
+
+        * runtime/DateConversion.h:
+
 2018-03-29  Mark Lam  <mark.lam@apple.com>
 
         Use MacroAssemblerCodePtr in Wasm code for code pointers instead of void*.
index d7b763e..8eafa95 100644 (file)
 
 #pragma once
 
+#include <wtf/Forward.h>
 #include <wtf/GregorianDateTime.h>
 
-namespace WTF {
-class String;
-} // namespace WTF
-
 namespace JSC {
 
 enum DateTimeFormat {
index 10c433b..a2ebfde 100644 (file)
@@ -1,3 +1,19 @@
+2018-03-29  JF Bastien  <jfbastien@apple.com>
+
+        Use Forward.h instead of forward-declaring WTF::String
+        https://bugs.webkit.org/show_bug.cgi?id=184172
+        <rdar://problem/39026146>
+
+        Reviewed by Yusuke Suzuki.
+
+        As part of #184164 I'm changing WTF::String, and the forward
+        declarations are just wrong because I'm making it templated. We
+        should use Forward.h anyways, so do that instead.
+
+        * wtf/Forward.h:
+        * wtf/HashTraits.h:
+        * wtf/PrintStream.h:
+
 2018-03-29  Yusuke Suzuki  <utatane.tea@gmail.com>
 
         Remove WTF_EXPORTDATA and JS_EXPORTDATA
index 42fad81..3d9c3fe 100644 (file)
@@ -46,6 +46,7 @@ class StringImpl;
 class StringView;
 class TextPosition;
 class TextStream;
+class UniquedStringImpl;
 class WallTime;
 
 struct FastMalloc;
index 35a27a5..bcd8d7b 100644 (file)
@@ -1,5 +1,5 @@
 /*
- * Copyright (C) 2005, 2006, 2007, 2008, 2011, 2012 Apple Inc. All rights reserved.
+ * Copyright (C) 2005-2018 Apple Inc. All rights reserved.
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
@@ -23,6 +23,7 @@
 
 #include <limits>
 #include <utility>
+#include <wtf/Forward.h>
 #include <wtf/HashFunctions.h>
 #include <wtf/KeyValuePair.h>
 #include <wtf/Optional.h>
 
 namespace WTF {
 
-class String;
-
-template<typename T> struct HashTraits;
-
 template<bool isInteger, typename T> struct GenericHashTraitsBase;
 
 template<typename T> struct GenericHashTraitsBase<false, T> {
index a69d1d0..d6c4364 100644 (file)
@@ -1,5 +1,5 @@
 /*
- * Copyright (C) 2012, 2014-2016 Apple Inc. All rights reserved.
+ * Copyright (C) 2012-2018 Apple Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
@@ -29,6 +29,7 @@
 #include <memory>
 #include <stdarg.h>
 #include <tuple>
+#include <wtf/Forward.h>
 #include <wtf/FastMalloc.h>
 #include <wtf/Noncopyable.h>
 #include <wtf/Optional.h>
 
 namespace WTF {
 
-class AtomicStringImpl;
-class CString;
-class String;
-class StringImpl;
-class StringView;
-class UniquedStringImpl;
-
 inline const char* boolForPrinting(bool value)
 {
     return value ? "true" : "false";
index 9611ff3..98e041c 100644 (file)
@@ -1,3 +1,23 @@
+2018-03-29  JF Bastien  <jfbastien@apple.com>
+
+        Use Forward.h instead of forward-declaring WTF::String
+        https://bugs.webkit.org/show_bug.cgi?id=184172
+        <rdar://problem/39026146>
+
+        Reviewed by Yusuke Suzuki.
+
+        As part of #184164 I'm changing WTF::String, and the forward
+        declarations are just wrong because I'm making it templated. We
+        should use Forward.h anyways, so do that instead.
+
+        * css/makeprop.pl:
+        * platform/cocoa/PlaybackSessionInterface.h:
+        * platform/ios/Device.h:
+        * platform/ios/PlaybackSessionInterfaceAVKit.h:
+        * platform/ios/VideoFullscreenInterfaceAVKit.h:
+        * platform/win/PathWalker.h:
+        * testing/js/WebCoreTestSupport.h:
+
 2018-03-29  Ross Kirsling  <ross.kirsling@sony.com>
 
         MSVC __forceinline slows down JSC release build fivefold after r229391
index 1383853..3af6952 100755 (executable)
@@ -3,7 +3,7 @@
 #   This file is part of the WebKit project
 #
 #   Copyright (C) 1999 Waldo Bastian (bastian@kde.org)
-#   Copyright (C) 2007-2017 Apple Inc. All rights reserved.
+#   Copyright (C) 2007-2018 Apple Inc. All rights reserved.
 #   Copyright (C) 2008 Nokia Corporation and/or its subsidiary(-ies)
 #   Copyright (C) 2010 Andras Becsi (abecsi@inf.u-szeged.hu), University of Szeged
 #   Copyright (C) 2013 Google Inc. All rights reserved.
@@ -415,14 +415,10 @@ print HEADER << "EOF";
 #pragma once
 
 #include <string.h>
+#include <wtf/Forward.h>
 #include <wtf/HashFunctions.h>
 #include <wtf/HashTraits.h>
 
-namespace WTF {
-class AtomicString;
-class String;
-}
-
 namespace WebCore {
 
 enum CSSPropertyID : uint16_t {
index 4105ef6..cdf2bbf 100644 (file)
@@ -1,5 +1,5 @@
 /*
- * Copyright (C) 2016 Apple Inc. All rights reserved.
+ * Copyright (C) 2016-2018 Apple Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
 
 #if PLATFORM(IOS) || (PLATFORM(MAC) && ENABLE(VIDEO_PRESENTATION_MODE))
 
+#include <wtf/Forward.h>
 #include <wtf/Vector.h>
 
-namespace WTF {
-class String;
-}
-
 namespace WebCore {
 
 class TimeRanges;
index 3480919..d9da467 100644 (file)
@@ -1,5 +1,5 @@
 /*
- * Copyright (C) 2015 Apple Inc. All rights reserved.
+ * Copyright (C) 2015-2018 Apple Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
 
 #if PLATFORM(IOS)
 
+#include <wtf/Forward.h>
 #include <pal/spi/ios/MobileGestaltSPI.h>
 
-namespace WTF {
-class String;
-}
-
 namespace WebCore {
 
 WEBCORE_EXPORT MGDeviceClass deviceClass();
index 9fcd7b4..ece7715 100644 (file)
@@ -1,5 +1,5 @@
 /*
- * Copyright (C) 2016 Apple Inc. All rights reserved.
+ * Copyright (C) 2016-2018 Apple Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
@@ -35,6 +35,7 @@
 #include "Timer.h"
 #include <functional>
 #include <objc/objc.h>
+#include <wtf/Forward.h>
 #include <wtf/RefCounted.h>
 #include <wtf/RefPtr.h>
 #include <wtf/RetainPtr.h>
@@ -45,10 +46,6 @@ OBJC_CLASS UIViewController;
 OBJC_CLASS UIWindow;
 OBJC_CLASS UIView;
 
-namespace WTF {
-class String;
-}
-
 namespace WebCore {
 class IntRect;
 class PlaybackSessionModel;
index e11fc25..ac16cd6 100644 (file)
@@ -34,6 +34,7 @@
 #include "PlaybackSessionInterfaceAVKit.h"
 #include "VideoFullscreenModel.h"
 #include <objc/objc.h>
+#include <wtf/Forward.h>
 #include <wtf/Function.h>
 #include <wtf/RefCounted.h>
 #include <wtf/RefPtr.h>
@@ -51,10 +52,6 @@ OBJC_CLASS WebAVPlayerViewController;
 OBJC_CLASS WebAVPlayerViewControllerDelegate;
 OBJC_CLASS NSError;
 
-namespace WTF {
-class String;
-}
-
 namespace WebCore {
 class IntRect;
 class FloatSize;
index 219c837..fc59d67 100644 (file)
@@ -1,5 +1,5 @@
 /*
- * Copyright (C) 2011 Apple Inc. All rights reserved.
+ * Copyright (C) 2011-2018 Apple Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  */
 
 #include <Windows.h>
+#include <wtf/Forward.h>
 #include <wtf/Noncopyable.h>
 
-namespace WTF {
-    class String;
-}
-
 namespace WebCore {
 
 class PathWalker {
index a26b2f7..b3f39cd 100644 (file)
@@ -1,6 +1,6 @@
 /*
  * Copyright (C) 2011, 2015 Google Inc. All rights reserved.
- * Copyright (C) 2016 Apple Inc. All rights reserved.
+ * Copyright (C) 2016-2018 Apple Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
@@ -26,6 +26,8 @@
 
 #pragma once
 
+#include <wtf/Forward.h>
+
 typedef const struct OpaqueJSContext* JSContextRef;
 typedef struct OpaqueJSString* JSStringRef;
 typedef struct OpaqueJSValue* JSObjectRef;
@@ -36,10 +38,6 @@ typedef struct OpaqueJSValue* JSObjectRef;
 #define TEST_SUPPORT_EXPORT
 #endif
 
-namespace WTF {
-class String;
-}
-
 namespace WebCore {
 class Frame;
 }
index ff0ec02..3b83c30 100644 (file)
@@ -1,3 +1,20 @@
+2018-03-29  JF Bastien  <jfbastien@apple.com>
+
+        Use Forward.h instead of forward-declaring WTF::String
+        https://bugs.webkit.org/show_bug.cgi?id=184172
+        <rdar://problem/39026146>
+
+        Reviewed by Yusuke Suzuki.
+
+        As part of #184164 I'm changing WTF::String, and the forward
+        declarations are just wrong because I'm making it templated. We
+        should use Forward.h anyways, so do that instead.
+
+        * Scripts/webkit/LegacyMessages-expected.h:
+        * Scripts/webkit/Messages-expected.h:
+        * Scripts/webkit/MessagesSuperclass-expected.h:
+        * UIProcess/WebOpenPanelResultListenerProxy.h:
+
 2018-03-29  Brian Burg  <bburg@apple.com>
 
         Web Automation: clipToViewport is ignored for element screenshots
index 2a0994c..52e896a 100644 (file)
@@ -1,5 +1,5 @@
 /*
- * Copyright (C) 2010 Apple Inc. All rights reserved.
+ * Copyright (C) 2010-2018 Apple Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
@@ -33,6 +33,7 @@
 #include <WebCore/KeyboardEvent.h>
 #include <WebCore/PluginData.h>
 #include <utility>
+#include <wtf/Forward.h>
 #include <wtf/HashMap.h>
 #include <wtf/ThreadSafeRefCounted.h>
 #include <wtf/Vector.h>
@@ -44,10 +45,6 @@ namespace IPC {
     class DummyType;
 }
 
-namespace WTF {
-    class String;
-}
-
 namespace WebKit {
     class WebTouchEvent;
     class WebPreferencesStore;
index 2a0994c..52e896a 100644 (file)
@@ -1,5 +1,5 @@
 /*
- * Copyright (C) 2010 Apple Inc. All rights reserved.
+ * Copyright (C) 2010-2018 Apple Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
@@ -33,6 +33,7 @@
 #include <WebCore/KeyboardEvent.h>
 #include <WebCore/PluginData.h>
 #include <utility>
+#include <wtf/Forward.h>
 #include <wtf/HashMap.h>
 #include <wtf/ThreadSafeRefCounted.h>
 #include <wtf/Vector.h>
@@ -44,10 +45,6 @@ namespace IPC {
     class DummyType;
 }
 
-namespace WTF {
-    class String;
-}
-
 namespace WebKit {
     class WebTouchEvent;
     class WebPreferencesStore;
index f65873a..2250084 100644 (file)
@@ -1,5 +1,5 @@
 /*
- * Copyright (C) 2010 Apple Inc. All rights reserved.
+ * Copyright (C) 2010-2018 Apple Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
 #pragma once
 
 #include "ArgumentCoders.h"
-
-namespace WTF {
-    class String;
-}
+#include <wtf/Forward.h>
 
 namespace Messages {
 namespace WebPage {
index 82aa46a..12cb3f5 100644 (file)
@@ -1,5 +1,5 @@
 /*
- * Copyright (C) 2010 Apple Inc. All rights reserved.
+ * Copyright (C) 2010-2018 Apple Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
@@ -32,7 +32,6 @@
 namespace API {
 class Array;
 class Data;
-class String;
 }
 
 namespace WebKit {
index 0fe6d8b..18a0904 100644 (file)
@@ -1,3 +1,17 @@
+2018-03-29  JF Bastien  <jfbastien@apple.com>
+
+        Use Forward.h instead of forward-declaring WTF::String
+        https://bugs.webkit.org/show_bug.cgi?id=184172
+        <rdar://problem/39026146>
+
+        Reviewed by Yusuke Suzuki.
+
+        As part of #184164 I'm changing WTF::String, and the forward
+        declarations are just wrong because I'm making it templated. We
+        should use Forward.h anyways, so do that instead.
+
+        * WebView/WebScriptDebugger.h:
+
 2018-03-28  Brent Fulgham  <bfulgham@apple.com>
 
         Protect against invalid mach ports returned by mach_port_request_notification
index 6970848..8357d91 100644 (file)
@@ -1,5 +1,5 @@
 /*
- * Copyright (C) 2008-2013 Apple Inc. All rights reserved.
+ * Copyright (C) 2008-2018 Apple Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
 #include <JavaScriptCore/Debugger.h>
 #include <JavaScriptCore/Strong.h>
 
+#include <wtf/Forward.h>
 #include <wtf/RetainPtr.h>
 
-namespace WTF {
-class String;
-}
-
 namespace JSC {
     class DebuggerCallFrame;
     class ExecState;