Remove unnecessary `using namespace WTF`s (or at least restrict their scope).
authorross.kirsling@sony.com <ross.kirsling@sony.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 29 Jan 2019 06:00:30 +0000 (06:00 +0000)
committerross.kirsling@sony.com <ross.kirsling@sony.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 29 Jan 2019 06:00:30 +0000 (06:00 +0000)
https://bugs.webkit.org/show_bug.cgi?id=193941

Reviewed by Alex Christensen.

Source/JavaScriptCore:

* API/JSWeakObjectMapRefPrivate.cpp:
* bytecompiler/NodesCodegen.cpp:
* heap/MachineStackMarker.cpp:
* jit/ExecutableAllocator.cpp:
* jsc.cpp:
* parser/Nodes.cpp:
* runtime/DateConstructor.cpp:
* runtime/DateConversion.cpp:
* runtime/DateInstance.cpp:
* runtime/DatePrototype.cpp:
* runtime/InitializeThreading.cpp:
* runtime/IteratorOperations.cpp:
* runtime/JSDateMath.cpp:
* runtime/JSGlobalObjectFunctions.cpp:
* runtime/StringPrototype.cpp:
* runtime/VM.cpp:
* testRegExp.cpp:
* tools/JSDollarVM.cpp:
* yarr/YarrInterpreter.cpp:
* yarr/YarrJIT.cpp:
* yarr/YarrPattern.cpp:
* yarr/YarrUnicodeProperties.cpp:

Source/WebCore:

* css/CSSBasicShapes.cpp:
* css/CSSPrimitiveValue.cpp:
* css/parser/CSSParser.cpp:
* css/parser/CSSParserSelector.cpp:
* css/parser/CSSPropertyParser.cpp:
* dom/Document.cpp:
* dom/EventListenerMap.cpp:
* dom/EventTarget.cpp:
* editing/Editor.cpp:
* html/HTMLElement.cpp:
* html/HTMLFontElement.cpp:
* html/parser/HTMLTokenizer.cpp:
* html/track/TrackBase.cpp:
* loader/FTPDirectoryParser.cpp:
* loader/TextResourceDecoder.cpp:
* loader/cache/CachedResource.cpp:
* page/ContextMenuController.cpp:
* page/Navigator.cpp:
* platform/Length.cpp:
* platform/cocoa/KeyEventCocoa.mm:
* platform/graphics/FontCascade.cpp:
* platform/graphics/WidthIterator.cpp:
* platform/graphics/opengl/GraphicsContext3DOpenGLCommon.cpp:
* platform/ios/KeyEventIOS.mm:
* platform/mac/KeyEventMac.mm:
* platform/network/HTTPParsers.cpp:
* platform/text/TextCodecUTF8.cpp:
* platform/text/TextEncodingRegistry.cpp:
* platform/win/KeyEventWin.cpp:
* rendering/BidiRun.cpp:
* rendering/FloatingObjects.cpp:
* rendering/RenderBlock.cpp:
* rendering/RenderListMarker.cpp:
* rendering/RenderText.cpp:

Source/WebKit:

* Shared/linux/WebMemorySamplerLinux.cpp:

Source/WebKitLegacy/win:

* Plugins/PluginMessageThrottlerWin.cpp:
* Plugins/PluginView.cpp:
* Plugins/PluginViewWin.cpp:

Tools:

* TestWebKitAPI/Tests/WTF/ConcurrentPtrHashSet.cpp:
* TestWebKitAPI/Tests/WTF/Condition.cpp:
* TestWebKitAPI/Tests/WTF/Lock.cpp:
* TestWebKitAPI/Tests/WTF/MetaAllocator.cpp:
* TestWebKitAPI/Tests/WTF/ParkingLot.cpp:
* TestWebKitAPI/Tests/WTF/RedBlackTree.cpp:
* TestWebKitAPI/Tests/WTF/ScopedLambda.cpp:
* TestWebKitAPI/Tests/WTF/Time.cpp:
* TestWebKitAPI/Tests/WTF/UniqueArray.cpp:
* TestWebKitAPI/Tests/WebCore/ApplicationManifestParser.cpp:
* TestWebKitAPI/Tests/WebCore/Logging.cpp:
* WebKitTestRunner/InjectedBundle/cocoa/InjectedBundlePageCocoa.mm:

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

77 files changed:
Source/JavaScriptCore/API/JSWeakObjectMapRefPrivate.cpp
Source/JavaScriptCore/ChangeLog
Source/JavaScriptCore/bytecompiler/NodesCodegen.cpp
Source/JavaScriptCore/heap/MachineStackMarker.cpp
Source/JavaScriptCore/jit/ExecutableAllocator.cpp
Source/JavaScriptCore/jsc.cpp
Source/JavaScriptCore/parser/Nodes.cpp
Source/JavaScriptCore/runtime/DateConstructor.cpp
Source/JavaScriptCore/runtime/DateConversion.cpp
Source/JavaScriptCore/runtime/DateInstance.cpp
Source/JavaScriptCore/runtime/DatePrototype.cpp
Source/JavaScriptCore/runtime/InitializeThreading.cpp
Source/JavaScriptCore/runtime/IteratorOperations.cpp
Source/JavaScriptCore/runtime/JSDateMath.cpp
Source/JavaScriptCore/runtime/JSGlobalObjectFunctions.cpp
Source/JavaScriptCore/runtime/StringPrototype.cpp
Source/JavaScriptCore/runtime/VM.cpp
Source/JavaScriptCore/testRegExp.cpp
Source/JavaScriptCore/tools/JSDollarVM.cpp
Source/JavaScriptCore/yarr/YarrInterpreter.cpp
Source/JavaScriptCore/yarr/YarrJIT.cpp
Source/JavaScriptCore/yarr/YarrPattern.cpp
Source/JavaScriptCore/yarr/YarrUnicodeProperties.cpp
Source/WebCore/ChangeLog
Source/WebCore/css/CSSBasicShapes.cpp
Source/WebCore/css/CSSPrimitiveValue.cpp
Source/WebCore/css/parser/CSSParser.cpp
Source/WebCore/css/parser/CSSParserSelector.cpp
Source/WebCore/css/parser/CSSPropertyParser.cpp
Source/WebCore/dom/Document.cpp
Source/WebCore/dom/EventListenerMap.cpp
Source/WebCore/dom/EventTarget.cpp
Source/WebCore/editing/Editor.cpp
Source/WebCore/html/HTMLElement.cpp
Source/WebCore/html/HTMLFontElement.cpp
Source/WebCore/html/parser/HTMLTokenizer.cpp
Source/WebCore/html/track/TrackBase.cpp
Source/WebCore/loader/FTPDirectoryParser.cpp
Source/WebCore/loader/TextResourceDecoder.cpp
Source/WebCore/loader/cache/CachedResource.cpp
Source/WebCore/page/ContextMenuController.cpp
Source/WebCore/page/Navigator.cpp
Source/WebCore/platform/Length.cpp
Source/WebCore/platform/cocoa/KeyEventCocoa.mm
Source/WebCore/platform/graphics/FontCascade.cpp
Source/WebCore/platform/graphics/WidthIterator.cpp
Source/WebCore/platform/graphics/opengl/GraphicsContext3DOpenGLCommon.cpp
Source/WebCore/platform/ios/KeyEventIOS.mm
Source/WebCore/platform/mac/KeyEventMac.mm
Source/WebCore/platform/network/HTTPParsers.cpp
Source/WebCore/platform/text/TextCodecUTF8.cpp
Source/WebCore/platform/text/TextEncodingRegistry.cpp
Source/WebCore/platform/win/KeyEventWin.cpp
Source/WebCore/rendering/BidiRun.cpp
Source/WebCore/rendering/FloatingObjects.cpp
Source/WebCore/rendering/RenderBlock.cpp
Source/WebCore/rendering/RenderListMarker.cpp
Source/WebCore/rendering/RenderText.cpp
Source/WebKit/ChangeLog
Source/WebKit/Shared/linux/WebMemorySamplerLinux.cpp
Source/WebKitLegacy/win/ChangeLog
Source/WebKitLegacy/win/Plugins/PluginMessageThrottlerWin.cpp
Source/WebKitLegacy/win/Plugins/PluginView.cpp
Source/WebKitLegacy/win/Plugins/PluginViewWin.cpp
Tools/ChangeLog
Tools/TestWebKitAPI/Tests/WTF/ConcurrentPtrHashSet.cpp
Tools/TestWebKitAPI/Tests/WTF/Condition.cpp
Tools/TestWebKitAPI/Tests/WTF/Lock.cpp
Tools/TestWebKitAPI/Tests/WTF/MetaAllocator.cpp
Tools/TestWebKitAPI/Tests/WTF/ParkingLot.cpp
Tools/TestWebKitAPI/Tests/WTF/RedBlackTree.cpp
Tools/TestWebKitAPI/Tests/WTF/ScopedLambda.cpp
Tools/TestWebKitAPI/Tests/WTF/Time.cpp
Tools/TestWebKitAPI/Tests/WTF/UniqueArray.cpp
Tools/TestWebKitAPI/Tests/WebCore/ApplicationManifestParser.cpp
Tools/TestWebKitAPI/Tests/WebCore/Logging.cpp
Tools/WebKitTestRunner/InjectedBundle/cocoa/InjectedBundlePageCocoa.mm

index d128836..c6b396d 100644 (file)
@@ -34,7 +34,6 @@
 #include "Weak.h"
 #include "WeakGCMapInlines.h"
 
-using namespace WTF;
 using namespace JSC;
 
 #ifdef __cplusplus
index b446191..8e3e595 100644 (file)
@@ -1,3 +1,33 @@
+2019-01-28  Ross Kirsling  <ross.kirsling@sony.com>
+
+        Remove unnecessary `using namespace WTF`s (or at least restrict their scope).
+        https://bugs.webkit.org/show_bug.cgi?id=193941
+
+        Reviewed by Alex Christensen.
+
+        * API/JSWeakObjectMapRefPrivate.cpp:
+        * bytecompiler/NodesCodegen.cpp:
+        * heap/MachineStackMarker.cpp:
+        * jit/ExecutableAllocator.cpp:
+        * jsc.cpp:
+        * parser/Nodes.cpp:
+        * runtime/DateConstructor.cpp:
+        * runtime/DateConversion.cpp:
+        * runtime/DateInstance.cpp:
+        * runtime/DatePrototype.cpp:
+        * runtime/InitializeThreading.cpp:
+        * runtime/IteratorOperations.cpp:
+        * runtime/JSDateMath.cpp:
+        * runtime/JSGlobalObjectFunctions.cpp:
+        * runtime/StringPrototype.cpp:
+        * runtime/VM.cpp:
+        * testRegExp.cpp:
+        * tools/JSDollarVM.cpp:
+        * yarr/YarrInterpreter.cpp:
+        * yarr/YarrJIT.cpp:
+        * yarr/YarrPattern.cpp:
+        * yarr/YarrUnicodeProperties.cpp:
+
 2019-01-28  Yusuke Suzuki  <ysuzuki@apple.com>
 
         [JSC] Reduce size of memory used for ShadowChicken
index 90b6d49..6d5045b 100644 (file)
@@ -45,8 +45,6 @@
 #include <wtf/Threading.h>
 #include <wtf/text/StringBuilder.h>
 
-using namespace WTF;
-
 namespace JSC {
 
 /*
index 0bc0b8e..ab19c0e 100644 (file)
@@ -30,8 +30,6 @@
 #include <wtf/PageBlock.h>
 #include <wtf/StdLibExtras.h>
 
-using namespace WTF;
-
 namespace JSC {
 
 MachineThreads::MachineThreads()
index 901764f..4606756 100644 (file)
@@ -82,10 +82,10 @@ extern "C" {
 
 #endif
 
-using namespace WTF;
-
 namespace JSC {
 
+using namespace WTF;
+
 #if defined(FIXED_EXECUTABLE_MEMORY_POOL_SIZE_IN_MB) && FIXED_EXECUTABLE_MEMORY_POOL_SIZE_IN_MB > 0
 static const size_t fixedExecutableMemoryPoolSize = FIXED_EXECUTABLE_MEMORY_POOL_SIZE_IN_MB * 1024 * 1024;
 #elif CPU(ARM)
index 7884013..be01806 100644 (file)
@@ -157,7 +157,6 @@ struct MemoryFootprint {
 #endif
 
 using namespace JSC;
-using namespace WTF;
 
 namespace {
 
index 95774d9..6143efd 100644 (file)
@@ -31,8 +31,6 @@
 #include "ModuleScopeData.h"
 #include <wtf/Assertions.h>
 
-using namespace WTF;
-
 namespace JSC {
 
 // ------------------------------ StatementNode --------------------------------
index dbaf636..25f9f02 100644 (file)
@@ -43,8 +43,6 @@
 #include <sys/timeb.h>
 #endif
 
-using namespace WTF;
-
 namespace JSC {
 
 EncodedJSValue JSC_HOST_CALL dateParse(ExecState*);
@@ -56,6 +54,8 @@ EncodedJSValue JSC_HOST_CALL dateUTC(ExecState*);
 
 namespace JSC {
 
+using namespace WTF;
+
 const ClassInfo DateConstructor::s_info = { "Function", &InternalFunction::s_info, &dateConstructorTable, nullptr, CREATE_METHOD_TABLE(DateConstructor) };
 
 /* Source for DateConstructor.lut.h
index 8a87cc6..fa0fb42 100644 (file)
 #include <wtf/text/win/WCharStringExtras.h>
 #endif
 
-using namespace WTF;
-
 namespace JSC {
 
+using namespace WTF;
+
 template<int width>
 static inline void appendNumber(StringBuilder& builder, int value)
 {
index d76937a..4dd8603 100644 (file)
 #include <math.h>
 #include <wtf/MathExtras.h>
 
-using namespace WTF;
-
 namespace JSC {
 
+using namespace WTF;
+
 const ClassInfo DateInstance::s_info = {"Date", &JSWrapperObject::s_info, nullptr, nullptr, CREATE_METHOD_TABLE(DateInstance)};
 
 DateInstance::DateInstance(VM& vm, Structure* structure)
index c2d42a0..71edba9 100644 (file)
 #include <CoreFoundation/CoreFoundation.h>
 #endif
 
-using namespace WTF;
-
 namespace JSC {
 
+using namespace WTF;
+
 EncodedJSValue JSC_HOST_CALL dateProtoFuncGetDate(ExecState*);
 EncodedJSValue JSC_HOST_CALL dateProtoFuncGetDay(ExecState*);
 EncodedJSValue JSC_HOST_CALL dateProtoFuncGetFullYear(ExecState*);
index 1136d6e..50a8052 100644 (file)
@@ -49,8 +49,6 @@
 #include <wtf/dtoa.h>
 #include <wtf/dtoa/cached-powers.h>
 
-using namespace WTF;
-
 namespace JSC {
 
 static_assert(sizeof(bool) == 1, "LLInt and JIT assume sizeof(bool) is always 1 when touching it directly from assembly code.");
index 9ebef78..141a147 100644 (file)
@@ -32,8 +32,6 @@
 #include "JSCInlines.h"
 #include "ObjectConstructor.h"
 
-using namespace WTF;
-
 namespace JSC {
 
 JSValue iteratorNext(ExecState* exec, IterationRecord iterationRecord, JSValue argument)
index c13b3de..0715d4e 100644 (file)
 #include <sys/timeb.h>
 #endif
 
-using namespace WTF;
-
 namespace JSC {
 
+using namespace WTF;
+
 static inline double timeToMS(double hour, double min, double sec, double ms)
 {
     return (((hour * minutesPerHour + min) * secondsPerMinute + sec) * msPerSecond + ms);
index 61fe076..68a82c2 100644 (file)
 #include <wtf/text/StringBuilder.h>
 #include <wtf/unicode/UTF8Conversion.h>
 
-using namespace WTF;
-using namespace Unicode;
-
 namespace JSC {
 
+using namespace WTF::Unicode;
+
 const ASCIILiteral ObjectProtoCalledOnNullOrUndefinedError { "Object.prototype.__proto__ called on null or undefined"_s };
 
 template<unsigned charactersCount>
index 16b3e97..28b862d 100644 (file)
@@ -55,8 +55,6 @@
 #include <wtf/text/StringView.h>
 #include <wtf/unicode/Collator.h>
 
-using namespace WTF;
-
 namespace JSC {
 
 STATIC_ASSERT_IS_TRIVIALLY_DESTRUCTIBLE(StringPrototype);
index 5f4aa78..9b954c2 100644 (file)
 #include "RegExp.h"
 #endif
 
-using namespace WTF;
-
 namespace JSC {
 
 #if ENABLE(JIT)
index e6cdd50..5091b71 100644 (file)
@@ -48,7 +48,6 @@
 const int MaxLineLength = 100 * 1024;
 
 using namespace JSC;
-using namespace WTF;
 
 struct CommandLine {
     CommandLine()
index ac4e00f..747aa29 100644 (file)
@@ -57,7 +57,6 @@
 #endif
 
 using namespace JSC;
-using namespace WTF;
 
 namespace {
 
index 910bd7b..6b9664d 100644 (file)
@@ -37,8 +37,6 @@
 #include <wtf/text/CString.h>
 #include <wtf/text/WTFString.h>
 
-using namespace WTF;
-
 namespace JSC { namespace Yarr {
 
 template<typename CharType>
@@ -1655,7 +1653,7 @@ private:
     bool unicode;
     unsigned* output;
     InputStream input;
-    BumpPointerPool* allocatorPool { nullptr };
+    WTF::BumpPointerPool* allocatorPool { nullptr };
     unsigned startOffset;
     unsigned remainingMatchCount;
 };
index be71e4c..851fe6f 100644 (file)
@@ -36,8 +36,6 @@
 
 #if ENABLE(YARR_JIT)
 
-using namespace WTF;
-
 namespace JSC { namespace Yarr {
 
 template<YarrJITCompileMode compileMode>
index 733fb2c..36fc4d2 100644 (file)
@@ -38,8 +38,6 @@
 #include <wtf/Vector.h>
 #include <wtf/text/WTFString.h>
 
-using namespace WTF;
-
 namespace JSC { namespace Yarr {
 
 #include "RegExpJitTables.h"
index b50ba03..e6a0866 100644 (file)
@@ -29,8 +29,6 @@
 #include "Yarr.h"
 #include "YarrPattern.h"
 
-using namespace WTF;
-
 namespace JSC { namespace Yarr {
 
 struct HashIndex {
index f728566..3756375 100644 (file)
@@ -1,3 +1,45 @@
+2019-01-28  Ross Kirsling  <ross.kirsling@sony.com>
+
+        Remove unnecessary `using namespace WTF`s (or at least restrict their scope).
+        https://bugs.webkit.org/show_bug.cgi?id=193941
+
+        Reviewed by Alex Christensen.
+
+        * css/CSSBasicShapes.cpp:
+        * css/CSSPrimitiveValue.cpp:
+        * css/parser/CSSParser.cpp:
+        * css/parser/CSSParserSelector.cpp:
+        * css/parser/CSSPropertyParser.cpp:
+        * dom/Document.cpp:
+        * dom/EventListenerMap.cpp:
+        * dom/EventTarget.cpp:
+        * editing/Editor.cpp:
+        * html/HTMLElement.cpp:
+        * html/HTMLFontElement.cpp:
+        * html/parser/HTMLTokenizer.cpp:
+        * html/track/TrackBase.cpp:
+        * loader/FTPDirectoryParser.cpp:
+        * loader/TextResourceDecoder.cpp:
+        * loader/cache/CachedResource.cpp:
+        * page/ContextMenuController.cpp:
+        * page/Navigator.cpp:
+        * platform/Length.cpp:
+        * platform/cocoa/KeyEventCocoa.mm:
+        * platform/graphics/FontCascade.cpp:
+        * platform/graphics/WidthIterator.cpp:
+        * platform/graphics/opengl/GraphicsContext3DOpenGLCommon.cpp:
+        * platform/ios/KeyEventIOS.mm:
+        * platform/mac/KeyEventMac.mm:
+        * platform/network/HTTPParsers.cpp:
+        * platform/text/TextCodecUTF8.cpp:
+        * platform/text/TextEncodingRegistry.cpp:
+        * platform/win/KeyEventWin.cpp:
+        * rendering/BidiRun.cpp:
+        * rendering/FloatingObjects.cpp:
+        * rendering/RenderBlock.cpp:
+        * rendering/RenderListMarker.cpp:
+        * rendering/RenderText.cpp:
+
 2019-01-28  Fujii Hironori  <Hironori.Fujii@sony.com>
 
         [Win] WebCore/platform/Process.h is conflicting with process.h
index e1eaf94..35fcef2 100644 (file)
@@ -28,7 +28,6 @@
  */
 
 #include "config.h"
-
 #include "CSSBasicShapes.h"
 
 #include "CSSMarkup.h"
@@ -39,9 +38,7 @@
 #include "SVGPathUtilities.h"
 #include <wtf/text/StringBuilder.h>
 
-
 namespace WebCore {
-using namespace WTF;
 
 static String serializePositionOffset(const Pair& offset, const Pair& other)
 {
@@ -365,7 +362,7 @@ static String buildInsetString(const String& top, const String& right, const Str
             }
 
             if (verticalRadii.size() != horizontalRadii.size()
-                || !VectorComparer<false, String>::compare(verticalRadii.data(), horizontalRadii.data(), verticalRadii.size())) {
+                || !WTF::VectorComparer<false, String>::compare(verticalRadii.data(), horizontalRadii.data(), verticalRadii.size())) {
                 result.appendLiteral(separator);
                 result.appendLiteral("/");
 
index 06ffb60..26eed35 100644 (file)
@@ -52,7 +52,6 @@
 
 
 namespace WebCore {
-using namespace WTF;
 
 static inline bool isValidCSSUnitTypeForDoubleConversion(CSSPrimitiveValue::UnitType unitType)
 {
index 0d6aef0..b4fa697 100644 (file)
@@ -54,7 +54,6 @@
 #include <wtf/text/StringBuilder.h>
 
 namespace WebCore {
-using namespace WTF;
 
 CSSParser::CSSParser(const CSSParserContext& context)
     : m_context(context)
index 26a18ac..1af424e 100644 (file)
@@ -32,8 +32,6 @@
 
 namespace WebCore {
 
-using namespace WTF;
-
 CSSParserSelector* CSSParserSelector::parsePagePseudoSelector(const AtomicString& pseudoTypeString)
 {
     CSSSelector::PagePseudoClassType pseudoType;
index 1ddaabd..2f536e9 100644 (file)
 #include <memory>
 #include <wtf/text/StringBuilder.h>
 
-
 namespace WebCore {
-using namespace WTF;
 
-    
 bool isCustomPropertyName(const String& propertyName)
 {
     return propertyName.length() > 2 && propertyName.characterAt(0) == '-' && propertyName.characterAt(1) == '-';
index 612c35f..0325412 100644 (file)
 #include "WebMetalRenderingContext.h"
 #endif
 
-
 namespace WebCore {
 
 WTF_MAKE_ISO_ALLOCATED_IMPL(Document);
 
-using namespace PAL;
-using namespace WTF;
-using namespace Unicode;
-
 using namespace HTMLNames;
+using namespace PAL;
+using namespace WTF::Unicode;
 
 static const unsigned cMaxWriteRecursionDepth = 21;
 bool Document::hasEverCreatedAnAXObjectCache = false;
index bf922a9..bdd2d56 100644 (file)
@@ -41,7 +41,6 @@
 
 
 namespace WebCore {
-using namespace WTF;
 
 #ifndef NDEBUG
 void EventListenerMap::assertNoActiveIterators() const
index c37910e..7ae5344 100644 (file)
@@ -51,8 +51,6 @@
 
 namespace WebCore {
 
-using namespace WTF;
-
 bool EventTarget::isNode() const
 {
     return false;
index 8fd53ba..df190e5 100644 (file)
@@ -200,8 +200,7 @@ void ClearTextCommand::CreateAndApply(const RefPtr<Frame> frame)
 }
 
 using namespace HTMLNames;
-using namespace WTF;
-using namespace Unicode;
+using namespace WTF::Unicode;
 
 TemporarySelectionChange::TemporarySelectionChange(Frame& frame, Optional<VisibleSelection> temporarySelection, OptionSet<TemporarySelectionOption> options)
     : m_frame(frame)
index 796e42d..7627e21 100644 (file)
@@ -76,7 +76,6 @@ namespace WebCore {
 WTF_MAKE_ISO_ALLOCATED_IMPL(HTMLElement);
 
 using namespace HTMLNames;
-using namespace WTF;
 
 Ref<HTMLElement> HTMLElement::create(const QualifiedName& tagName, Document& document)
 {
index 8d6c347..3cab734 100644 (file)
@@ -39,7 +39,6 @@ namespace WebCore {
 WTF_MAKE_ISO_ALLOCATED_IMPL(HTMLFontElement);
 
 using namespace HTMLNames;
-using namespace WTF;
 
 HTMLFontElement::HTMLFontElement(const QualifiedName& tagName, Document& document)
     : HTMLElement(tagName, document)
index 8ac2f85..154dece 100644 (file)
@@ -35,7 +35,6 @@
 
 
 namespace WebCore {
-using namespace WTF;
 
 using namespace HTMLNames;
 
index 746ac2e..ff5f41a 100644 (file)
@@ -35,7 +35,6 @@
 #include "HTMLMediaElement.h"
 
 namespace WebCore {
-using namespace WTF;
 
 static int s_uniqueId = 0;
 
index 7775389..482d9bc 100644 (file)
@@ -34,7 +34,7 @@
 
 
 namespace WebCore {
-using namespace WTF;
+
 #if OS(WINDOWS) && !defined(gmtime_r)
 #define gmtime_r(x, y) gmtime_s((y), (x))
 #endif
index 0a11235..ea5dbfb 100644 (file)
@@ -34,7 +34,6 @@
 
 
 namespace WebCore {
-using namespace WTF;
 
 using namespace HTMLNames;
 
index 366c583..025e555 100644 (file)
 #include "QuickLook.h"
 #endif
 
-
 #define RELEASE_LOG_IF_ALLOWED(fmt, ...) RELEASE_LOG_IF(cachedResourceLoader.isAlwaysOnLoggingAllowed(), Network, "%p - CachedResource::" fmt, this, ##__VA_ARGS__)
 
 namespace WebCore {
-using namespace WTF;
 
 ResourceLoadPriority CachedResource::defaultPriorityForResourceType(Type type)
 {
@@ -115,7 +113,7 @@ static Seconds deadDecodedDataDeletionIntervalForResourceType(CachedResource::Ty
     return MemoryCache::singleton().deadDecodedDataDeletionInterval();
 }
 
-DEFINE_DEBUG_ONLY_GLOBAL(RefCountedLeakCounter, cachedResourceLeakCounter, ("CachedResource"));
+DEFINE_DEBUG_ONLY_GLOBAL(WTF::RefCountedLeakCounter, cachedResourceLeakCounter, ("CachedResource"));
 
 CachedResource::CachedResource(CachedResourceRequest&& request, Type type, const PAL::SessionID& sessionID, const CookieJar* cookieJar)
     : m_options(request.options())
index c775dee..c2cf0c2 100644 (file)
@@ -73,8 +73,8 @@
 
 
 namespace WebCore {
-using namespace WTF;
-using namespace Unicode;
+
+using namespace WTF::Unicode;
 
 ContextMenuController::ContextMenuController(Page& page, ContextMenuClient& client)
     : m_page(page)
index 91e3f4d..6e2faec 100644 (file)
@@ -47,7 +47,6 @@
 #include <wtf/WeakPtr.h>
 
 namespace WebCore {
-using namespace WTF;
 
 Navigator::Navigator(ScriptExecutionContext* context, DOMWindow& window)
     : NavigatorBase(context)
index 2982852..1d048f5 100644 (file)
@@ -37,7 +37,6 @@
 
 
 namespace WebCore {
-using namespace WTF;
 
 static Length parseLength(const UChar* data, unsigned length)
 {
index 05c0a8c..b8f5854 100644 (file)
@@ -37,8 +37,6 @@
 #import "KeyEventCodesIOS.h"
 #endif
 
-using namespace WTF;
-
 namespace WebCore {
 
 bool PlatformKeyboardEvent::currentCapsLockState()
index b3b6468..b99179a 100644 (file)
@@ -46,8 +46,8 @@
 #endif
 
 namespace WebCore {
-using namespace WTF;
-using namespace Unicode;
+
+using namespace WTF::Unicode;
 
 static bool useBackslashAsYenSignForFamily(const AtomicString& family)
 {
index 2c6d951..2d0972a 100644 (file)
@@ -32,8 +32,8 @@
 
 
 namespace WebCore {
-using namespace WTF;
-using namespace Unicode;
+
+using namespace WTF::Unicode;
 
 WidthIterator::WidthIterator(const FontCascade* font, const TextRun& run, HashSet<const Font*>* fallbackFonts, bool accountForGlyphBounds, bool forTextEmphasis)
     : m_font(font)
index 9bd57e7..33c95f3 100644 (file)
@@ -36,8 +36,6 @@
 #import <pal/spi/cocoa/IOKitSPI.h>
 #import <wtf/MainThread.h>
 
-using namespace WTF;
-
 namespace WebCore {
 
 int windowsKeyCodeForKeyCode(uint16_t keyCode)
index 2bcb510..c73f4ab 100644 (file)
@@ -35,7 +35,6 @@
 #import <wtf/MainThread.h>
 
 namespace WebCore {
-using namespace WTF;
 
 int windowsKeyCodeForKeyCode(uint16_t keyCode)
 {
index 333631b..7fa2169 100644 (file)
@@ -44,7 +44,6 @@
 
 
 namespace WebCore {
-using namespace WTF;
 
 // true if there is more to parse, after incrementing pos past whitespace.
 // Note: Might return pos == str.length()
index 21f038b..a3dad4e 100644 (file)
@@ -33,7 +33,7 @@
 #include <wtf/unicode/CharacterNames.h>
 
 namespace WebCore {
-using namespace WTF;
+
 using namespace WTF::Unicode;
 
 const int nonCharacter = -1;
index 4f43f9b..5444f02 100644 (file)
@@ -45,7 +45,6 @@
 #include <wtf/text/CString.h>
 
 namespace WebCore {
-using namespace WTF;
 
 const size_t maxEncodingNameLength = 63;
 
index a5da187..da903a1 100644 (file)
@@ -33,9 +33,7 @@
 #define MAPVK_VSC_TO_VK_EX 3
 #endif
 
-
 namespace WebCore {
-using namespace WTF;
 
 static const unsigned short HIGH_BIT_MASK_SHORT = 0x8000;
 
index 4aecd19..a4e7583 100644 (file)
 #include <wtf/RefCountedLeakCounter.h>
 #include <wtf/StdLibExtras.h>
 
-
 namespace WebCore {
-using namespace WTF;
 
-DEFINE_DEBUG_ONLY_GLOBAL(RefCountedLeakCounter, bidiRunCounter, ("BidiRun"));
+DEFINE_DEBUG_ONLY_GLOBAL(WTF::RefCountedLeakCounter, bidiRunCounter, ("BidiRun"));
 
 BidiRun::BidiRun(unsigned start, unsigned stop, RenderObject& renderer, BidiContext* context, UCharDirection dir)
     : BidiCharacterRun(start, stop, context, dir)
index 35ec17c..1115d88 100644 (file)
@@ -28,9 +28,7 @@
 #include "RenderBox.h"
 #include "RenderView.h"
 
-
 namespace WebCore {
-using namespace WTF;
 
 struct SameSizeAsFloatingObject {
     void* pointers[2];
index 3909c5a..5d35202 100644 (file)
 #include "ShadowRoot.h"
 #include "ShapeOutsideInfo.h"
 #include "TransformState.h"
-
 #include <wtf/IsoMallocInlines.h>
 #include <wtf/NeverDestroyed.h>
 #include <wtf/Optional.h>
 #include <wtf/SetForScope.h>
 #include <wtf/StackStats.h>
 
-
 namespace WebCore {
-using namespace WTF;
-using namespace Unicode;
 
 using namespace HTMLNames;
+using namespace WTF::Unicode;
 
 WTF_MAKE_ISO_ALLOCATED_IMPL(RenderBlock);
 
index 495e20f..32e1a86 100644 (file)
 #include <wtf/text/StringBuilder.h>
 #include <wtf/unicode/CharacterNames.h>
 
-
 namespace WebCore {
-using namespace WTF;
-using namespace Unicode;
+
+using namespace WTF::Unicode;
 
 WTF_MAKE_ISO_ALLOCATED_IMPL(RenderListMarker);
 
index b675cc8..b2e07f2 100644 (file)
 #include "SelectionRect.h"
 #endif
 
-
 namespace WebCore {
-using namespace WTF;
-using namespace Unicode;
+
+using namespace WTF::Unicode;
 
 WTF_MAKE_ISO_ALLOCATED_IMPL(RenderText);
 
index cf9e56f..d8e226f 100644 (file)
@@ -1,3 +1,12 @@
+2019-01-28  Ross Kirsling  <ross.kirsling@sony.com>
+
+        Remove unnecessary `using namespace WTF`s (or at least restrict their scope).
+        https://bugs.webkit.org/show_bug.cgi?id=193941
+
+        Reviewed by Alex Christensen.
+
+        * Shared/linux/WebMemorySamplerLinux.cpp:
+
 2019-01-28  Fujii Hironori  <Hironori.Fujii@sony.com>
 
         [Win] WebCore/platform/Process.h is conflicting with process.h
index aca0a0a..7f00eff 100644 (file)
@@ -41,9 +41,9 @@
 #include <wtf/text/WTFString.h>
 
 namespace WebKit {
-using namespace WebCore;
+
 using namespace JSC;
-using namespace WTF;
+using namespace WebCore;
 
 static const unsigned int maxBuffer = 128;
 static const unsigned int maxProcessPath = 35;
@@ -113,7 +113,7 @@ WebMemoryStatistics WebMemorySampler::sampleWebKit() const
     size_t totalBytesInUse = 0;
     size_t totalBytesCommitted = 0;
 
-    FastMallocStatistics fastMallocStatistics = WTF::fastMallocStatistics();
+    auto fastMallocStatistics = WTF::fastMallocStatistics();
     size_t fastMallocBytesInUse = fastMallocStatistics.committedVMBytes - fastMallocStatistics.freeListBytes;
     size_t fastMallocBytesCommitted = fastMallocStatistics.committedVMBytes;
     totalBytesInUse += fastMallocBytesInUse;
index 5acadf1..bdc4fe1 100644 (file)
@@ -1,3 +1,14 @@
+2019-01-28  Ross Kirsling  <ross.kirsling@sony.com>
+
+        Remove unnecessary `using namespace WTF`s (or at least restrict their scope).
+        https://bugs.webkit.org/show_bug.cgi?id=193941
+
+        Reviewed by Alex Christensen.
+
+        * Plugins/PluginMessageThrottlerWin.cpp:
+        * Plugins/PluginView.cpp:
+        * Plugins/PluginViewWin.cpp:
+
 2019-01-24  Ross Kirsling  <ross.kirsling@sony.com>
 
         Move FileSystem to WTF
index 8b5df5d..cdbe3a7 100644 (file)
@@ -30,8 +30,6 @@
 #include <wtf/ASCIICType.h>
 #include <wtf/MonotonicTime.h>
 
-using namespace WTF;
-
 namespace WebCore {
 
 // Set a timer to make sure we process any queued messages at least every 16ms.
index 3b21cf1..2136317 100644 (file)
@@ -88,8 +88,6 @@ using JSC::JSValue;
 
 #if ENABLE(NETSCAPE_PLUGIN_API)
 
-using namespace WTF;
-
 namespace WebCore {
 
 using namespace HTMLNames;
index 49b9ad2..4a663e0 100644 (file)
@@ -84,16 +84,10 @@ static inline HWND windowHandleForPageClient(PlatformPageClient client)
     return client;
 }
 
-using JSC::ExecState;
-using JSC::JSLock;
-using JSC::JSObject;
-
-using std::min;
-
-using namespace WTF;
-
 namespace WebCore {
 
+using JSC::JSLock;
+
 using namespace HTMLNames;
 
 const LPCWSTR kWebPluginViewClassName = L"WebPluginView";
index 8b58036..8cf0c9c 100644 (file)
@@ -1,3 +1,23 @@
+2019-01-28  Ross Kirsling  <ross.kirsling@sony.com>
+
+        Remove unnecessary `using namespace WTF`s (or at least restrict their scope).
+        https://bugs.webkit.org/show_bug.cgi?id=193941
+
+        Reviewed by Alex Christensen.
+
+        * TestWebKitAPI/Tests/WTF/ConcurrentPtrHashSet.cpp:
+        * TestWebKitAPI/Tests/WTF/Condition.cpp:
+        * TestWebKitAPI/Tests/WTF/Lock.cpp:
+        * TestWebKitAPI/Tests/WTF/MetaAllocator.cpp:
+        * TestWebKitAPI/Tests/WTF/ParkingLot.cpp:
+        * TestWebKitAPI/Tests/WTF/RedBlackTree.cpp:
+        * TestWebKitAPI/Tests/WTF/ScopedLambda.cpp:
+        * TestWebKitAPI/Tests/WTF/Time.cpp:
+        * TestWebKitAPI/Tests/WTF/UniqueArray.cpp:
+        * TestWebKitAPI/Tests/WebCore/ApplicationManifestParser.cpp:
+        * TestWebKitAPI/Tests/WebCore/Logging.cpp:
+        * WebKitTestRunner/InjectedBundle/cocoa/InjectedBundlePageCocoa.mm:
+
 2019-01-28  Yusuke Suzuki  <ysuzuki@apple.com>
 
         Unreviewed, reorder my emails to fix autocomplete in bugzilla
index 679a155..489a07d 100644 (file)
@@ -26,8 +26,6 @@
 #include "config.h"
 #include <wtf/ConcurrentPtrHashSet.h>
 
-using namespace WTF;
-
 namespace TestWebKitAPI {
 
 namespace {
index 0a329c2..26c8f99 100644 (file)
@@ -34,8 +34,6 @@
 #include <wtf/Threading.h>
 #include <wtf/Vector.h>
 
-using namespace WTF;
-
 namespace TestWebKitAPI {
 
 namespace {
index 6a4e538..2dccedc 100644 (file)
@@ -29,8 +29,6 @@
 #include <wtf/ThreadingPrimitives.h>
 #include <wtf/WordLock.h>
 
-using namespace WTF;
-
 namespace TestWebKitAPI {
 
 struct LockInspector {
index 8530400..ef01b3a 100644 (file)
 #undef small
 #endif
 
-using namespace WTF;
-
 namespace TestWebKitAPI {
 
+using namespace WTF;
+
 class MetaAllocatorTest: public testing::Test {
 public:
     enum SanityCheckMode { RunSanityCheck, DontRunSanityCheck };
index a47b20c..452e227 100644 (file)
@@ -34,8 +34,6 @@
 #include <wtf/Threading.h>
 #include <wtf/ThreadingPrimitives.h>
 
-using namespace WTF;
-
 namespace TestWebKitAPI {
 
 namespace {
index 5e5c8ad..d833b2a 100644 (file)
 #include <wtf/RedBlackTree.h>
 #include <wtf/Vector.h>
 
-using namespace WTF;
-
 namespace TestWebKitAPI {
 
+using namespace WTF;
+
 class TestNode : public RedBlackTree<TestNode, char>::Node {
 public:
     TestNode(char key, unsigned value)
index 3dfee86..5b6d746 100644 (file)
@@ -27,8 +27,6 @@
 #include <wtf/ScopedLambda.h>
 #include <wtf/Vector.h>
 
-using namespace WTF;
-
 namespace TestWebKitAPI {
 
 // This test relies on this module being compiled with -fno-elide-constructors
index d7fdb6e..189e902 100644 (file)
@@ -59,8 +59,6 @@ std::basic_ostream<char>& operator<<(std::basic_ostream<char>& out, TimeWithDyna
 
 } // namespace WTF
 
-using namespace WTF;
-
 namespace TestWebKitAPI {
 
 namespace {
index c1d0836..d1fde95 100644 (file)
@@ -26,8 +26,6 @@
 #include "config.h"
 #include <wtf/UniqueArray.h>
 
-using namespace WTF;
-
 namespace TestWebKitAPI {
 
 static unsigned numberOfConstructions { 0 };
index e993ffb..c83bb6f 100644 (file)
@@ -31,7 +31,6 @@
 #include <WebCore/ApplicationManifestParser.h>
 #include <wtf/RunLoop.h>
 
-using namespace WTF;
 using namespace WebCore;
 
 namespace WebCore {
index 1e4c342..0c15a6d 100644 (file)
@@ -31,8 +31,6 @@
 
 #define LOG_CHANNEL_PREFIX Test
 
-using namespace WTF;
-
 const char* logTestingSubsystem = "com.webkit.testing";
 
 DEFINE_LOG_CHANNEL(Channel1, logTestingSubsystem);
index 6e2edbc..8176182 100644 (file)
@@ -35,8 +35,6 @@
 
 namespace WTR {
 
-using namespace WTF;
-
 void InjectedBundlePage::platformDidStartProvisionalLoadForFrame(WKBundleFrameRef frame)
 {
     if (!WKBundleFrameIsMainFrame(frame))