Drop RefPtr::clear() method
authorcdumez@apple.com <cdumez@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sat, 4 Jul 2015 19:42:18 +0000 (19:42 +0000)
committercdumez@apple.com <cdumez@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sat, 4 Jul 2015 19:42:18 +0000 (19:42 +0000)
https://bugs.webkit.org/show_bug.cgi?id=146556

Reviewed by Brady Eidson.

Source/JavaScriptCore:

Drop RefPtr::clear() method in favor of "= nullptr;" pattern.

Source/WebCore:

Drop RefPtr::clear() method in favor of "= nullptr;" pattern.

Source/WebKit:

Drop RefPtr::clear() method in favor of "= nullptr;" pattern.

Source/WebKit/mac:

Drop RefPtr::clear() method in favor of "= nullptr;" pattern.

Source/WebKit/win:

Drop RefPtr::clear() method in favor of "= nullptr;" pattern.

Source/WebKit2:

Drop RefPtr::clear() method in favor of "= nullptr;" pattern.

Source/WTF:

Drop RefPtr::clear() method in favor of "= nullptr;" pattern.
Also made the "= nullptr;" pattern as efficient as clear()
by providing a operator=(nullptr_t) overload. Local micro-
benchmarking showed that "= nullptr;" used to be ~1.7% slower
than clear().

Tools:

Drop RefPtr::clear() method in favor of "= nullptr;" pattern.

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

199 files changed:
Source/JavaScriptCore/API/JSValueRef.cpp
Source/JavaScriptCore/ChangeLog
Source/JavaScriptCore/bytecode/CallLinkInfo.cpp
Source/JavaScriptCore/bytecode/StructureStubInfo.h
Source/JavaScriptCore/debugger/Debugger.cpp
Source/JavaScriptCore/dfg/DFGGraph.h
Source/JavaScriptCore/dfg/DFGOperations.cpp
Source/JavaScriptCore/inspector/ConsoleMessage.cpp
Source/JavaScriptCore/inspector/JSGlobalObjectInspectorController.cpp
Source/JavaScriptCore/inspector/agents/InspectorAgent.cpp
Source/JavaScriptCore/inspector/agents/InspectorConsoleAgent.cpp
Source/JavaScriptCore/inspector/agents/InspectorDebuggerAgent.cpp
Source/JavaScriptCore/inspector/agents/JSGlobalObjectRuntimeAgent.cpp
Source/JavaScriptCore/parser/Parser.cpp
Source/JavaScriptCore/runtime/Executable.cpp
Source/JavaScriptCore/runtime/JSLock.cpp
Source/JavaScriptCore/runtime/Structure.cpp
Source/WTF/ChangeLog
Source/WTF/wtf/RefPtr.h
Source/WTF/wtf/text/StringBuilder.h
Source/WebCore/ChangeLog
Source/WebCore/Modules/indexeddb/IDBCursor.cpp
Source/WebCore/Modules/indexeddb/IDBDatabaseBackend.cpp
Source/WebCore/Modules/indexeddb/IDBObjectStore.cpp
Source/WebCore/Modules/indexeddb/IDBOpenDBRequest.cpp
Source/WebCore/Modules/indexeddb/IDBRequest.cpp
Source/WebCore/Modules/indexeddb/IDBTransaction.cpp
Source/WebCore/Modules/mediasource/MediaSource.cpp
Source/WebCore/Modules/mediastream/RTCSessionDescriptionRequestImpl.cpp
Source/WebCore/Modules/mediastream/RTCStatsRequestImpl.cpp
Source/WebCore/Modules/mediastream/RTCVoidRequestImpl.cpp
Source/WebCore/Modules/webaudio/AudioContext.cpp
Source/WebCore/Modules/webdatabase/SQLTransactionBackend.cpp
Source/WebCore/Modules/websockets/WorkerThreadableWebSocketChannel.cpp
Source/WebCore/accessibility/atk/AXObjectCacheAtk.cpp
Source/WebCore/bindings/js/ScriptController.cpp
Source/WebCore/bindings/js/WebCoreJSClientData.h
Source/WebCore/bindings/js/WorkerScriptController.cpp
Source/WebCore/bridge/runtime_object.cpp
Source/WebCore/contentextensions/ContentExtensionParser.cpp
Source/WebCore/css/CSSParser.cpp
Source/WebCore/css/CSSValuePool.cpp
Source/WebCore/css/StyleProperties.cpp
Source/WebCore/css/ViewportStyleResolver.cpp
Source/WebCore/dom/Document.cpp
Source/WebCore/dom/DocumentMarker.h
Source/WebCore/dom/DocumentStyleSheetCollection.cpp
Source/WebCore/dom/Element.cpp
Source/WebCore/dom/MutationObserverRegistration.cpp
Source/WebCore/dom/NodeIterator.cpp
Source/WebCore/dom/Position.h
Source/WebCore/dom/ProcessingInstruction.cpp
Source/WebCore/dom/RangeBoundaryPoint.h
Source/WebCore/dom/SpaceSplitString.h
Source/WebCore/dom/StyledElement.cpp
Source/WebCore/dom/default/PlatformMessagePortChannel.cpp
Source/WebCore/editing/AlternativeTextController.cpp
Source/WebCore/editing/ApplyBlockElementCommand.cpp
Source/WebCore/editing/DeleteSelectionCommand.cpp
Source/WebCore/editing/EditingStyle.cpp
Source/WebCore/editing/Editor.cpp
Source/WebCore/editing/FrameSelection.cpp
Source/WebCore/editing/FrameSelection.h
Source/WebCore/editing/IndentOutdentCommand.cpp
Source/WebCore/editing/ReplaceSelectionCommand.cpp
Source/WebCore/editing/SpellChecker.cpp
Source/WebCore/editing/TextCheckingHelper.cpp
Source/WebCore/fileapi/FileReader.cpp
Source/WebCore/fileapi/FileReaderLoader.cpp
Source/WebCore/history/HistoryItem.cpp
Source/WebCore/html/BaseChooserOnlyDateAndTimeInputType.cpp
Source/WebCore/html/FTPDirectoryDocument.cpp
Source/WebCore/html/FileInputType.cpp
Source/WebCore/html/HTMLCanvasElement.cpp
Source/WebCore/html/HTMLLinkElement.cpp
Source/WebCore/html/HTMLMediaElement.cpp
Source/WebCore/html/HTMLPlugInElement.cpp
Source/WebCore/html/HTMLTableElement.cpp
Source/WebCore/html/PluginDocument.cpp
Source/WebCore/html/TextFieldInputType.cpp
Source/WebCore/html/ValidationMessage.cpp
Source/WebCore/html/canvas/WebGLProgram.cpp
Source/WebCore/html/canvas/WebGLRenderingContextBase.cpp
Source/WebCore/html/canvas/WebGLVertexArrayObjectBase.cpp
Source/WebCore/html/track/TextTrack.cpp
Source/WebCore/html/track/VTTCue.cpp
Source/WebCore/inspector/InspectorApplicationCacheAgent.cpp
Source/WebCore/inspector/InspectorCSSAgent.cpp
Source/WebCore/inspector/InspectorController.cpp
Source/WebCore/inspector/InspectorDOMAgent.cpp
Source/WebCore/inspector/InspectorDOMDebuggerAgent.cpp
Source/WebCore/inspector/InspectorDOMStorageAgent.cpp
Source/WebCore/inspector/InspectorDatabaseAgent.cpp
Source/WebCore/inspector/InspectorIndexedDBAgent.cpp
Source/WebCore/inspector/InspectorLayerTreeAgent.cpp
Source/WebCore/inspector/InspectorOverlay.cpp
Source/WebCore/inspector/InspectorPageAgent.cpp
Source/WebCore/inspector/InspectorReplayAgent.cpp
Source/WebCore/inspector/InspectorResourceAgent.cpp
Source/WebCore/inspector/InspectorStyleSheet.cpp
Source/WebCore/inspector/InspectorTimelineAgent.cpp
Source/WebCore/inspector/InspectorWorkerAgent.cpp
Source/WebCore/inspector/PageRuntimeAgent.cpp
Source/WebCore/inspector/WebInjectedScriptManager.cpp
Source/WebCore/inspector/WorkerInspectorController.cpp
Source/WebCore/inspector/WorkerRuntimeAgent.cpp
Source/WebCore/loader/DocumentLoader.cpp
Source/WebCore/loader/DocumentThreadableLoader.cpp
Source/WebCore/loader/DocumentWriter.cpp
Source/WebCore/loader/FrameLoader.cpp
Source/WebCore/loader/ProgressTracker.cpp
Source/WebCore/loader/ResourceLoader.cpp
Source/WebCore/loader/appcache/ApplicationCacheGroup.cpp
Source/WebCore/loader/cache/CachedCSSStyleSheet.cpp
Source/WebCore/loader/cache/CachedImage.cpp
Source/WebCore/loader/cache/CachedRawResource.cpp
Source/WebCore/loader/cache/CachedResource.cpp
Source/WebCore/loader/icon/IconRecord.cpp
Source/WebCore/page/EventHandler.cpp
Source/WebCore/page/Page.cpp
Source/WebCore/platform/audio/mac/CARingBuffer.cpp
Source/WebCore/platform/graphics/Font.cpp
Source/WebCore/platform/graphics/FontCache.cpp
Source/WebCore/platform/graphics/GraphicsContext.cpp
Source/WebCore/platform/graphics/MediaPlayer.cpp
Source/WebCore/platform/graphics/ca/GraphicsLayerCA.cpp
Source/WebCore/platform/graphics/cairo/BackingStoreBackendCairoX11.cpp
Source/WebCore/platform/graphics/cairo/BitmapImageCairo.cpp
Source/WebCore/platform/graphics/filters/FilterEffect.cpp
Source/WebCore/platform/graphics/gstreamer/ImageGStreamerCairo.cpp
Source/WebCore/platform/graphics/gstreamer/WebKitWebSourceGStreamer.cpp
Source/WebCore/platform/graphics/texmap/BitmapTextureGL.cpp
Source/WebCore/platform/graphics/texmap/GraphicsLayerTextureMapper.cpp
Source/WebCore/platform/graphics/texmap/TextureMapperGL.cpp
Source/WebCore/platform/graphics/texmap/TextureMapperLayer.cpp
Source/WebCore/platform/graphics/texmap/TextureMapperTiledBackingStore.cpp
Source/WebCore/platform/graphics/texmap/coordinated/CoordinatedGraphicsLayer.cpp
Source/WebCore/platform/graphics/texmap/coordinated/CoordinatedImageBacking.cpp
Source/WebCore/platform/gtk/GamepadsGtk.cpp
Source/WebCore/platform/ios/PlatformSpeechSynthesizerIOS.mm
Source/WebCore/platform/mock/GeolocationClientMock.cpp
Source/WebCore/platform/mock/PlatformSpeechSynthesizerMock.cpp
Source/WebCore/platform/network/cf/ResourceRequestCFNet.cpp
Source/WebCore/rendering/InlineFlowBox.cpp
Source/WebCore/rendering/RenderBox.cpp
Source/WebCore/rendering/RenderBoxRegionInfo.h
Source/WebCore/rendering/RenderCounter.cpp
Source/WebCore/rendering/RenderMenuList.cpp
Source/WebCore/rendering/RenderSearchField.cpp
Source/WebCore/rendering/style/FillLayer.h
Source/WebCore/rendering/style/RenderStyle.cpp
Source/WebCore/svg/SVGTRefElement.cpp
Source/WebCore/testing/Internals.cpp
Source/WebCore/workers/WorkerEventQueue.cpp
Source/WebCore/workers/WorkerMessagingProxy.cpp
Source/WebCore/workers/WorkerThread.cpp
Source/WebCore/xml/XMLHttpRequest.cpp
Source/WebCore/xml/XSLTProcessor.cpp
Source/WebKit/ChangeLog
Source/WebKit/Storage/StorageAreaSync.cpp
Source/WebKit/mac/ChangeLog
Source/WebKit/mac/Plugins/Hosted/WebHostedNetscapePluginView.mm
Source/WebKit/mac/WebCoreSupport/WebEditorClient.mm
Source/WebKit/mac/WebCoreSupport/WebUserMediaClient.mm
Source/WebKit/win/ChangeLog
Source/WebKit/win/Plugins/PluginStream.cpp
Source/WebKit/win/Plugins/PluginView.cpp
Source/WebKit/win/WebFramePolicyListener.cpp
Source/WebKit/win/WebView.cpp
Source/WebKit2/ChangeLog
Source/WebKit2/DatabaseProcess/IndexedDB/UniqueIDBDatabase.cpp
Source/WebKit2/PluginProcess/PluginControllerProxy.cpp
Source/WebKit2/Shared/ChildProcessProxy.cpp
Source/WebKit2/Shared/CoordinatedGraphics/CoordinatedBackingStore.cpp
Source/WebKit2/Shared/Downloads/soup/DownloadSoup.cpp
Source/WebKit2/UIProcess/API/efl/EwkView.cpp
Source/WebKit2/UIProcess/API/gtk/WebKitFormSubmissionRequest.cpp
Source/WebKit2/UIProcess/API/mac/WKView.mm
Source/WebKit2/UIProcess/InspectorServer/WebSocketServerConnection.cpp
Source/WebKit2/UIProcess/Network/CustomProtocols/mac/CustomProtocolManagerProxyMac.mm
Source/WebKit2/UIProcess/StatisticsRequest.cpp
Source/WebKit2/UIProcess/WebFrameListenerProxy.cpp
Source/WebKit2/UIProcess/WebFrameProxy.cpp
Source/WebKit2/UIProcess/WebOpenPanelResultListenerProxy.cpp
Source/WebKit2/UIProcess/WebPageProxy.cpp
Source/WebKit2/UIProcess/ios/WKGeolocationProviderIOS.mm
Source/WebKit2/WebProcess/InjectedBundle/mac/InjectedBundleMac.mm
Source/WebKit2/WebProcess/Network/WebResourceLoader.cpp
Source/WebKit2/WebProcess/Plugins/Netscape/NetscapePlugin.cpp
Source/WebKit2/WebProcess/Plugins/PDF/PDFPlugin.mm
Source/WebKit2/WebProcess/Plugins/PluginView.cpp
Source/WebKit2/WebProcess/WebPage/WebPage.cpp
Source/WebKit2/WebProcess/WebPage/gtk/WebPrintOperationGtk.cpp
Source/WebKit2/WebProcess/WebPage/ios/WebPageIOS.mm
Source/WebKit2/WebProcess/WebProcess.cpp
Tools/ChangeLog
Tools/DumpRenderTree/mac/DumpRenderTree.mm
Tools/DumpRenderTree/win/DumpRenderTree.cpp
Tools/TestWebKitAPI/Tests/WTF/RefPtr.cpp

index 759a65a..54405e2 100644 (file)
@@ -422,7 +422,7 @@ JSStringRef JSValueToStringCopy(JSContextRef ctx, JSValueRef value, JSValueRef*
     
     RefPtr<OpaqueJSString> stringRef(OpaqueJSString::create(jsValue.toString(exec)->value(exec)));
     if (handleExceptionIfNeeded(exec, exception) == ExceptionStatus::DidThrow)
-        stringRef.clear();
+        stringRef = nullptr;
     return stringRef.release().leakRef();
 }
 
index 28572f4..017dccc 100644 (file)
@@ -1,3 +1,12 @@
+2015-07-04  Chris Dumez  <cdumez@apple.com>
+
+        Drop RefPtr::clear() method
+        https://bugs.webkit.org/show_bug.cgi?id=146556
+
+        Reviewed by Brady Eidson.
+
+        Drop RefPtr::clear() method in favor of "= nullptr;" pattern.
+
 2015-07-03  Dan Bernstein  <mitz@apple.com>
 
         Just give up on -Wunreachable-code in JavaScriptCore.
index 72a0c24..d920098 100644 (file)
@@ -43,7 +43,7 @@ void CallLinkInfo::clearStub()
         return;
 
     m_stub->clearCallNodesFor(this);
-    m_stub.clear();
+    m_stub = nullptr;
 }
 
 void CallLinkInfo::unlink(RepatchBuffer& repatchBuffer)
index ec8b79c..d402ace 100644 (file)
@@ -149,8 +149,8 @@ struct StructureStubInfo {
     {
         deref();
         accessType = access_unset;
-        stubRoutine.clear();
-        watchpoints.clear();
+        stubRoutine = nullptr;
+        watchpoints = nullptr;
     }
 
     void deref();
index 5741069..bc1b57e 100644 (file)
@@ -117,7 +117,7 @@ public:
     {
         if (m_debugger.m_currentDebuggerCallFrame) {
             m_debugger.m_currentDebuggerCallFrame->invalidate();
-            m_debugger.m_currentDebuggerCallFrame = 0;
+            m_debugger.m_currentDebuggerCallFrame = nullptr;
         }
     }
 
index 2ce7d60..5148c26 100644 (file)
@@ -444,7 +444,7 @@ public:
     
     void killBlock(BlockIndex blockIndex)
     {
-        m_blocks[blockIndex].clear();
+        m_blocks[blockIndex] = nullptr;
     }
     
     void killBlock(BasicBlock* basicBlock)
index a99a3fa..6f9512b 100644 (file)
@@ -1447,7 +1447,7 @@ char* JIT_OPERATION triggerOSREntryNow(
         
         // OSR entry failed. Oh no! This implies that we need to retry. We retry
         // without exponential backoff and we only do this for the entry code block.
-        jitCode->osrEntryBlock.clear();
+        jitCode->osrEntryBlock = nullptr;
         jitCode->osrEntryRetry = 0;
         return 0;
     }
index a04a97d..9ac0dde 100644 (file)
@@ -273,7 +273,7 @@ void ConsoleMessage::clear()
         m_message = ASCIILiteral("<message collected>");
 
     if (m_arguments)
-        m_arguments.clear();
+        m_arguments = nullptr;
 }
 
 JSC::ExecState* ConsoleMessage::scriptState() const
index 57f52c5..11d86ed 100644 (file)
@@ -131,7 +131,7 @@ void JSGlobalObjectInspectorController::disconnectFrontend(DisconnectReason reas
     m_agents.willDestroyFrontendAndBackend(reason);
 
     m_backendDispatcher->clearFrontend();
-    m_backendDispatcher.clear();
+    m_backendDispatcher = nullptr;
     m_frontendChannel = nullptr;
 
     m_isAutomaticInspection = false;
index bf241ba..0538690 100644 (file)
@@ -57,7 +57,7 @@ void InspectorAgent::didCreateFrontendAndBackend(FrontendChannel* frontendChanne
 void InspectorAgent::willDestroyFrontendAndBackend(DisconnectReason)
 {
     m_frontendDispatcher = nullptr;
-    m_backendDispatcher.clear();
+    m_backendDispatcher = nullptr;
 
     m_pendingEvaluateTestCommands.clear();
 
index 814a0c1..9ffd048 100644 (file)
@@ -64,7 +64,7 @@ void InspectorConsoleAgent::didCreateFrontendAndBackend(FrontendChannel* fronten
 void InspectorConsoleAgent::willDestroyFrontendAndBackend(DisconnectReason)
 {
     m_frontendDispatcher = nullptr;
-    m_backendDispatcher.clear();
+    m_backendDispatcher = nullptr;
 
     String errorString;
     disable(errorString);
index a75e0b2..f0f0b48 100644 (file)
@@ -77,7 +77,7 @@ void InspectorDebuggerAgent::didCreateFrontendAndBackend(FrontendChannel* fronte
 void InspectorDebuggerAgent::willDestroyFrontendAndBackend(DisconnectReason reason)
 {
     m_frontendDispatcher = nullptr;
-    m_backendDispatcher.clear();
+    m_backendDispatcher = nullptr;
 
     bool skipRecompile = reason == DisconnectReason::InspectedTargetDestroyed;
     disable(skipRecompile);
index 658c67a..46049f5 100644 (file)
@@ -49,7 +49,7 @@ void JSGlobalObjectRuntimeAgent::didCreateFrontendAndBackend(FrontendChannel* fr
 void JSGlobalObjectRuntimeAgent::willDestroyFrontendAndBackend(DisconnectReason reason)
 {
     m_frontendDispatcher = nullptr;
-    m_backendDispatcher.clear();
+    m_backendDispatcher = nullptr;
 
     InspectorRuntimeAgent::willDestroyFrontendAndBackend(reason);
 }
index dc0f55f..a3c5e31 100644 (file)
@@ -491,7 +491,7 @@ template <class TreeBuilder> TreeExpression Parser<LexerType>::parseVarDeclarati
     JSToken lastIdentToken; 
     do {
         lastIdent = 0;
-        lastPattern = 0;
+        lastPattern = TreeDestructuringPattern(0);
         JSTokenLocation location(tokenLocation());
         next();
         TreeExpression node = 0;
@@ -894,7 +894,7 @@ template <class TreeBuilder> TreeStatement Parser<LexerType>::parseForStatement(
             declsEnd = lastTokenEndPosition();
             if (pattern && (match(INTOKEN) || (match(IDENT) && *m_token.m_data.ident == m_vm->propertyNames->of)))
                 goto enumerationLoop;
-            pattern = 0;
+            pattern = TreeDestructuringPattern(0);
             restoreSavePoint(savePoint);
         }
         m_allowsIn = false;
index 6cb09b0..55240fd 100644 (file)
@@ -52,8 +52,8 @@ void ExecutableBase::destroy(JSCell* cell)
 void ExecutableBase::clearCode()
 {
 #if ENABLE(JIT)
-    m_jitCodeForCall.clear();
-    m_jitCodeForConstruct.clear();
+    m_jitCodeForCall = nullptr;
+    m_jitCodeForConstruct = nullptr;
     m_jitCodeForCallWithArityCheck = MacroAssemblerCodePtr();
     m_jitCodeForConstructWithArityCheck = MacroAssemblerCodePtr();
     m_jitCodeForCallWithArityCheckAndPreserveRegs = MacroAssemblerCodePtr();
@@ -464,7 +464,7 @@ void EvalExecutable::unlinkCalls()
 
 void EvalExecutable::clearCode()
 {
-    m_evalCodeBlock.clear();
+    m_evalCodeBlock = nullptr;
     m_unlinkedEvalCodeBlock.clear();
     Base::clearCode();
 }
@@ -543,7 +543,7 @@ void ProgramExecutable::visitChildren(JSCell* cell, SlotVisitor& visitor)
 
 void ProgramExecutable::clearCode()
 {
-    m_programCodeBlock.clear();
+    m_programCodeBlock = nullptr;
     m_unlinkedProgramCodeBlock.clear();
     Base::clearCode();
 }
@@ -587,8 +587,8 @@ void FunctionExecutable::clearUnlinkedCodeForRecompilation()
 
 void FunctionExecutable::clearCode()
 {
-    m_codeBlockForCall.clear();
-    m_codeBlockForConstruct.clear();
+    m_codeBlockForCall = nullptr;
+    m_codeBlockForConstruct = nullptr;
     Base::clearCode();
 }
 
index 2e7994c..c2ee0a5 100644 (file)
@@ -73,7 +73,7 @@ void JSLockHolder::init()
 JSLockHolder::~JSLockHolder()
 {
     RefPtr<JSLock> apiLock(&m_vm->apiLock());
-    m_vm.clear();
+    m_vm = nullptr;
     apiLock->unlock();
 }
 
index f771b60..3985805 100644 (file)
@@ -737,7 +737,7 @@ void Structure::pin()
     ASSERT(propertyTable());
     setIsPinnedPropertyTable(true);
     clearPreviousID();
-    m_nameInPrevious.clear();
+    m_nameInPrevious = nullptr;
 }
 
 void Structure::allocateRareData(VM& vm)
index 710226a..8dc36de 100644 (file)
@@ -1,3 +1,16 @@
+2015-07-04  Chris Dumez  <cdumez@apple.com>
+
+        Drop RefPtr::clear() method
+        https://bugs.webkit.org/show_bug.cgi?id=146556
+
+        Reviewed by Brady Eidson.
+
+        Drop RefPtr::clear() method in favor of "= nullptr;" pattern.
+        Also made the "= nullptr;" pattern as efficient as clear()
+        by providing a operator=(nullptr_t) overload. Local micro-
+        benchmarking showed that "= nullptr;" used to be ~1.7% slower
+        than clear().
+
 2015-07-03  Dan Bernstein  <mitz@apple.com>
 
         [Xcode] Update some build settings as recommended by Xcode 7
index 2911f0d..7c8305e 100644 (file)
@@ -60,7 +60,6 @@ public:
 
     T* get() const { return m_ptr; }
     
-    void clear();
     PassRefPtr<T> release() { PassRefPtr<T> tmp = adoptRef(m_ptr); m_ptr = nullptr; return tmp; }
     Ref<T> releaseNonNull() { ASSERT(m_ptr); Ref<T> tmp(adoptRef(*m_ptr)); m_ptr = nullptr; return tmp; }
 
@@ -77,6 +76,7 @@ public:
     
     RefPtr& operator=(const RefPtr&);
     RefPtr& operator=(T*);
+    RefPtr& operator=(std::nullptr_t);
     RefPtr& operator=(const PassRefPtr<T>&);
     template<typename U> RefPtr& operator=(const RefPtr<U>&);
     template<typename U> RefPtr& operator=(const PassRefPtr<U>&);
@@ -109,11 +109,6 @@ template<typename T> template<typename U> inline RefPtr<T>::RefPtr(Ref<U>&& refe
 {
 }
 
-template<typename T> inline void RefPtr<T>::clear()
-{
-    derefIfNotNull(std::exchange(m_ptr, nullptr));
-}
-
 template<typename T>
 inline T* RefPtr<T>::leakRef()
 {
@@ -141,6 +136,12 @@ template<typename T> inline RefPtr<T>& RefPtr<T>::operator=(T* optr)
     return *this;
 }
 
+template<typename T> inline RefPtr<T>& RefPtr<T>::operator=(std::nullptr_t)
+{
+    derefIfNotNull(std::exchange(m_ptr, nullptr));
+    return *this;
+}
+
 template<typename T> inline RefPtr<T>& RefPtr<T>::operator=(const PassRefPtr<T>& o)
 {
     RefPtr ptr = o;
index be5c117..1dc4ac6 100644 (file)
@@ -264,7 +264,7 @@ public:
     {
         m_length = 0;
         m_string = String();
-        m_buffer = 0;
+        m_buffer = nullptr;
         m_bufferCharacters8 = 0;
         m_is8Bit = true;
     }
index 679e6f4..a57cef9 100644 (file)
@@ -1,3 +1,12 @@
+2015-07-04  Chris Dumez  <cdumez@apple.com>
+
+        Drop RefPtr::clear() method
+        https://bugs.webkit.org/show_bug.cgi?id=146556
+
+        Reviewed by Brady Eidson.
+
+        Drop RefPtr::clear() method in favor of "= nullptr;" pattern.
+
 2015-07-03  Dan Bernstein  <mitz@apple.com>
 
         Fixed the iOS 8 build.
index a20e1e1..c4011dc 100644 (file)
@@ -259,7 +259,7 @@ void IDBCursor::close()
     m_transactionNotifier.cursorFinished();
     if (m_request) {
         m_request->finishCursor();
-        m_request.clear();
+        m_request = nullptr;
     }
 }
 
index fc23102..09e4e27 100644 (file)
@@ -333,7 +333,7 @@ void IDBDatabaseBackend::transactionFinished(IDBTransactionBackend* rawTransacti
     m_transactions.remove(transaction->id());
     if (transaction->mode() == IndexedDB::TransactionMode::VersionChange) {
         ASSERT(transaction.get() == m_runningVersionChangeTransaction.get());
-        m_runningVersionChangeTransaction.clear();
+        m_runningVersionChangeTransaction = nullptr;
     }
 }
 
index e8bf9dd..a6d2814 100644 (file)
@@ -337,7 +337,7 @@ private:
             // Now that we are done indexing, tell the backend to go
             // back to processing tasks of type NormalTask.
             m_databaseBackend->setIndexesReady(m_transactionId, m_objectStoreId, indexIds);
-            m_databaseBackend.clear();
+            m_databaseBackend = nullptr;
         }
 
     }
index 637807e..71579b0 100644 (file)
@@ -142,7 +142,7 @@ bool IDBOpenDBRequest::dispatchEvent(PassRefPtr<Event> event)
     // If the connection closed between onUpgradeNeeded and the delivery of the "success" event,
     // an "error" event should be fired instead.
     if (event->type() == eventNames().successEvent && m_result->type() == IDBAny::IDBDatabaseType && m_result->idbDatabase()->isClosePending()) {
-        m_result.clear();
+        m_result = nullptr;
         onError(IDBDatabaseError::create(IDBDatabaseException::AbortError, "The connection was closed."));
         return false;
     }
index 9e501b4..864de53 100644 (file)
@@ -170,9 +170,9 @@ void IDBRequest::abort()
     m_enqueuedEvents.clear();
 
     m_errorCode = 0;
-    m_error.clear();
+    m_error = nullptr;
     m_errorMessage = String();
-    m_result.clear();
+    m_result = nullptr;
     onError(IDBDatabaseError::create(IDBDatabaseException::AbortError));
     m_requestAborted = true;
 }
@@ -194,10 +194,10 @@ void IDBRequest::setPendingCursor(PassRefPtr<IDBCursor> cursor)
     ASSERT(cursor == getResultCursor());
 
     m_pendingCursor = cursor;
-    m_result.clear();
+    m_result = nullptr;
     m_readyState = PENDING;
     m_errorCode = 0;
-    m_error.clear();
+    m_error = nullptr;
     m_errorMessage = String();
     m_transaction->registerRequest(this);
 }
@@ -256,7 +256,7 @@ void IDBRequest::onError(PassRefPtr<IDBDatabaseError> error)
     m_errorCode = error->code();
     m_errorMessage = error->message();
     m_error = DOMError::create(IDBDatabaseException::getErrorName(error->idbCode()));
-    m_pendingCursor.clear();
+    m_pendingCursor = nullptr;
     enqueueEvent(Event::create(eventNames().errorEvent, true, true));
 }
 
@@ -404,7 +404,7 @@ void IDBRequest::onSuccessInternal(const Deprecated::ScriptValue& value)
     m_result = IDBAny::create(value);
     if (m_pendingCursor) {
         m_pendingCursor->close();
-        m_pendingCursor.clear();
+        m_pendingCursor = nullptr;
     }
     enqueueEvent(createSuccessEvent());
 }
@@ -553,7 +553,7 @@ void IDBRequest::transactionDidFinishAndDispatch()
     ASSERT(m_transaction->isVersionChange());
     ASSERT(m_readyState == DONE);
     ASSERT(scriptExecutionContext());
-    m_transaction.clear();
+    m_transaction = nullptr;
     m_readyState = PENDING;
 }
 
index 9948bc7..6ec29fa 100644 (file)
@@ -240,7 +240,7 @@ void IDBTransaction::OpenCursorNotifier::cursorFinished()
     if (m_cursor) {
         m_transaction->unregisterOpenCursor(m_cursor);
         m_cursor = nullptr;
-        m_transaction.clear();
+        m_transaction = nullptr;
     }
 }
 
index 91170aa..c40d6e3 100644 (file)
@@ -394,7 +394,7 @@ void MediaSource::setReadyState(const AtomicString& state)
     LOG(MediaSource, "MediaSource::setReadyState(%p) : %s -> %s", this, oldState.string().ascii().data(), state.string().ascii().data());
 
     if (state == closedKeyword()) {
-        m_private.clear();
+        m_private = nullptr;
         m_mediaElement = nullptr;
         m_duration = MediaTime::invalidTime();
     }
@@ -811,7 +811,7 @@ void MediaSource::stop()
     m_asyncEventQueue.close();
     if (!isClosed())
         setReadyState(closedKeyword());
-    m_private.clear();
+    m_private = nullptr;
 }
 
 bool MediaSource::canSuspendForPageCache() const
index d710dc1..301334d 100644 (file)
@@ -97,8 +97,8 @@ bool RTCSessionDescriptionRequestImpl::canSuspendForPageCache() const
 
 void RTCSessionDescriptionRequestImpl::clear()
 {
-    m_successCallback.clear();
-    m_errorCallback.clear();
+    m_successCallback = nullptr;
+    m_errorCallback = nullptr;
 }
 
 } // namespace WebCore
index deb19a9..d000b11 100644 (file)
@@ -104,7 +104,7 @@ bool RTCStatsRequestImpl::canSuspendForPageCache() const
 
 void RTCStatsRequestImpl::clear()
 {
-    m_successCallback.clear();
+    m_successCallback = nullptr;
 }
 
 
index 82f8a12..706f907 100644 (file)
@@ -94,8 +94,8 @@ bool RTCVoidRequestImpl::canSuspendForPageCache() const
 
 void RTCVoidRequestImpl::clear()
 {
-    m_successCallback.clear();
-    m_errorCallback.clear();
+    m_successCallback = nullptr;
+    m_errorCallback = nullptr;
 }
 
 } // namespace WebCore
index 5a3dea0..faa996b 100644 (file)
@@ -231,7 +231,7 @@ void AudioContext::clear()
 {
     // We have to release our reference to the destination node before the context will ever be deleted since the destination node holds a reference to the context.
     if (m_destinationNode)
-        m_destinationNode.clear();
+        m_destinationNode = nullptr;
 
     // Audio thread is dead. Nobody will schedule node deletion action. Let's do it ourselves.
     do {
index da254b6..d311380 100644 (file)
@@ -845,7 +845,7 @@ void SQLTransactionBackend::releaseOriginLockIfNeeded()
 {
     if (m_originLock) {
         m_originLock->unlock();
-        m_originLock.clear();
+        m_originLock = nullptr;
     }
 }
 
index bcbdaf9..34b64d7 100644 (file)
@@ -126,7 +126,7 @@ void WorkerThreadableWebSocketChannel::fail(const String& reason)
 void WorkerThreadableWebSocketChannel::disconnect()
 {
     m_bridge->disconnect();
-    m_bridge.clear();
+    m_bridge = nullptr;
 }
 
 void WorkerThreadableWebSocketChannel::suspend()
index 2f35f74..bc38411 100644 (file)
@@ -153,7 +153,7 @@ static void notifyChildrenSelectionChange(AccessibilityObject* object)
 
     AccessibilityObject* listObject = getListObject(object);
     if (!listObject) {
-        oldListObject.get() = 0;
+        oldListObject.get() = nullptr;
         return;
     }
 
@@ -166,7 +166,7 @@ static void notifyChildrenSelectionChange(AccessibilityObject* object)
     // further comparisons make sense. Otherwise, just reset
     // oldFocusedObject so it won't be taken into account.
     if (oldListObject.get() != listObject)
-        oldFocusedObject.get() = 0;
+        oldFocusedObject.get() = nullptr;
 
     AtkObject* axItem = item ? item->wrapper() : nullptr;
     AtkObject* axOldFocusedObject = oldFocusedObject.get() ? oldFocusedObject.get()->wrapper() : nullptr;
index 3abdd2b..c5199cf 100644 (file)
@@ -102,7 +102,7 @@ ScriptController::~ScriptController()
     if (m_cacheableBindingRootObject) {
         JSLockHolder lock(JSDOMWindowBase::commonVM());
         m_cacheableBindingRootObject->invalidate();
-        m_cacheableBindingRootObject = 0;
+        m_cacheableBindingRootObject = nullptr;
     }
 
     // It's likely that destroying m_windowShells will create a lot of garbage.
@@ -489,7 +489,7 @@ void ScriptController::clearScriptObjects()
 
     if (m_bindingRootObject) {
         m_bindingRootObject->invalidate();
-        m_bindingRootObject = 0;
+        m_bindingRootObject = nullptr;
     }
 
 #if ENABLE(NETSCAPE_PLUGIN_API)
@@ -498,7 +498,7 @@ void ScriptController::clearScriptObjects()
         // script object properly.
         // This shouldn't cause any problems for plugins since they should have already been stopped and destroyed at this point.
         _NPN_DeallocateObject(m_windowScriptNPObject);
-        m_windowScriptNPObject = 0;
+        m_windowScriptNPObject = nullptr;
     }
 #endif
 }
index 6545133..4302d68 100644 (file)
@@ -44,7 +44,7 @@ public:
         ASSERT(m_worldSet.contains(m_normalWorld.get()));
         ASSERT(m_worldSet.size() == 1);
         ASSERT(m_normalWorld->hasOneRef());
-        m_normalWorld.clear();
+        m_normalWorld = nullptr;
         ASSERT(m_worldSet.isEmpty());
     }
 
index 90e0ff5..28c9ef0 100644 (file)
@@ -62,7 +62,7 @@ WorkerScriptController::~WorkerScriptController()
 {
     JSLockHolder lock(vm());
     m_workerGlobalScopeWrapper.clear();
-    m_vm.clear();
+    m_vm = nullptr;
 }
 
 void WorkerScriptController::initScript()
index 91f07bc..5d0fa4f 100644 (file)
@@ -59,7 +59,7 @@ void RuntimeObject::invalidate()
     ASSERT(m_instance);
     if (m_instance)
         m_instance->willInvalidateRuntimeObject();
-    m_instance = 0;
+    m_instance = nullptr;
 }
 
 EncodedJSValue RuntimeObject::fallbackObjectGetter(ExecState* exec, JSObject* slotBase, EncodedJSValue, PropertyName propertyName)
index 022652b..7bf56e7 100644 (file)
@@ -277,7 +277,7 @@ std::error_code parseRuleList(const String& rules, Vector<ContentExtensionRule>&
     ExecState* exec = globalObject->globalExec();
     auto error = loadEncodedRules(*exec, rules, ruleList);
 
-    vm.clear();
+    vm = nullptr;
 
     if (error)
         return error;
index 5e8f05c..9cc308b 100644 (file)
@@ -4502,15 +4502,15 @@ void CSSParser::parseFillPosition(CSSParserValueList& valueList, RefPtr<CSSValue
     if (value2)
         valueList.next();
     else {
-        value1.clear();
+        value1 = nullptr;
         return;
     }
 
     RefPtr<CSSPrimitiveValue> parsedValue1 = downcast<CSSPrimitiveValue>(value1.get());
     RefPtr<CSSPrimitiveValue> parsedValue2 = downcast<CSSPrimitiveValue>(value2.get());
 
-    value1.clear();
-    value2.clear();
+    value1 = nullptr;
+    value2 = nullptr;
 
     // Per CSS3 syntax, <position> can't have 'center' as its second keyword as we have more arguments to follow.
     if (parsedValue2->getValueID() == CSSValueCenter)
@@ -4549,7 +4549,7 @@ void CSSParser::parse2ValuesFillPosition(CSSParserValueList& valueList, RefPtr<C
             valueList.next();
         else {
             if (!inShorthand()) {
-                value1.clear();
+                value1 = nullptr;
                 return;
             }
         }
@@ -7587,8 +7587,8 @@ struct ShadowParseContext {
         }
 
         // Now reset for the next shadow value.
-        x = 0;
-        y = 0;
+        x = nullptr;
+        y = nullptr;
         blur = nullptr;
         spread = nullptr;
         style = nullptr;
@@ -12184,7 +12184,7 @@ PassRefPtr<CSSRuleSourceData> CSSParser::popRuleData()
         return nullptr;
 
     ASSERT(!m_currentRuleDataStack->isEmpty());
-    m_currentRuleData.clear();
+    m_currentRuleData = nullptr;
     RefPtr<CSSRuleSourceData> data = m_currentRuleDataStack->last();
     m_currentRuleDataStack->removeLast();
     return data.release();
@@ -12605,7 +12605,7 @@ void CSSParser::markRuleBodyStart()
 {
     if (!isExtractingSourceData())
         return;
-    m_currentRuleData.clear();
+    m_currentRuleData = nullptr;
     unsigned offset = tokenStartOffset();
     if (tokenStartChar() == '{')
         ++offset; // Skip the rule body opening brace.
index 1f41311..ae3e0fd 100644 (file)
@@ -153,12 +153,12 @@ void CSSValuePool::drain()
     m_fontFamilyValueCache.clear();
 
     for (int i = 0; i < numCSSValueKeywords; ++i)
-        m_identifierValueCache[i] = 0;
+        m_identifierValueCache[i] = nullptr;
 
     for (int i = 0; i < maximumCacheableIntegerValue; ++i) {
-        m_pixelValueCache[i] = 0;
-        m_percentValueCache[i] = 0;
-        m_numberValueCache[i] = 0;
+        m_pixelValueCache[i] = nullptr;
+        m_percentValueCache[i] = nullptr;
+        m_numberValueCache[i] = nullptr;
     }
 }
 
index b90908d..dedd070 100644 (file)
@@ -397,9 +397,9 @@ String StyleProperties::getLayeredShorthandValue(const StylePropertyShorthand& s
                     // Color only belongs in the last layer.
                     if (shorthand.properties()[j] == CSSPropertyBackgroundColor) {
                         if (i != numLayers - 1)
-                            value = 0;
+                            value = nullptr;
                     } else if (i) // Other singletons only belong in the first layer.
-                        value = 0;
+                        value = nullptr;
                 }
             }
 
index f89eff0..dbb6cc4 100644 (file)
@@ -71,7 +71,7 @@ void ViewportStyleResolver::addViewportRule(StyleRuleViewport* viewportRule)
 
 void ViewportStyleResolver::clearDocument()
 {
-    m_document = 0;
+    m_document = nullptr;
 }
 
 void ViewportStyleResolver::resolve()
@@ -94,7 +94,7 @@ void ViewportStyleResolver::resolve()
     m_document->setViewportArguments(arguments);
     m_document->updateViewportArguments();
 
-    m_propertySet = 0;
+    m_propertySet = nullptr;
 }
 
 float ViewportStyleResolver::getViewportArgumentValue(CSSPropertyID id) const
index e502669..f640b7c 100644 (file)
@@ -2424,7 +2424,7 @@ void Document::detachParser()
     if (!m_parser)
         return;
     m_parser->detach();
-    m_parser.clear();
+    m_parser = nullptr;
 }
 
 void Document::cancelParsing()
@@ -5882,7 +5882,7 @@ void Document::clearScriptedAnimationController()
     // FIXME: consider using ActiveDOMObject.
     if (m_scriptedAnimationController)
         m_scriptedAnimationController->clearDocumentPointer();
-    m_scriptedAnimationController.clear();
+    m_scriptedAnimationController = nullptr;
 }
 #endif
     
@@ -6300,7 +6300,7 @@ void Document::updateHoverActiveState(const HitTestRequest& request, Element* in
             curr->setActive(false);
             m_userActionElements.setInActiveChain(curr, false);
         }
-        m_activeElement.clear();
+        m_activeElement = nullptr;
     } else {
         Element* newActiveElement = innerElementInDocument;
         if (!oldActiveElement && newActiveElement && request.active() && !request.touchMove()) {
index 47949fe..ec52489 100644 (file)
@@ -131,7 +131,7 @@ public:
     DocumentMarkerDetails* details() const;
 
     void setActiveMatch(bool);
-    void clearDetails() { m_details.clear(); }
+    void clearDetails() { m_details = nullptr; }
 
     // Offset modifications are done by DocumentMarkerController.
     // Other classes should not call following setters.
index 1cbbe85..322ac84 100644 (file)
@@ -104,7 +104,7 @@ CSSStyleSheet* DocumentStyleSheetCollection::pageUserSheet()
 void DocumentStyleSheetCollection::clearPageUserSheet()
 {
     if (m_pageUserSheet) {
-        m_pageUserSheet = 0;
+        m_pageUserSheet = nullptr;
         m_document.styleResolverChanged(DeferRecalcStyle);
     }
 }
index be884eb..6c8c9e1 100644 (file)
@@ -3207,7 +3207,7 @@ void Element::cloneAttributesFromElement(const Element& other)
 
     other.synchronizeAllAttributes();
     if (!other.m_elementData) {
-        m_elementData.clear();
+        m_elementData = nullptr;
         return;
     }
 
index 44e8f70..70a31de 100644 (file)
@@ -89,7 +89,7 @@ void MutationObserverRegistration::clearTransientRegistrations()
     m_transientRegistrationNodes = nullptr;
 
     ASSERT(m_registrationNodeKeepAlive);
-    m_registrationNodeKeepAlive = 0; // Balanced in observeSubtreeNodeWillDetach.
+    m_registrationNodeKeepAlive = nullptr; // Balanced in observeSubtreeNodeWillDetach.
 }
 
 void MutationObserverRegistration::unregisterAndDelete(MutationObserverRegistration* registry)
index 58b2b9f..61ae28f 100644 (file)
@@ -45,7 +45,7 @@ NodeIterator::NodePointer::NodePointer(PassRefPtr<Node> n, bool b)
 
 void NodeIterator::NodePointer::clear()
 {
-    node.clear();
+    node = nullptr;
 }
 
 bool NodeIterator::NodePointer::moveToNext(Node* root)
@@ -151,7 +151,7 @@ void NodeIterator::detach()
 {
     root()->document().detachNodeIterator(this);
     m_detached = true;
-    m_referenceNode.node.clear();
+    m_referenceNode.node = nullptr;
 }
 
 void NodeIterator::nodeWillBeRemoved(Node& removedNode)
index 0f316da..603e88f 100644 (file)
@@ -92,7 +92,7 @@ public:
 
     AnchorType anchorType() const { return static_cast<AnchorType>(m_anchorType); }
 
-    void clear() { m_anchorNode.clear(); m_offset = 0; m_anchorType = PositionIsOffsetInAnchor; m_isLegacyEditingPosition = false; }
+    void clear() { m_anchorNode = nullptr; m_offset = 0; m_anchorType = PositionIsOffsetInAnchor; m_isLegacyEditingPosition = false; }
 
     // These are always DOM compliant values.  Editing positions like [img, 0] (aka [img, before])
     // will return img->parentNode() and img->computeNodeIndex() from these functions.
index 3ecb646..b343ca3 100644 (file)
@@ -281,7 +281,7 @@ void ProcessingInstruction::removedFrom(ContainerNode& insertionPoint)
     if (m_sheet) {
         ASSERT(m_sheet->ownerNode() == this);
         m_sheet->clearOwnerNode();
-        m_sheet = 0;
+        m_sheet = nullptr;
     }
 
     // If we're in document teardown, then we don't need to do any notification of our sheet's removal.
index b04dc08..5b14a3d 100644 (file)
@@ -112,9 +112,9 @@ inline int RangeBoundaryPoint::offset() const
 
 inline void RangeBoundaryPoint::clear()
 {
-    m_containerNode.clear();
+    m_containerNode = nullptr;
     m_offsetInContainer = 0;
-    m_childBeforeBoundary = 0;
+    m_childBeforeBoundary = nullptr;
 }
 
 inline void RangeBoundaryPoint::set(PassRefPtr<Node> container, int offset, Node* childBefore)
@@ -149,7 +149,7 @@ inline void RangeBoundaryPoint::setToStartOfNode(PassRefPtr<Node> container)
     ASSERT(container);
     m_containerNode = container;
     m_offsetInContainer = 0;
-    m_childBeforeBoundary = 0;
+    m_childBeforeBoundary = nullptr;
 }
 
 inline void RangeBoundaryPoint::setToEndOfNode(PassRefPtr<Node> container)
@@ -158,7 +158,7 @@ inline void RangeBoundaryPoint::setToEndOfNode(PassRefPtr<Node> container)
     m_containerNode = container;
     if (m_containerNode->offsetInCharacters()) {
         m_offsetInContainer = m_containerNode->maxCharacterOffset();
-        m_childBeforeBoundary = 0;
+        m_childBeforeBoundary = nullptr;
     } else {
         m_childBeforeBoundary = m_containerNode->lastChild();
         m_offsetInContainer = m_childBeforeBoundary ? invalidOffset : 0;
index ef56801..f85b964 100644 (file)
@@ -105,7 +105,7 @@ public:
     bool operator!=(const SpaceSplitString& other) const { return m_data != other.m_data; }
 
     void set(const AtomicString&, bool shouldFoldCase);
-    void clear() { m_data.clear(); }
+    void clear() { m_data = nullptr; }
 
     bool contains(const AtomicString& string) const { return m_data && m_data->contains(string); }
     bool containsAll(const SpaceSplitString& names) const { return !names.m_data || (m_data && m_data->containsAll(*names.m_data)); }
index 5b26966..0ec0d7f 100644 (file)
@@ -185,7 +185,7 @@ inline void StyledElement::setInlineStyleFromString(const AtomicString& newStyle
     // We reconstruct the property set instead of mutating if there is no CSSOM wrapper.
     // This makes wrapperless property sets immutable and so cacheable.
     if (inlineStyle && !is<MutableStyleProperties>(*inlineStyle))
-        inlineStyle.clear();
+        inlineStyle = nullptr;
 
     if (!inlineStyle)
         inlineStyle = CSSParser::parseInlineStyleDeclaration(newStyleString, this);
@@ -202,7 +202,7 @@ void StyledElement::styleAttributeChanged(const AtomicString& newStyleString, At
     if (newStyleString.isNull()) {
         if (PropertySetCSSStyleDeclaration* cssomWrapper = inlineStyleCSSOMWrapper())
             cssomWrapper->clearParentElement();
-        ensureUniqueElementData().m_inlineStyle.clear();
+        ensureUniqueElementData().m_inlineStyle = nullptr;
     } else if (reason == ModifiedByCloning || document().contentSecurityPolicy()->allowInlineStyle(document().url(), startLineNumber))
         setInlineStyleFromString(newStyleString);
 
index c68611f..c417486 100644 (file)
@@ -183,9 +183,9 @@ void PlatformMessagePortChannel::closeInternal()
 {
     MutexLocker lock(m_mutex);
     // Disentangle ourselves from the other end. We still maintain a reference to our incoming queue, since previously-existing messages should still be delivered.
-    m_remotePort = 0;
-    m_entangledChannel = 0;
-    m_outgoingQueue = 0;
+    m_remotePort = nullptr;
+    m_entangledChannel = nullptr;
+    m_outgoingQueue = nullptr;
 }
 
 } // namespace WebCore
index f5bc948..275e430 100644 (file)
@@ -147,7 +147,7 @@ void AlternativeTextController::startAlternativeTextUITimer(AlternativeTextType
 
     // If type is PanelTypeReversion, then the new range has been set. So we shouldn't clear it.
     if (type == AlternativeTextTypeCorrection)
-        m_alternativeTextInfo.rangeWithAlternative.clear();
+        m_alternativeTextInfo.rangeWithAlternative = nullptr;
     m_alternativeTextInfo.type = type;
     m_timer.startOneShot(correctionPanelTimerInterval);
 }
@@ -155,7 +155,7 @@ void AlternativeTextController::startAlternativeTextUITimer(AlternativeTextType
 void AlternativeTextController::stopAlternativeTextUITimer()
 {
     m_timer.stop();
-    m_alternativeTextInfo.rangeWithAlternative.clear();
+    m_alternativeTextInfo.rangeWithAlternative = nullptr;
 }
 
 void AlternativeTextController::stopPendingCorrection(const VisibleSelection& oldSelection)
@@ -183,7 +183,7 @@ void AlternativeTextController::applyPendingCorrection(const VisibleSelection& s
     if (doApplyCorrection)
         handleAlternativeTextUIResult(dismissSoon(ReasonForDismissingAlternativeTextAccepted)); 
     else
-        m_alternativeTextInfo.rangeWithAlternative.clear();
+        m_alternativeTextInfo.rangeWithAlternative = nullptr;
 }
 
 bool AlternativeTextController::hasPendingCorrection() const
@@ -359,7 +359,7 @@ void AlternativeTextController::timerFired()
         Vector<String> suggestions;
         textChecker()->getGuessesForWord(m_alternativeTextInfo.originalText, paragraphText, suggestions);
         if (suggestions.isEmpty()) {
-            m_alternativeTextInfo.rangeWithAlternative.clear();
+            m_alternativeTextInfo.rangeWithAlternative = nullptr;
             break;
         }
         String topSuggestion = suggestions.first();
@@ -422,7 +422,7 @@ void AlternativeTextController::handleAlternativeTextUIResult(const String& resu
         break;
     }
 
-    m_alternativeTextInfo.rangeWithAlternative.clear();
+    m_alternativeTextInfo.rangeWithAlternative = nullptr;
 }
 
 bool AlternativeTextController::isAutomaticSpellingCorrectionEnabled()
index 433e953..21abe71 100644 (file)
@@ -143,7 +143,7 @@ void ApplyBlockElementCommand::formatSelection(const VisiblePosition& startOfSel
         // Don't put the next paragraph in the blockquote we just created for this paragraph unless 
         // the next paragraph is in the same cell.
         if (enclosingCell && enclosingCell != enclosingNodeOfType(endOfNextParagraph.deepEquivalent(), &isTableCell))
-            blockquoteForNextIndent = 0;
+            blockquoteForNextIndent = nullptr;
 
         // indentIntoBlockquote could move more than one paragraph if the paragraph
         // is in a list item or a table. As a result, endAfterSelection could refer to a position
index eb3a5f3..c6491d2 100644 (file)
@@ -302,7 +302,7 @@ void DeleteSelectionCommand::saveTypingStyleState()
     if (enclosingNodeOfType(m_selectionToDelete.start(), isMailBlockquote))
         m_deleteIntoBlockquoteStyle = EditingStyle::create(m_selectionToDelete.end());
     else
-        m_deleteIntoBlockquoteStyle = 0;
+        m_deleteIntoBlockquoteStyle = nullptr;
 }
 
 bool DeleteSelectionCommand::handleSpecialCaseBRDelete()
@@ -728,11 +728,11 @@ void DeleteSelectionCommand::calculateTypingStyleAfterDelete()
     // If we deleted into a blockquote, but are now no longer in a blockquote, use the alternate typing style
     if (m_deleteIntoBlockquoteStyle && !enclosingNodeOfType(m_endingPosition, isMailBlockquote, CanCrossEditingBoundary))
         m_typingStyle = m_deleteIntoBlockquoteStyle;
-    m_deleteIntoBlockquoteStyle = 0;
+    m_deleteIntoBlockquoteStyle = nullptr;
 
     m_typingStyle->prepareToApplyAt(m_endingPosition);
     if (m_typingStyle->isEmpty())
-        m_typingStyle = 0;
+        m_typingStyle = nullptr;
     // This is where we've deleted all traces of a style but not a whole paragraph (that's handled above).
     // In this case if we start typing, the new characters should have the same style as the just deleted ones,
     // but, if we change the selection, come back and start typing that style should be lost.  Also see 
index 5cf12cd..17aab29 100644 (file)
@@ -638,7 +638,7 @@ void EditingStyle::overrideTypingStyleAt(const EditingStyle& style, const Positi
 
 void EditingStyle::clear()
 {
-    m_mutableStyle.clear();
+    m_mutableStyle = nullptr;
     m_shouldUseFixedDefaultFontSize = false;
     m_fontSizeDelta = NoFontDelta;
     setUnderlineChange(TextDecorationChange::None);
index a3c3c9c..e7687ec 100644 (file)
@@ -828,7 +828,7 @@ void Editor::removeFormattingAndStyle()
 
 void Editor::clearLastEditCommand() 
 {
-    m_lastEditCommand.clear();
+    m_lastEditCommand = nullptr;
 }
 #if PLATFORM(IOS)
 // If the selection is adjusted from UIKit without closing the typing, the typing command may
@@ -1066,7 +1066,7 @@ void Editor::unappliedEditing(PassRefPtr<EditCommandComposition> cmd)
 
     m_alternativeTextController->respondToUnappliedEditing(cmd.get());
 
-    m_lastEditCommand = 0;
+    m_lastEditCommand = nullptr;
     if (client())
         client()->registerRedoStep(cmd);
     respondToChangedContents(newSelection);
@@ -1084,7 +1084,7 @@ void Editor::reappliedEditing(PassRefPtr<EditCommandComposition> cmd)
     
     updateEditorUINowIfScheduled();
 
-    m_lastEditCommand = 0;
+    m_lastEditCommand = nullptr;
     if (client())
         client()->registerUndoStep(cmd);
     respondToChangedContents(newSelection);
@@ -1727,7 +1727,7 @@ void Editor::setComposition(const String& text, SetCompositionMode mode)
     if (text.isEmpty() && mode != CancelComposition)
         TypingCommand::deleteSelection(document(), 0);
 
-    m_compositionNode = 0;
+    m_compositionNode = nullptr;
     m_customCompositionUnderlines.clear();
 
     insertTextForConfirmedComposition(text);
@@ -1802,7 +1802,7 @@ void Editor::setComposition(const String& text, const Vector<CompositionUnderlin
     if (text.isEmpty())
         TypingCommand::deleteSelection(document(), TypingCommand::PreventSpellChecking);
 
-    m_compositionNode = 0;
+    m_compositionNode = nullptr;
     m_customCompositionUnderlines.clear();
 
     if (!text.isEmpty()) {
index b0e2535..fa08837 100644 (file)
@@ -1359,7 +1359,7 @@ void FrameSelection::prepareForDestruction()
         view->clearSelection();
 
     setSelectionWithoutUpdatingAppearance(VisibleSelection(), defaultSetSelectionOptions(), AlignCursorOnScrollIfNeeded, CharacterGranularity);
-    m_previousCaretNode.clear();
+    m_previousCaretNode = nullptr;
 }
 
 void FrameSelection::setStart(const VisiblePosition &pos, EUserTriggered trigger)
index ed8796e..310a74a 100644 (file)
@@ -359,7 +359,7 @@ inline EditingStyle* FrameSelection::typingStyle() const
 
 inline void FrameSelection::clearTypingStyle()
 {
-    m_typingStyle.clear();
+    m_typingStyle = nullptr;
 }
 
 inline void FrameSelection::setTypingStyle(PassRefPtr<EditingStyle> style)
index b238f87..ec4d272 100644 (file)
@@ -232,7 +232,7 @@ void IndentOutdentCommand::formatSelection(const VisiblePosition& startOfSelecti
 void IndentOutdentCommand::formatRange(const Position& start, const Position& end, const Position&, RefPtr<Element>& blockquoteForNextIndent)
 {
     if (tryIndentingAsListItem(start, end))
-        blockquoteForNextIndent = 0;
+        blockquoteForNextIndent = nullptr;
     else
         indentIntoBlockquote(start, end, blockquoteForNextIndent);
 }
index 2e25eef..eee220c 100644 (file)
@@ -354,8 +354,8 @@ inline void ReplaceSelectionCommand::InsertedNodes::willRemoveNodePreservingChil
 inline void ReplaceSelectionCommand::InsertedNodes::willRemoveNode(Node* node)
 {
     if (m_firstNodeInserted == node && m_lastNodeInserted == node) {
-        m_firstNodeInserted = 0;
-        m_lastNodeInserted = 0;
+        m_firstNodeInserted = nullptr;
+        m_lastNodeInserted = nullptr;
     } else if (m_firstNodeInserted == node)
         m_firstNodeInserted = NodeTraversal::nextSkippingChildren(*m_firstNodeInserted);
     else if (m_lastNodeInserted == node)
index 8b4d541..03f2e51 100644 (file)
@@ -214,7 +214,7 @@ void SpellChecker::didCheck(int sequence, const Vector<TextCheckingResult>& resu
     if (m_lastProcessedSequence < sequence)
         m_lastProcessedSequence = sequence;
 
-    m_processingRequest.clear();
+    m_processingRequest = nullptr;
     if (!m_requestQueue.isEmpty())
         m_timerToProcessQueuedRequest.startOneShot(0);
 }
index 024d2ad..e96134d 100644 (file)
@@ -146,7 +146,7 @@ void TextCheckingParagraph::expandRangeToNextEnd()
 void TextCheckingParagraph::invalidateParagraphRangeValues()
 {
     m_checkingStart = m_checkingEnd = -1;
-    m_offsetAsRange = 0;
+    m_offsetAsRange = nullptr;
     m_text = String();
 }
 
index 7f4c05d..61fc2de 100644 (file)
@@ -140,7 +140,7 @@ void FileReader::readInternal(Blob* blob, FileReaderLoader::ReadType type, Excep
     m_blob = blob;
     m_readType = type;
     m_state = LOADING;
-    m_error = 0;
+    m_error = nullptr;
 
     m_loader = std::make_unique<FileReaderLoader>(m_readType, this);
     m_loader->setEncoding(m_encoding);
index 94e0a4a..d8ffc10 100644 (file)
@@ -121,11 +121,11 @@ void FileReaderLoader::terminate()
 
 void FileReaderLoader::cleanup()
 {
-    m_loader = 0;
+    m_loader = nullptr;
 
     // If we get any error, we do not need to keep a buffer around.
     if (m_errorCode) {
-        m_rawData = 0;
+        m_rawData = nullptr;
         m_stringResult = "";
     }
 }
index 229bab8..829573f 100644 (file)
@@ -196,10 +196,10 @@ void HistoryItem::reset()
 
     m_itemSequenceNumber = generateSequenceNumber();
 
-    m_stateObject = 0;
+    m_stateObject = nullptr;
     m_documentSequenceNumber = generateSequenceNumber();
 
-    m_formData = 0;
+    m_formData = nullptr;
     m_formContentType = String();
 
     clearChildren();
@@ -524,7 +524,7 @@ void HistoryItem::setFormInfoFromRequest(const ResourceRequest& request)
         m_formData = request.httpBody();
         m_formContentType = request.httpContentType();
     } else {
-        m_formData = 0;
+        m_formData = nullptr;
         m_formContentType = String();
     }
 }
index e9a6cc0..dbea0e9 100644 (file)
@@ -98,7 +98,7 @@ void BaseChooserOnlyDateAndTimeInputType::didChooseValue(const String& value)
 
 void BaseChooserOnlyDateAndTimeInputType::didEndChooser()
 {
-    m_dateTimeChooser.clear();
+    m_dateTimeChooser = nullptr;
 }
 
 void BaseChooserOnlyDateAndTimeInputType::closeDateTimeChooser()
index e7553d7..00de9ff 100644 (file)
@@ -411,7 +411,7 @@ void FTPDirectoryDocumentParser::finish()
         m_carryOver = String();
     }
 
-    m_tableElement = 0;
+    m_tableElement = nullptr;
     fastFree(m_buffer);
 
     HTMLDocumentParser::finish();
index a3d3b6b..489d799 100644 (file)
@@ -250,7 +250,7 @@ void FileInputType::setValue(const String&, bool, TextFieldEventBehavior)
 {
     // FIXME: Should we clear the file list, or replace it with a new empty one here? This is observable from JavaScript through custom properties.
     m_fileList->clear();
-    m_icon.clear();
+    m_icon = nullptr;
     element().setNeedsStyleRecalc();
 }
 
index 86d45a4..ce5d7b2 100644 (file)
@@ -423,7 +423,7 @@ void HTMLCanvasElement::makePresentationCopy()
 
 void HTMLCanvasElement::clearPresentationCopy()
 {
-    m_presentedImage.clear();
+    m_presentedImage = nullptr;
 }
 
 void HTMLCanvasElement::setSurfaceSize(const IntSize& size)
@@ -644,7 +644,7 @@ void HTMLCanvasElement::clearImageBuffer() const
 
 void HTMLCanvasElement::clearCopiedImage()
 {
-    m_copiedImage.clear();
+    m_copiedImage = nullptr;
     m_didClearImageBuffer = false;
 }
 
index c23edff..e4e1281 100644 (file)
@@ -264,7 +264,7 @@ void HTMLLinkElement::clearSheet()
     ASSERT(m_sheet);
     ASSERT(m_sheet->ownerNode() == this);
     m_sheet->clearOwnerNode();
-    m_sheet = 0;
+    m_sheet = nullptr;
 }
 
 Node::InsertionNotificationRequest HTMLLinkElement::insertedInto(ContainerNode& insertionPoint)
index 4feaeb9..dd8892f 100644 (file)
@@ -457,7 +457,7 @@ HTMLMediaElement::~HTMLMediaElement()
 
     if (m_mediaController) {
         m_mediaController->removeMediaElement(this);
-        m_mediaController = 0;
+        m_mediaController = nullptr;
     }
 
 #if ENABLE(MEDIA_SOURCE)
@@ -942,7 +942,7 @@ void HTMLMediaElement::prepareForLoad()
 
     // 1 - Abort any already-running instance of the resource selection algorithm for this element.
     m_loadState = WaitingForSource;
-    m_currentSourceNode = 0;
+    m_currentSourceNode = nullptr;
 
     // 2 - If there are any tasks from the media element's media element event task source in 
     // one of the task queues, then remove those tasks.
@@ -1006,7 +1006,7 @@ void HTMLMediaElement::prepareForLoad()
     setPlaybackRate(defaultPlaybackRate());
 
     // 6 - Set the error attribute to null and the autoplaying flag to true.
-    m_error = 0;
+    m_error = nullptr;
     m_autoplaying = true;
 
     // 7 - Invoke the media element's resource selection algorithm.
@@ -1101,7 +1101,7 @@ void HTMLMediaElement::selectMediaResource()
         if (auto firstSource = childrenOfType<HTMLSourceElement>(*this).first()) {
             mode = children;
             m_nextChildNodeToConsider = firstSource;
-            m_currentSourceNode = 0;
+            m_currentSourceNode = nullptr;
         } else {
             // Otherwise the media element has neither a src attribute nor a source element 
             // child: set the networkState to NETWORK_EMPTY, and abort these steps; the 
@@ -1277,7 +1277,7 @@ void HTMLMediaElement::loadResource(const URL& initialURL, ContentType& contentT
         else {
             // Forget our reference to the MediaSource, so we leave it alone
             // while processing remainder of load failure.
-            m_mediaSource = 0;
+            m_mediaSource = nullptr;
             mediaLoadingFailed(MediaPlayer::FormatError);
         }
     } else
@@ -1800,7 +1800,7 @@ void HTMLMediaElement::noneSupported()
 
     stopPeriodicTimers();
     m_loadState = WaitingForSource;
-    m_currentSourceNode = 0;
+    m_currentSourceNode = nullptr;
 
     // 4.8.10.5 
     // 6 - Reaching this step indicates that the media resource failed to load or that the given 
@@ -2939,7 +2939,7 @@ void HTMLMediaElement::closeMediaSource()
         return;
 
     m_mediaSource->close();
-    m_mediaSource = 0;
+    m_mediaSource = nullptr;
 }
 #endif
 
@@ -4112,8 +4112,8 @@ check_again:
         m_currentSourceNode = source;
         m_nextChildNodeToConsider = source->nextSibling();
     } else {
-        m_currentSourceNode = 0;
-        m_nextChildNodeToConsider = 0;
+        m_currentSourceNode = nullptr;
+        m_nextChildNodeToConsider = nullptr;
     }
 
 #if !LOG_DISABLED
@@ -4193,7 +4193,7 @@ void HTMLMediaElement::sourceWasRemoved(HTMLSourceElement* source)
         // Clear the current source node pointer, but don't change the movie as the spec says:
         // 4.8.8 - Dynamically modifying a source element and its attribute when the element is already 
         // inserted in a video or audio element will have no effect.
-        m_currentSourceNode = 0;
+        m_currentSourceNode = nullptr;
         LOG(Media, "HTMLMediaElement::sourceRemoved(%p) - m_currentSourceNode set to 0", this);
     }
 }
@@ -4795,7 +4795,7 @@ void HTMLMediaElement::userCancelledLoad()
     setShouldDelayLoadEvent(false);
 
     // 6 - Abort the overall resource selection algorithm.
-    m_currentSourceNode = 0;
+    m_currentSourceNode = nullptr;
 
     // Reset m_readyState since m_player is gone.
     m_readyState = HAVE_NOTHING;
@@ -4812,8 +4812,8 @@ void HTMLMediaElement::clearMediaPlayer(int flags)
 
 #if USE(PLATFORM_TEXT_TRACK_MENU)
     if (platformTextTrackMenu()) {
-        m_platformMenu->setClient(0);
-        m_platformMenu = 0;
+        m_platformMenu->setClient(nullptr);
+        m_platformMenu = nullptr;
     }
 #endif
 
index 22046ea..d786df4 100644 (file)
@@ -104,7 +104,7 @@ bool HTMLPlugInElement::willRespondToMouseClickEvents()
 
 void HTMLPlugInElement::willDetachRenderers()
 {
-    m_instance.clear();
+    m_instance = nullptr;
 
     if (m_isCapturingMouseEvents) {
         if (Frame* frame = document().frame())
@@ -122,7 +122,7 @@ void HTMLPlugInElement::willDetachRenderers()
 
 void HTMLPlugInElement::resetInstance()
 {
-    m_instance.clear();
+    m_instance = nullptr;
 }
 
 PassRefPtr<JSC::Bindings::Instance> HTMLPlugInElement::getInstance()
index 4fb6569..7457361 100644 (file)
@@ -402,7 +402,7 @@ void HTMLTableElement::parseAttribute(const QualifiedName& name, const AtomicStr
         HTMLElement::parseAttribute(name, value);
 
     if (bordersBefore != cellBorders() || oldPadding != m_padding) {
-        m_sharedCellStyle = 0;
+        m_sharedCellStyle = nullptr;
         bool cellChanged = false;
         for (Node* child = firstChild(); child; child = child->nextSibling())
             cellChanged |= setTableCellsChanged(child);
index 7796c27..e967709 100644 (file)
@@ -168,8 +168,8 @@ void PluginDocument::setPluginElement(PassRefPtr<HTMLPlugInElement> element)
 void PluginDocument::detachFromPluginElement()
 {
     // Release the plugin Element so that we don't have a circular reference.
-    m_pluginElement = 0;
-    frame()->loader().client().redirectDataToPlugin(0);
+    m_pluginElement = nullptr;
+    frame()->loader().client().redirectDataToPlugin(nullptr);
 }
 
 void PluginDocument::cancelManualPluginLoad()
index 4b28aa8..eae3db2 100644 (file)
@@ -462,7 +462,7 @@ void TextFieldInputType::updatePlaceholderText()
     if (placeholderText.isEmpty()) {
         if (m_placeholder) {
             m_placeholder->parentNode()->removeChild(m_placeholder.get(), ASSERT_NO_EXCEPTION);
-            m_placeholder.clear();
+            m_placeholder = nullptr;
         }
         return;
     }
index 53b675b..f61e1b5 100644 (file)
@@ -235,10 +235,10 @@ void ValidationMessage::deleteBubbleTree()
 {
     ASSERT(!validationMessageClient());
     if (m_bubble) {
-        m_messageHeading = 0;
-        m_messageBody = 0;
+        m_messageHeading = nullptr;
+        m_messageBody = nullptr;
         m_element->userAgentShadowRoot()->removeChild(m_bubble.get(), ASSERT_NO_EXCEPTION);
-        m_bubble = 0;
+        m_bubble = nullptr;
     }
     m_message = String();
 }
index 92c0511..1b752e6 100644 (file)
@@ -58,11 +58,11 @@ void WebGLProgram::deleteObjectImpl(GraphicsContext3D* context3d, Platform3DObje
     context3d->deleteProgram(obj);
     if (m_vertexShader) {
         m_vertexShader->onDetached(context3d);
-        m_vertexShader = 0;
+        m_vertexShader = nullptr;
     }
     if (m_fragmentShader) {
         m_fragmentShader->onDetached(context3d);
-        m_fragmentShader = 0;
+        m_fragmentShader = nullptr;
     }
 }
 
@@ -148,12 +148,12 @@ bool WebGLProgram::detachShader(WebGLShader* shader)
     case GraphicsContext3D::VERTEX_SHADER:
         if (m_vertexShader != shader)
             return false;
-        m_vertexShader = 0;
+        m_vertexShader = nullptr;
         return true;
     case GraphicsContext3D::FRAGMENT_SHADER:
         if (m_fragmentShader != shader)
             return false;
-        m_fragmentShader = 0;
+        m_fragmentShader = nullptr;
         return true;
     default:
         return false;
index b66b669..7a1cff3 100644 (file)
@@ -494,10 +494,10 @@ void WebGLRenderingContextBase::initializeNewContext()
     m_unpackFlipY = false;
     m_unpackPremultiplyAlpha = false;
     m_unpackColorspaceConversion = GraphicsContext3D::BROWSER_DEFAULT_WEBGL;
-    m_boundArrayBuffer = 0;
-    m_currentProgram = 0;
-    m_framebufferBinding = 0;
-    m_renderbufferBinding = 0;
+    m_boundArrayBuffer = nullptr;
+    m_currentProgram = nullptr;
+    m_framebufferBinding = nullptr;
+    m_renderbufferBinding = nullptr;
     m_depthMask = true;
     m_stencilEnabled = false;
     m_stencilMask = 0xFFFFFFFF;
@@ -623,7 +623,7 @@ void WebGLRenderingContextBase::destroyGraphicsContext3D()
     if (m_context) {
         m_context->setContextLostCallback(nullptr);
         m_context->setErrorMessageCallback(nullptr);
-        m_context.clear();
+        m_context = nullptr;
     }
 }
 
@@ -1479,7 +1479,7 @@ void WebGLRenderingContextBase::deleteBuffer(WebGLBuffer* buffer)
     if (!deleteObject(buffer))
         return;
     if (m_boundArrayBuffer == buffer)
-        m_boundArrayBuffer = 0;
+        m_boundArrayBuffer = nullptr;
 
     m_boundVertexArrayObject->unbindBuffer(buffer);
 }
@@ -1489,7 +1489,7 @@ void WebGLRenderingContextBase::deleteFramebuffer(WebGLFramebuffer* framebuffer)
     if (!deleteObject(framebuffer))
         return;
     if (framebuffer == m_framebufferBinding) {
-        m_framebufferBinding = 0;
+        m_framebufferBinding = nullptr;
         m_context->bindFramebuffer(GraphicsContext3D::FRAMEBUFFER, 0);
     }
 }
@@ -1506,7 +1506,7 @@ void WebGLRenderingContextBase::deleteRenderbuffer(WebGLRenderbuffer* renderbuff
     if (!deleteObject(renderbuffer))
         return;
     if (renderbuffer == m_renderbufferBinding)
-        m_renderbufferBinding = 0;
+        m_renderbufferBinding = nullptr;
     if (m_framebufferBinding)
         m_framebufferBinding->removeAttachmentFromBoundFramebuffer(renderbuffer);
 }
index f6d8da6..f39c0dd 100644 (file)
@@ -76,7 +76,7 @@ void WebGLVertexArrayObjectBase::unbindBuffer(PassRefPtr<WebGLBuffer> buffer)
 {
     if (m_boundElementArrayBuffer == buffer) {
         m_boundElementArrayBuffer->onDetached(context()->graphicsContext3D());
-        m_boundElementArrayBuffer = 0;
+        m_boundElementArrayBuffer = nullptr;
     }
     
     for (size_t i = 0; i < m_vertexAttribState.size(); ++i) {
@@ -95,7 +95,7 @@ void WebGLVertexArrayObjectBase::unbindBuffer(PassRefPtr<WebGLBuffer> buffer)
                 state.originalStride = 0;
                 state.offset = 0;
             } else
-                state.bufferBinding = 0;
+                state.bufferBinding = nullptr;
         }
     }
 }
index 74ecf16..2a133a5 100644 (file)
@@ -245,7 +245,7 @@ TextTrackCueList* TextTrack::cues()
     // http://www.whatwg.org/specs/web-apps/current-work/#dom-texttrack-cues
     if (m_mode != disabledKeyword())
         return ensureTextTrackCueList();
-    return 0;
+    return nullptr;
 }
 
 void TextTrack::removeAllCues()
@@ -257,9 +257,9 @@ void TextTrack::removeAllCues()
         m_client->textTrackRemoveCues(this, m_cues.get());
     
     for (size_t i = 0; i < m_cues->length(); ++i)
-        m_cues->item(i)->setTrack(0);
+        m_cues->item(i)->setTrack(nullptr);
     
-    m_cues = 0;
+    m_cues = nullptr;
 }
 
 TextTrackCueList* TextTrack::activeCues() const
@@ -272,7 +272,7 @@ TextTrackCueList* TextTrack::activeCues() const
     // http://www.whatwg.org/specs/web-apps/current-work/#dom-texttrack-activecues
     if (m_cues && m_mode != disabledKeyword())
         return m_cues->activeCues();
-    return 0;
+    return nullptr;
 }
 
 void TextTrack::addCue(PassRefPtr<TextTrackCue> prpCue, ExceptionCode& ec)
index 34a2ee7..0b9c717 100644 (file)
@@ -482,7 +482,7 @@ void VTTCue::setText(const String& text)
     willChange();
     // Clear the document fragment but don't bother to create it again just yet as we can do that
     // when it is requested.
-    m_webVTTNodeTree = 0;
+    m_webVTTNodeTree = nullptr;
     m_content = text;
     didChange();
 }
index fd4caf6..2dcd26e 100644 (file)
@@ -58,7 +58,7 @@ void InspectorApplicationCacheAgent::didCreateFrontendAndBackend(FrontendChannel
 void InspectorApplicationCacheAgent::willDestroyFrontendAndBackend(Inspector::DisconnectReason)
 {
     m_frontendDispatcher = nullptr;
-    m_backendDispatcher.clear();
+    m_backendDispatcher = nullptr;
 
     m_instrumentingAgents->setInspectorApplicationCacheAgent(nullptr);
 }
index a9083d0..3d06208 100644 (file)
@@ -360,7 +360,7 @@ void InspectorCSSAgent::didCreateFrontendAndBackend(Inspector::FrontendChannel*
 void InspectorCSSAgent::willDestroyFrontendAndBackend(Inspector::DisconnectReason)
 {
     m_frontendDispatcher = nullptr;
-    m_backendDispatcher.clear();
+    m_backendDispatcher = nullptr;
 
     resetNonPersistentData();
 }
index f23a5ea..bc80bca 100644 (file)
@@ -263,7 +263,7 @@ void InspectorController::disconnectFrontend(DisconnectReason reason)
     m_agents.willDestroyFrontendAndBackend(reason);
 
     m_backendDispatcher->clearFrontend();
-    m_backendDispatcher.clear();
+    m_backendDispatcher = nullptr;
     m_frontendChannel = nullptr;
 
     m_isAutomaticInspection = false;
index 0ed2c5c..719f761 100644 (file)
@@ -238,7 +238,7 @@ void InspectorDOMAgent::didCreateFrontendAndBackend(Inspector::FrontendChannel*
 void InspectorDOMAgent::willDestroyFrontendAndBackend(Inspector::DisconnectReason)
 {
     m_frontendDispatcher = nullptr;
-    m_backendDispatcher.clear();
+    m_backendDispatcher = nullptr;
 
     m_history.reset();
     m_domEditor.reset();
@@ -272,7 +272,7 @@ void InspectorDOMAgent::reset()
     discardBindings();
     if (m_revalidateStyleAttrTask)
         m_revalidateStyleAttrTask->reset();
-    m_document = 0;
+    m_document = nullptr;
 }
 
 void InspectorDOMAgent::setDOMListener(DOMListener* listener)
index f778dd9..d51d4be 100644 (file)
@@ -109,7 +109,7 @@ void InspectorDOMDebuggerAgent::didCreateFrontendAndBackend(Inspector::FrontendC
 
 void InspectorDOMDebuggerAgent::willDestroyFrontendAndBackend(Inspector::DisconnectReason)
 {
-    m_backendDispatcher.clear();
+    m_backendDispatcher = nullptr;
 
     disable();
 }
index b1687cf..04af88b 100644 (file)
@@ -76,7 +76,7 @@ void InspectorDOMStorageAgent::didCreateFrontendAndBackend(Inspector::FrontendCh
 void InspectorDOMStorageAgent::willDestroyFrontendAndBackend(Inspector::DisconnectReason)
 {
     m_frontendDispatcher = nullptr;
-    m_backendDispatcher.clear();
+    m_backendDispatcher = nullptr;
 
     ErrorString unused;
     disable(unused);
index 9134dfe..be27f95 100644 (file)
@@ -231,7 +231,7 @@ void InspectorDatabaseAgent::didCreateFrontendAndBackend(Inspector::FrontendChan
 void InspectorDatabaseAgent::willDestroyFrontendAndBackend(Inspector::DisconnectReason)
 {
     m_frontendDispatcher = nullptr;
-    m_backendDispatcher.clear();
+    m_backendDispatcher = nullptr;
 
     ErrorString unused;
     disable(unused);
index 9bf1291..172fb58 100644 (file)
@@ -582,7 +582,7 @@ void InspectorIndexedDBAgent::didCreateFrontendAndBackend(Inspector::FrontendCha
 
 void InspectorIndexedDBAgent::willDestroyFrontendAndBackend(Inspector::DisconnectReason)
 {
-    m_backendDispatcher.clear();
+    m_backendDispatcher = nullptr;
 
     ErrorString unused;
     disable(unused);
index be64496..f6e41fa 100644 (file)
@@ -65,7 +65,7 @@ void InspectorLayerTreeAgent::didCreateFrontendAndBackend(Inspector::FrontendCha
 void InspectorLayerTreeAgent::willDestroyFrontendAndBackend(Inspector::DisconnectReason)
 {
     m_frontendDispatcher = nullptr;
-    m_backendDispatcher.clear();
+    m_backendDispatcher = nullptr;
 
     ErrorString unused;
     disable(unused);
index 6d3a0f3..faea7ce 100644 (file)
@@ -243,9 +243,9 @@ void InspectorOverlay::setPausedInDebuggerMessage(const String* message)
 
 void InspectorOverlay::hideHighlight()
 {
-    m_highlightNode.clear();
-    m_highlightNodeList.clear();
-    m_highlightQuad.reset();
+    m_highlightNode = nullptr;
+    m_highlightNodeList = nullptr;
+    m_highlightQuad = nullptr;
     update();
 }
 
@@ -253,7 +253,7 @@ void InspectorOverlay::highlightNodeList(PassRefPtr<NodeList> nodes, const Highl
 {
     m_nodeHighlightConfig = highlightConfig;
     m_highlightNodeList = nodes;
-    m_highlightNode.clear();
+    m_highlightNode = nullptr;
     update();
 }
 
@@ -261,7 +261,7 @@ void InspectorOverlay::highlightNode(Node* node, const HighlightConfig& highligh
 {
     m_nodeHighlightConfig = highlightConfig;
     m_highlightNode = node;
-    m_highlightNodeList.clear();
+    m_highlightNodeList = nullptr;
     update();
 }
 
index 6f52bfa..3e9febe 100644 (file)
@@ -348,7 +348,7 @@ void InspectorPageAgent::didCreateFrontendAndBackend(Inspector::FrontendChannel*
 void InspectorPageAgent::willDestroyFrontendAndBackend(Inspector::DisconnectReason)
 {
     m_frontendDispatcher = nullptr;
-    m_backendDispatcher.clear();
+    m_backendDispatcher = nullptr;
 
     ErrorString unused;
     disable(unused);
@@ -374,7 +374,7 @@ void InspectorPageAgent::enable(ErrorString&)
 void InspectorPageAgent::disable(ErrorString&)
 {
     m_enabled = false;
-    m_scriptsToEvaluateOnLoad.clear();
+    m_scriptsToEvaluateOnLoad = nullptr;
     m_instrumentingAgents->setInspectorPageAgent(nullptr);
 
     ErrorString unused;
index 64cb811..181a609 100644 (file)
@@ -193,7 +193,7 @@ void InspectorReplayAgent::didCreateFrontendAndBackend(Inspector::FrontendChanne
 void InspectorReplayAgent::willDestroyFrontendAndBackend(Inspector::DisconnectReason)
 {
     m_frontendDispatcher = nullptr;
-    m_backendDispatcher.clear();
+    m_backendDispatcher = nullptr;
 
     m_instrumentingAgents->setInspectorReplayAgent(nullptr);
 
index 854489c..7b56a0f 100644 (file)
@@ -181,7 +181,7 @@ void InspectorResourceAgent::didCreateFrontendAndBackend(Inspector::FrontendChan
 void InspectorResourceAgent::willDestroyFrontendAndBackend(Inspector::DisconnectReason)
 {
     m_frontendDispatcher = nullptr;
-    m_backendDispatcher.clear();
+    m_backendDispatcher = nullptr;
 
     ErrorString unused;
     disable(unused);
index b47c654..06ec1b6 100644 (file)
@@ -1261,7 +1261,7 @@ void InspectorStyleSheetForInlineStyle::didModifyElementAttribute()
     m_isStyleTextValid = false;
     if (m_element->isStyledElement() && m_element->style() != m_inspectorStyle->cssStyle())
         m_inspectorStyle = InspectorStyle::create(InspectorCSSId(id(), 0), inlineStyle(), this);
-    m_ruleSourceData.clear();
+    m_ruleSourceData = nullptr;
 }
 
 bool InspectorStyleSheetForInlineStyle::getText(String* result) const
@@ -1285,7 +1285,7 @@ bool InspectorStyleSheetForInlineStyle::setStyleText(CSSStyleDeclaration* style,
 
     m_styleText = text;
     m_isStyleTextValid = true;
-    m_ruleSourceData.clear();
+    m_ruleSourceData = nullptr;
     return !ec;
 }
 
@@ -1304,7 +1304,7 @@ bool InspectorStyleSheetForInlineStyle::ensureParsedDataReady()
     // The "style" property value can get changed indirectly, e.g. via element.style.borderWidth = "2px".
     const String& currentStyleText = elementStyleText();
     if (m_styleText != currentStyleText) {
-        m_ruleSourceData.clear();
+        m_ruleSourceData = nullptr;
         m_styleText = currentStyleText;
         m_isStyleTextValid = true;
     }
index 0e32bc5..f5756a2 100644 (file)
@@ -103,7 +103,7 @@ void InspectorTimelineAgent::didCreateFrontendAndBackend(Inspector::FrontendChan
 void InspectorTimelineAgent::willDestroyFrontendAndBackend(Inspector::DisconnectReason reason)
 {
     m_frontendDispatcher = nullptr;
-    m_backendDispatcher.clear();
+    m_backendDispatcher = nullptr;
 
     m_instrumentingAgents->setPersistentInspectorTimelineAgent(nullptr);
 
index debc698..5e21f35 100644 (file)
@@ -127,7 +127,7 @@ void InspectorWorkerAgent::willDestroyFrontendAndBackend(Inspector::DisconnectRe
     disable(unused);
 
     m_frontendDispatcher = nullptr;
-    m_backendDispatcher.clear();
+    m_backendDispatcher = nullptr;
 }
 
 void InspectorWorkerAgent::enable(ErrorString&)
index 93db572..e2445ab 100644 (file)
@@ -69,7 +69,7 @@ void PageRuntimeAgent::didCreateFrontendAndBackend(Inspector::FrontendChannel* f
 void PageRuntimeAgent::willDestroyFrontendAndBackend(Inspector::DisconnectReason reason)
 {
     m_frontendDispatcher = nullptr;
-    m_backendDispatcher.clear();
+    m_backendDispatcher = nullptr;
 
     String unused;
     disable(unused);
index 9f4af42..56196f0 100644 (file)
@@ -44,7 +44,7 @@ void WebInjectedScriptManager::disconnect()
     InjectedScriptManager::disconnect();
 
     m_commandLineAPIHost->disconnect();
-    m_commandLineAPIHost.clear();
+    m_commandLineAPIHost = nullptr;
 }
 
 void WebInjectedScriptManager::didCreateInjectedScript(InjectedScript injectedScript)
index f1fe410..d834e0e 100644 (file)
@@ -126,7 +126,7 @@ void WorkerInspectorController::disconnectFrontend(Inspector::DisconnectReason r
 
     m_agents.willDestroyFrontendAndBackend(reason);
     m_backendDispatcher->clearFrontend();
-    m_backendDispatcher.clear();
+    m_backendDispatcher = nullptr;
     m_frontendChannel = nullptr;
 }
 
index 206a41f..1222e55 100644 (file)
@@ -60,7 +60,7 @@ void WorkerRuntimeAgent::didCreateFrontendAndBackend(Inspector::FrontendChannel*
 
 void WorkerRuntimeAgent::willDestroyFrontendAndBackend(Inspector::DisconnectReason reason)
 {
-    m_backendDispatcher.clear();
+    m_backendDispatcher = nullptr;
 
     InspectorRuntimeAgent::willDestroyFrontendAndBackend(reason);
 }
index 5ec47ac..7ca1f23 100644 (file)
@@ -1524,7 +1524,7 @@ void DocumentLoader::getIconLoadDecisionForIconURL(const String& urlString)
 void DocumentLoader::continueIconLoadWithDecision(IconLoadDecision decision)
 {
     ASSERT(m_iconLoadDecisionCallback);
-    m_iconLoadDecisionCallback = 0;
+    m_iconLoadDecisionCallback = nullptr;
     if (m_frame)
         m_frame->loader().icon().continueLoadWithDecision(decision);
 }
index 1e86fac..b0e5898 100644 (file)
@@ -64,7 +64,7 @@ PassRefPtr<DocumentThreadableLoader> DocumentThreadableLoader::create(Document&
 {
     RefPtr<DocumentThreadableLoader> loader = adoptRef(new DocumentThreadableLoader(document, client, LoadAsynchronously, request, options));
     if (!loader->m_resource)
-        loader = 0;
+        loader = nullptr;
     return loader.release();
 }
 
index cc51393..a1c377f 100644 (file)
@@ -90,7 +90,7 @@ void DocumentWriter::replaceDocument(const String& source, Document* ownerDocume
 
 void DocumentWriter::clear()
 {
-    m_decoder = 0;
+    m_decoder = nullptr;
     m_hasReceivedSomeData = false;
     if (!m_encodingWasChosenByUser)
         m_encoding = String();
@@ -245,7 +245,7 @@ void DocumentWriter::end()
     if (!m_parser)
         return;
     m_parser->finish();
-    m_parser = 0;
+    m_parser = nullptr;
 }
 
 void DocumentWriter::setEncoding(const String& name, bool userChosen)
index 522b157..3bdb621 100644 (file)
@@ -692,7 +692,7 @@ void FrameLoader::didBeginDocument(bool dispatch)
 
     if (m_pendingStateObject) {
         m_frame.document()->statePopped(m_pendingStateObject.get());
-        m_pendingStateObject.clear();
+        m_pendingStateObject = nullptr;
     }
 
     if (dispatch)
@@ -818,7 +818,7 @@ void FrameLoader::checkCompleted()
 
     // OK, completed.
     m_isComplete = true;
-    m_requestedHistoryItem = 0;
+    m_requestedHistoryItem = nullptr;
     m_frame.document()->setReadyState(Document::Complete);
 
 #if PLATFORM(IOS)
@@ -1137,7 +1137,7 @@ void FrameLoader::setupForReplace()
     m_client.revertToProvisionalState(m_documentLoader.get());
     setState(FrameStateProvisional);
     m_provisionalDocumentLoader = m_documentLoader;
-    m_documentLoader = 0;
+    m_documentLoader = nullptr;
     detachChildren();
 }
 
index 04054de..d56ca05 100644 (file)
@@ -113,7 +113,7 @@ void ProgressTracker::reset()
     m_lastNotifiedProgressTime = std::chrono::steady_clock::time_point();
     m_finalProgressChangedSent = false;
     m_numProgressTrackedFrames = 0;
-    m_originatingProgressFrame = 0;
+    m_originatingProgressFrame = nullptr;
 
     m_heartbeatsWithNoProgress = 0;
     m_totalBytesReceivedBeforePreviousHeartbeat = 0;
index 431a0cb..166a454 100644 (file)
@@ -90,8 +90,8 @@ void ResourceLoader::releaseResources()
     // has been deallocated and also to avoid reentering this method.
     Ref<ResourceLoader> protect(*this);
 
-    m_frame = 0;
-    m_documentLoader = 0;
+    m_frame = nullptr;
+    m_documentLoader = nullptr;
     
     // We need to set reachedTerminalState to true before we release
     // the resources to prevent a double dealloc of WebView <rdar://problem/4372628>
@@ -104,11 +104,11 @@ void ResourceLoader::releaseResources()
         // Clear out the ResourceHandle's client so that it doesn't try to call
         // us back after we release it, unless it has been replaced by someone else.
         if (m_handle->client() == this)
-            m_handle->setClient(0);
-        m_handle = 0;
+            m_handle->setClient(nullptr);
+        m_handle = nullptr;
     }
 
-    m_resourceData = 0;
+    m_resourceData = nullptr;
     m_deferredRequest = ResourceRequest();
 }
 
@@ -225,7 +225,7 @@ void ResourceLoader::setDefersLoading(bool defers)
 FrameLoader* ResourceLoader::frameLoader() const
 {
     if (!m_frame)
-        return 0;
+        return nullptr;
     return &m_frame->loader();
 }
 
@@ -244,7 +244,7 @@ void ResourceLoader::setDataBufferingPolicy(DataBufferingPolicy dataBufferingPol
 
     // Reset any already buffered data
     if (dataBufferingPolicy == DoNotBufferData)
-        m_resourceData = 0;
+        m_resourceData = nullptr;
 }
     
 void ResourceLoader::willSwitchToSubstituteResource()
@@ -530,7 +530,7 @@ void ResourceLoader::cancel(const ResourceError& error)
         m_documentLoader->cancelPendingSubstituteLoad(this);
         if (m_handle) {
             m_handle->cancel();
-            m_handle = 0;
+            m_handle = nullptr;
         }
         cleanupForError(nonNullError);
     }
index 627c0fe..59bcac9 100644 (file)
@@ -327,7 +327,7 @@ void ApplicationCacheGroup::stopLoading()
         m_manifestHandle->setClient(0);
 
         m_manifestHandle->cancel();
-        m_manifestHandle = 0;
+        m_manifestHandle = nullptr;
     }
     
     if (m_currentHandle) {
@@ -338,11 +338,11 @@ void ApplicationCacheGroup::stopLoading()
         m_currentHandle->setClient(0);
 
         m_currentHandle->cancel();
-        m_currentHandle = 0;
+        m_currentHandle = nullptr;
     }    
 
     // FIXME: Resetting just a tiny part of the state in this function is confusing. Callers have to take care of a lot more.
-    m_cacheBeingUpdated = 0;
+    m_cacheBeingUpdated = nullptr;
     m_pendingEntries.clear();
 }    
 
@@ -521,7 +521,7 @@ void ApplicationCacheGroup::didReceiveResponse(ResourceHandle* handle, const Res
             m_cacheBeingUpdated->addResource(ApplicationCacheResource::create(url, newestCachedResource->response(), type, newestCachedResource->data(), newestCachedResource->path()));
             m_pendingEntries.remove(m_currentHandle->firstRequest().url());
             m_currentHandle->cancel();
-            m_currentHandle = 0;
+            m_currentHandle = nullptr;
             // Load the next resource, if any.
             startLoadingEntry();
             return;
@@ -538,7 +538,7 @@ void ApplicationCacheGroup::didReceiveResponse(ResourceHandle* handle, const Res
         } else if (response.httpStatusCode() == 404 || response.httpStatusCode() == 410) {
             // Skip this resource. It is dropped from the cache.
             m_currentHandle->cancel();
-            m_currentHandle = 0;
+            m_currentHandle = nullptr;
             m_pendingEntries.remove(url);
             // Load the next resource, if any.
             startLoadingEntry();
@@ -551,7 +551,7 @@ void ApplicationCacheGroup::didReceiveResponse(ResourceHandle* handle, const Res
             m_cacheBeingUpdated->addResource(ApplicationCacheResource::create(url, newestCachedResource->response(), type, newestCachedResource->data(), newestCachedResource->path()));
             m_pendingEntries.remove(m_currentHandle->firstRequest().url());
             m_currentHandle->cancel();
-            m_currentHandle = 0;
+            m_currentHandle = nullptr;
             // Load the next resource, if any.
             startLoadingEntry();
         }
@@ -595,7 +595,7 @@ void ApplicationCacheGroup::didFinishLoading(ResourceHandle* handle, double fini
     ASSERT(m_cacheBeingUpdated);
 
     m_cacheBeingUpdated->addResource(m_currentResource.release());
-    m_currentHandle = 0;
+    m_currentHandle = nullptr;
 
     // While downloading check to see if we have exceeded the available quota.
     // We can stop immediately if we have already previously failed
@@ -603,7 +603,7 @@ void ApplicationCacheGroup::didFinishLoading(ResourceHandle* handle, double fini
     // of the quota being reached and decided not to increase it then.
     // FIXME: Should we break earlier and prevent redownloading on later page loads?
     if (m_originQuotaExceededPreviously && m_availableSpaceInQuota < m_cacheBeingUpdated->estimatedSizeInStorage()) {
-        m_currentResource = 0;
+        m_currentResource = nullptr;
         m_frame->document()->addConsoleMessage(MessageSource::AppCache, MessageLevel::Error, ASCIILiteral("Application Cache update failed, because size quota was exceeded."));
         cacheUpdateFailed();
         return;
@@ -631,7 +631,7 @@ void ApplicationCacheGroup::didFail(ResourceHandle* handle, const ResourceError&
         url.removeFragmentIdentifier();
 
     ASSERT(!m_currentResource || !m_pendingEntries.contains(url));
-    m_currentResource = 0;
+    m_currentResource = nullptr;
     m_pendingEntries.remove(url);
 
     if ((type & ApplicationCacheResource::Explicit) || (type & ApplicationCacheResource::Fallback)) {
@@ -695,7 +695,7 @@ void ApplicationCacheGroup::didFinishLoadingManifest()
         return;
     }
 
-    m_manifestHandle = 0;
+    m_manifestHandle = nullptr;
 
     // Check if the manifest was not modified.
     if (isUpgradeAttempt) {
@@ -706,7 +706,7 @@ void ApplicationCacheGroup::didFinishLoadingManifest()
             (newestManifest->data()->size() == m_manifestResource->data()->size() && !memcmp(newestManifest->data()->data(), m_manifestResource->data()->data(), newestManifest->data()->size()))) {
 
             m_completionType = NoUpdate;
-            m_manifestResource = 0;
+            m_manifestResource = nullptr;
             deliverDelayedMainResources();
 
             return;
@@ -782,7 +782,7 @@ void ApplicationCacheGroup::didReachOriginQuota(int64_t totalSpaceNeeded)
 void ApplicationCacheGroup::cacheUpdateFailed()
 {
     stopLoading();
-    m_manifestResource = 0;
+    m_manifestResource = nullptr;
 
     // Wait for master resource loads to finish.
     m_completionType = Failure;
@@ -807,7 +807,7 @@ void ApplicationCacheGroup::manifestNotFound()
     stopLoading();
 
     ASSERT(m_pendingEntries.isEmpty());
-    m_manifestResource = 0;
+    m_manifestResource = nullptr;
 
     while (!m_pendingMasterResourceLoaders.isEmpty()) {
         HashSet<DocumentLoader*>::iterator it = m_pendingMasterResourceLoaders.begin();
index 7c2fe7d..6d11880 100644 (file)
@@ -138,7 +138,7 @@ void CachedCSSStyleSheet::destroyDecodedData()
         return;
 
     m_parsedStyleSheetCache->removedFromMemoryCache();
-    m_parsedStyleSheetCache.clear();
+    m_parsedStyleSheetCache = nullptr;
 
     setDecodedSize(0);
 }
@@ -146,11 +146,11 @@ void CachedCSSStyleSheet::destroyDecodedData()
 PassRefPtr<StyleSheetContents> CachedCSSStyleSheet::restoreParsedStyleSheet(const CSSParserContext& context, CachePolicy cachePolicy)
 {
     if (!m_parsedStyleSheetCache)
-        return 0;
+        return nullptr;
     if (!m_parsedStyleSheetCache->subresourcesAllowReuse(cachePolicy)) {
         m_parsedStyleSheetCache->removedFromMemoryCache();
-        m_parsedStyleSheetCache.clear();
-        return 0;
+        m_parsedStyleSheetCache = nullptr;
+        return nullptr;
     }
 
     ASSERT(m_parsedStyleSheetCache->isCacheable());
@@ -158,7 +158,7 @@ PassRefPtr<StyleSheetContents> CachedCSSStyleSheet::restoreParsedStyleSheet(cons
 
     // Contexts must be identical so we know we would get the same exact result if we parsed again.
     if (m_parsedStyleSheetCache->parserContext() != context)
-        return 0;
+        return nullptr;
 
     didAccessDecodedData(monotonicallyIncreasingTime());
 
index 52769db..df33675 100644 (file)
@@ -365,8 +365,8 @@ inline void CachedImage::clearImage()
     // If our Image has an observer, it's always us so we need to clear the back pointer
     // before dropping our reference.
     if (m_image)
-        m_image->setImageObserver(0);
-    m_image.clear();
+        m_image->setImageObserver(nullptr);
+    m_image = nullptr;
 }
 
 void CachedImage::addIncrementalDataBuffer(SharedBuffer& data)
@@ -458,7 +458,7 @@ void CachedImage::destroyDecodedData()
 {
     bool canDeleteImage = !m_image || (m_image->hasOneRef() && m_image->isBitmapImage());
     if (canDeleteImage && !isLoading() && !hasClients()) {
-        m_image = 0;
+        m_image = nullptr;
         setDecodedSize(0);
     } else if (m_image && !errorOccurred())
         m_image->destroyDecodedData();
index 3c1b295..54bbde1 100644 (file)
@@ -268,7 +268,7 @@ bool CachedRawResource::canReuse(const ResourceRequest& newRequest) const
 
 void CachedRawResource::clear()
 {
-    m_data.clear();
+    m_data = nullptr;
     setEncodedSize(0);
     if (m_loader)
         m_loader->clearResourceData();
index f4431bf..3bef21e 100644 (file)
@@ -317,7 +317,7 @@ void CachedResource::error(CachedResource::Status status)
 {
     setStatus(status);
     ASSERT(errorOccurred());
-    m_data.clear();
+    m_data = nullptr;
 
     setLoading(false);
     checkNotify();
index d4bc0d1..e7f1d4b 100644 (file)
@@ -71,7 +71,7 @@ void IconRecord::setImageData(PassRefPtr<SharedBuffer> data)
     // Copy the provided data into the buffer of the new Image object.
     if (!m_image->setData(data, true)) {
         LOG(IconDatabase, "Manual image data for iconURL '%s' FAILED - it was probably invalid image data", m_iconURL.ascii().data());
-        m_image.clear();
+        m_image = nullptr;
     }
     
     m_dataSet = true;
index 654a4fe..3aa66d9 100644 (file)
@@ -485,7 +485,7 @@ void EventHandler::clear()
     m_mousePositionIsUnknown = true;
     m_lastKnownMousePosition = IntPoint();
     m_lastKnownMouseGlobalPosition = IntPoint();
-    m_mousePressNode = 0;
+    m_mousePressNode = nullptr;
     m_mousePressed = false;
     m_capturesDragging = false;
     m_capturingMouseEventsElement = nullptr;
@@ -494,7 +494,7 @@ void EventHandler::clear()
 #endif
 #if ENABLE(TOUCH_EVENTS) && !ENABLE(IOS_TOUCH_EVENTS)
     m_originatingTouchPointTargets.clear();
-    m_originatingTouchPointDocument.clear();
+    m_originatingTouchPointDocument = nullptr;
     m_originatingTouchPointTargetKey = 0;
 #endif
     m_maxMouseMovedDuration = 0;
@@ -3994,7 +3994,7 @@ bool EventHandler::handleTouchEvent(const PlatformTouchEvent& event)
     }
     m_touchPressed = touches->length() > 0;
     if (allTouchReleased)
-        m_originatingTouchPointDocument.clear();
+        m_originatingTouchPointDocument = nullptr;
 
     // Now iterate the changedTouches list and m_targets within it, sending events to the targets as required.
     bool swallowedEvent = false;
index 1d3dcc2..f86d81f 100644 (file)
@@ -516,7 +516,7 @@ void Page::refreshPlugins(bool reload)
     Vector<Ref<Frame>> framesNeedingReload;
 
     for (auto& page : *allPages) {
-        page->m_pluginData.clear();
+        page->m_pluginData = nullptr;
 
         if (!reload)
             continue;
index f76e8df..05442ad 100644 (file)
@@ -86,7 +86,7 @@ void CARingBuffer::allocate(uint32_t channelCount, size_t bytesPerFrame, size_t
 void CARingBuffer::deallocate()
 {
     if (m_buffers)
-        m_buffers.clear();
+        m_buffers = nullptr;
 
     m_channelCount = 0;
     m_capacityBytes = 0;
index 603eba8..d9265b5 100644 (file)
@@ -363,7 +363,7 @@ const OpenTypeMathData* Font::mathData() const
     if (!m_mathData) {
         m_mathData = OpenTypeMathData::create(m_platformData);
         if (!m_mathData->hasMathData())
-            m_mathData.clear();
+            m_mathData = nullptr;
     }
     return m_mathData.get();
 }
index a6c4049..8d1a835 100644 (file)
@@ -320,7 +320,7 @@ PassRefPtr<OpenTypeVerticalData> FontCache::getVerticalData(const FontFileKey& k
 
     RefPtr<OpenTypeVerticalData> verticalData = OpenTypeVerticalData::create(platformData);
     if (!verticalData->isOpenType())
-        verticalData.clear();
+        verticalData = nullptr;
     fontVerticalDataCache.set(key, verticalData);
     return verticalData;
 }
index d38c1fa..e6b37fd 100644 (file)
@@ -178,8 +178,8 @@ void GraphicsContext::setStrokeColor(const Color& color, ColorSpace colorSpace)
 {
     m_state.strokeColor = color;
     m_state.strokeColorSpace = colorSpace;
-    m_state.strokeGradient.clear();
-    m_state.strokePattern.clear();
+    m_state.strokeGradient = nullptr;
+    m_state.strokePattern = nullptr;
     setPlatformStrokeColor(color, colorSpace);
 }
 
@@ -245,8 +245,8 @@ void GraphicsContext::setFillColor(const Color& color, ColorSpace colorSpace)
 {
     m_state.fillColor = color;
     m_state.fillColorSpace = colorSpace;
-    m_state.fillGradient.clear();
-    m_state.fillPattern.clear();
+    m_state.fillGradient = nullptr;
+    m_state.fillPattern = nullptr;
     setPlatformFillColor(color, colorSpace);
 }
 
@@ -269,26 +269,26 @@ void GraphicsContext::setAntialiasedFontDilationEnabled(bool antialiasedFontDila
 
 void GraphicsContext::setStrokePattern(Ref<Pattern>&& pattern)
 {
-    m_state.strokeGradient.clear();
+    m_state.strokeGradient = nullptr;
     m_state.strokePattern = WTF::move(pattern);
 }
 
 void GraphicsContext::setFillPattern(Ref<Pattern>&& pattern)
 {
-    m_state.fillGradient.clear();
+    m_state.fillGradient = nullptr;
     m_state.fillPattern = WTF::move(pattern);
 }
 
 void GraphicsContext::setStrokeGradient(Ref<Gradient>&& gradient)
 {
     m_state.strokeGradient = WTF::move(gradient);
-    m_state.strokePattern.clear();
+    m_state.strokePattern = nullptr;
 }
 
 void GraphicsContext::setFillGradient(Ref<Gradient>&& gradient)
 {
     m_state.fillGradient = WTF::move(gradient);
-    m_state.fillPattern.clear();
+    m_state.fillPattern = nullptr;
 }
 
 void GraphicsContext::beginTransparencyLayer(float opacity)
index 294a76d..012795e 100644 (file)
@@ -314,10 +314,10 @@ bool MediaPlayer::load(const URL& url, const ContentType& contentType, const Str
     m_contentMIMETypeWasInferredFromExtension = false;
 
 #if ENABLE(MEDIA_SOURCE)
-    m_mediaSource = 0;
+    m_mediaSource = nullptr;
 #endif
 #if ENABLE(MEDIA_STREAM)
-    m_mediaStream = 0;
+    m_mediaStream = nullptr;
 #endif
 
     // If the MIME type is missing or is not meaningful, try to figure it out from the URL.
index 82c0769..34f4ce1 100644 (file)
@@ -2208,7 +2208,7 @@ void GraphicsLayerCA::updateContentsImage()
     } else {
         // No image.
         // m_contentsLayer will be removed via updateSublayerList.
-        m_contentsLayer = 0;
+        m_contentsLayer = nullptr;
     }
 }
 
index 7b8631a..437dafc 100644 (file)
@@ -43,7 +43,7 @@ BackingStoreBackendCairoX11::BackingStoreBackendCairoX11(unsigned long rootWindo
 BackingStoreBackendCairoX11::~BackingStoreBackendCairoX11()
 {
     // The pixmap needs to exist when the surface is destroyed, so begin by clearing it.
-    m_surface.clear();
+    m_surface = nullptr;
 }
 
 void BackingStoreBackendCairoX11::scroll(const IntRect& scrollRect, const IntSize& scrollOffset)
index 26df5df..f0d3a7a 100644 (file)
@@ -159,7 +159,7 @@ bool FrameData::clear(bool clearMetadata)
         m_haveMetadata = false;
 
     if (m_frame) {
-        m_frame.clear();
+        m_frame = nullptr;
         return true;
     }
     return false;
index 7627008..c98d5b9 100644 (file)
@@ -245,12 +245,12 @@ void FilterEffect::clearResult()
     if (m_imageBufferResult)
         m_imageBufferResult.reset();
     if (m_unmultipliedImageResult)
-        m_unmultipliedImageResult.clear();
+        m_unmultipliedImageResult = nullptr;
     if (m_premultipliedImageResult)
-        m_premultipliedImageResult.clear();
+        m_premultipliedImageResult = nullptr;
 #if ENABLE(OPENCL)
     if (m_openCLImageResult)
-        m_openCLImageResult.clear();
+        m_openCLImageResult = nullptr;
 #endif
 }
 
@@ -540,7 +540,7 @@ void FilterEffect::transformResultColorSpace(ColorSpace dstColorSpace)
 #if ENABLE(OPENCL)
     if (openCLImage()) {
         if (m_imageBufferResult)
-            m_imageBufferResult.clear();
+            m_imageBufferResult = nullptr;
         FilterContextOpenCL* context = FilterContextOpenCL::context();
         ASSERT(context);
         context->openCLTransformColorSpace(m_openCLImageResult, absolutePaintRect(), m_resultColorSpace, dstColorSpace);
@@ -558,9 +558,9 @@ void FilterEffect::transformResultColorSpace(ColorSpace dstColorSpace)
     m_resultColorSpace = dstColorSpace;
 
     if (m_unmultipliedImageResult)
-        m_unmultipliedImageResult.clear();
+        m_unmultipliedImageResult = nullptr;
     if (m_premultipliedImageResult)
-        m_premultipliedImageResult.clear();
+        m_premultipliedImageResult = nullptr;
 #endif
 }
 
index 06faeca..ee27277 100644 (file)
@@ -71,9 +71,7 @@ ImageGStreamer::ImageGStreamer(GstSample* sample)
 ImageGStreamer::~ImageGStreamer()
 {
     if (m_image)
-        m_image.clear();
-
-    m_image = 0;
+        m_image = nullptr;
 
     // We keep the buffer memory mapped until the image is destroyed because the internal
     // cairo_surface_t was created using cairo_image_surface_create_for_data().
index 934daf8..c2da370 100644 (file)
@@ -1169,8 +1169,7 @@ ResourceHandleStreamingClient::~ResourceHandleStreamingClient()
 {
     if (m_resource) {
         m_resource->cancel();
-        m_resource.release();
-        m_resource = 0;
+        m_resource = nullptr;
     }
 }
 
index 975c38f..0e52d7b 100644 (file)
@@ -278,7 +278,7 @@ PassRefPtr<BitmapTexture> BitmapTextureGL::applyFilters(TextureMapper* textureMa
             texmapGL->drawFiltered(*resultSurface.get(), spareSurface.get(), *filter, j);
             if (!j && filter->type() == FilterOperation::DROP_SHADOW) {
                 spareSurface = resultSurface;
-                resultSurface.clear();
+                resultSurface = nullptr;
             }
             std::swap(resultSurface, intermediateSurface);
         }
index 9ee4a78..d534704 100644 (file)
@@ -301,8 +301,8 @@ void GraphicsLayerTextureMapper::setContentsToImage(Image* image)
             m_compositedImage = TextureMapperTiledBackingStore::create();
         m_compositedImage->setContentsToImage(image);
     } else {
-        m_compositedNativeImagePtr = 0;
-        m_compositedImage = 0;
+        m_compositedNativeImagePtr = nullptr;
+        m_compositedImage = nullptr;
     }
 
     setContentsToPlatformLayer(m_compositedImage.get(), ContentsLayerForImage);
index 0bce8c7..4c54805 100644 (file)
@@ -691,7 +691,7 @@ void TextureMapperGL::bindDefaultSurface()
     data().projectionMatrix = createProjectionMatrix(viewportSize, data().PaintFlags & PaintingMirrored);
     m_context3D->viewport(data().viewport[0], data().viewport[1], viewportSize.width(), viewportSize.height());
     m_clipStack.apply(m_context3D.get());
-    data().currentSurface.clear();
+    data().currentSurface = nullptr;
 }
 
 void TextureMapperGL::bindSurface(BitmapTexture *surface)
index c864202..3455398 100644 (file)
@@ -408,7 +408,7 @@ void TextureMapperLayer::paintWithIntermediateSurface(const TextureMapperPaintOp
 
     if (replicaSurface && options.opacity == 1) {
         commitSurface(options, replicaSurface, rect, 1);
-        replicaSurface.clear();
+        replicaSurface = nullptr;
     }
 
     mainSurface = paintIntoSurface(paintOptions, rect.size());
index 2080462..d35710a 100644 (file)
@@ -39,7 +39,7 @@ void TextureMapperTiledBackingStore::updateContentsFromImageIfNeeded(TextureMapp
         return;
 
     updateContents(textureMapper, m_image.get(), m_image->size(), enclosingIntRect(m_image->rect()), BitmapTexture::UpdateCannotModifyOriginalImageData);
-    m_image.clear();
+    m_image = nullptr;
 }
 
 TransformationMatrix TextureMapperTiledBackingStore::adjustedTransformForRect(const FloatRect& targetRect)
index e2e1f4a..faa165e 100644 (file)
@@ -470,8 +470,8 @@ void CoordinatedGraphicsLayer::setContentsToImage(Image* image)
         m_compositedImage = image;
         m_compositedNativeImagePtr = newNativeImagePtr;
     } else {
-        m_compositedImage = 0;
-        m_compositedNativeImagePtr = 0;
+        m_compositedImage = nullptr;
+        m_compositedNativeImagePtr = nullptr;
     }
 
     GraphicsLayer::setContentsToImage(image);
@@ -820,7 +820,7 @@ void CoordinatedGraphicsLayer::releaseImageBackingIfNeeded()
 
     ASSERT(m_coordinator);
     m_coordinatedImageBacking->removeHost(this);
-    m_coordinatedImageBacking.clear();
+    m_coordinatedImageBacking = nullptr;
     m_layerState.imageID = InvalidCoordinatedImageBackingID;
     m_layerState.imageChanged = true;
 }
index d22b15e..e26150f 100644 (file)
@@ -141,7 +141,7 @@ void CoordinatedImageBacking::releaseSurfaceIfNeeded()
 {
     // We must keep m_surface until UI Process reads m_surface.
     // If m_surface exists, it was created in the previous update.
-    m_surface.clear();
+    m_surface = nullptr;
 }
 
 static const double clearContentsTimerInterval = 3;
index 26f9be6..9fb29f2 100644 (file)
@@ -155,7 +155,7 @@ void GamepadsGtk::unregisterDevice(String deviceFile)
     size_t index = m_slots.find(gamepadDevice);
     ASSERT(index != notFound);
 
-    m_slots[index].clear();
+    m_slots[index] = nullptr;
 }
 
 void GamepadsGtk::updateGamepadList(GamepadList* into)
index ab14c70..dc0caa8 100644 (file)
@@ -179,7 +179,7 @@ SOFT_LINK_CONSTANT(AVFoundation, AVSpeechUtteranceMaximumSpeechRate, float)
     
     // Clear the m_utterance variable in case finish speaking kicks off a new speaking job immediately.
     RefPtr<WebCore::PlatformSpeechSynthesisUtterance> platformUtterance = m_utterance;
-    m_utterance = 0;
+    m_utterance = nullptr;
     
     m_synthesizerObject->client()->didFinishSpeaking(platformUtterance);
 }
@@ -213,7 +213,7 @@ SOFT_LINK_CONSTANT(AVFoundation, AVSpeechUtteranceMaximumSpeechRate, float)
     
     // Clear the m_utterance variable in case finish speaking kicks off a new speaking job immediately.
     RefPtr<WebCore::PlatformSpeechSynthesisUtterance> platformUtterance = m_utterance;
-    m_utterance = 0;
+    m_utterance = nullptr;
     
     m_synthesizerObject->client()->didFinishSpeaking(platformUtterance);
 }
index 8186a6a..b7cfb9b 100644 (file)
@@ -126,7 +126,7 @@ void GeolocationClientMock::permissionTimerFired()
 
 void GeolocationClientMock::reset()
 {
-    m_lastPosition = 0;
+    m_lastPosition = nullptr;
     clearError();
     m_permissionState = PermissionStateUnset;
 }
index 3838372..c0b4dfa 100644 (file)
@@ -45,7 +45,7 @@ void PlatformSpeechSynthesizerMock::speakingFinished()
 {
     ASSERT(m_utterance.get());
     RefPtr<PlatformSpeechSynthesisUtterance> protect(m_utterance);
-    m_utterance = 0;
+    m_utterance = nullptr;
     
     client()->didFinishSpeaking(protect);
 }
@@ -78,7 +78,7 @@ void PlatformSpeechSynthesizerMock::cancel()
     
     m_speakingFinishedTimer.stop();
     client()->speakingErrorOccurred(m_utterance);
-    m_utterance = 0;
+    m_utterance = nullptr;
 }
 
 void PlatformSpeechSynthesizerMock::pause()
index cdc7d73..a4d43df 100644 (file)
@@ -295,7 +295,7 @@ void ResourceRequest::doUpdateResourceRequest()
 void ResourceRequest::doUpdateResourceHTTPBody()
 {
     if (!m_cfRequest) {
-        m_httpBody = 0;
+        m_httpBody = nullptr;
         return;
     }
 
index 41682c8..2d6fb69 100644 (file)
@@ -954,7 +954,7 @@ void InlineFlowBox::computeOverflow(LayoutUnit lineTop, LayoutUnit lineBottom, G
         return;
 
     if (m_overflow)
-        m_overflow.clear();
+        m_overflow = nullptr;
 
     // Visual overflow just includes overflow for stuff we need to repaint ourselves.  Self-painting layers are ignored.
     // Layout overflow is used to determine scrolling extent, so it still includes child layers and also factors in
index 502df66..6171db5 100644 (file)
@@ -4497,7 +4497,7 @@ void RenderBox::addVisualOverflow(const LayoutRect& rect)
 
 void RenderBox::clearOverflow()
 {
-    m_overflow.clear();
+    m_overflow = nullptr;
     RenderFlowThread* flowThread = flowThreadContainingBlock();
     if (flowThread)
         flowThread->clearRegionsOverflow(this);
index 623608d..ea23c1e 100644 (file)
@@ -53,7 +53,7 @@ public:
     void clearOverflow()
     {
         if (m_overflow)
-            m_overflow.clear();
+            m_overflow = nullptr;
     }
 
 private:
index 918a4f8..c745639 100644 (file)
@@ -186,8 +186,8 @@ static bool findPlaceForCounter(RenderElement& counterOwner, const AtomicString&
     // towards the begining of the document for counters with the same identifier as the one
     // we are trying to find a place for. This is the next renderer to be checked.
     RenderElement* currentRenderer = previousInPreOrder(counterOwner);
-    previousSibling = 0;
-    RefPtr<CounterNode> previousSiblingProtector = 0;
+    previousSibling = nullptr;
+    RefPtr<CounterNode> previousSiblingProtector;
 
     while (currentRenderer) {
         CounterNode* currentCounter = makeCounterNode(*currentRenderer, identifier, false);
@@ -203,7 +203,7 @@ static bool findPlaceForCounter(RenderElement& counterOwner, const AtomicString&
                             // hence we are the next sibling of that counter if that reset is not a root or
                             // we are a root node if that reset is a root.
                             parent = currentCounter->parent();
-                            previousSibling = parent ? currentCounter : 0;
+                            previousSibling = parent ? currentCounter : nullptr;
                             return parent;
                         }
                         // We are not a reset node or the previous reset must be on an ancestor of our owner renderer
@@ -213,7 +213,7 @@ static bool findPlaceForCounter(RenderElement& counterOwner, const AtomicString&
                         // In these cases the identified previousSibling will be invalid as its parent is different from
                         // our identified parent.
                         if (previousSiblingProtector->parent() != currentCounter)
-                            previousSiblingProtector = 0;
+                            previousSiblingProtector = nullptr;
 
                         previousSibling = previousSiblingProtector.get();
                         return true;
index ff9a2c5..db99499 100644 (file)
@@ -88,7 +88,7 @@ RenderMenuList::~RenderMenuList()
 #if !PLATFORM(IOS)
     if (m_popup)
         m_popup->disconnectClient();
-    m_popup = 0;
+    m_popup = nullptr;
 #endif
 }
 
index fa6e396..89dc34c 100644 (file)
@@ -62,7 +62,7 @@ RenderSearchField::~RenderSearchField()
 {
     if (m_searchPopup) {
         m_searchPopup->popupMenu()->disconnectClient();
-        m_searchPopup = 0;
+        m_searchPopup = nullptr;
     }
 }
 
index 8b06ce3..0309863 100644 (file)
@@ -121,8 +121,8 @@ public:
     void setSize(FillSize f) { m_sizeType = f.type; m_sizeLength = f.size; }
     void setMaskSourceType(EMaskSourceType m) { m_maskSourceType = m; m_maskSourceTypeSet = true; }
 
-    void clearMaskImage() { m_maskImageOperation.clear(); }
-    void clearImage() { m_image.clear(); m_imageSet = false; }
+    void clearMaskImage() { m_maskImageOperation = nullptr; }
+    void clearImage() { m_image = nullptr; m_imageSet = false; }
     void clearXPosition() { m_xPosSet = false; m_backgroundOriginSet = false; }
     void clearYPosition() { m_yPosSet = false; m_backgroundOriginSet = false; }
 
index 87c58e9..47fee89 100644 (file)
@@ -950,7 +950,7 @@ void RenderStyle::setQuotes(PassRefPtr<QuotesData> q)
 void RenderStyle::clearCursorList()
 {
     if (rareInheritedData->cursorData)
-        rareInheritedData.access()->cursorData = 0;
+        rareInheritedData.access()->cursorData = nullptr;
 }
 
 void RenderStyle::clearContent()
index 225a304..e3fad8d 100644 (file)
@@ -104,7 +104,7 @@ void SVGTRefTargetEventListener::detach()
 
     m_target->removeEventListener(eventNames().DOMSubtreeModifiedEvent, this, false);
     m_target->removeEventListener(eventNames().DOMNodeRemovedFromDocumentEvent, this, false);
-    m_target.clear();
+    m_target = nullptr;
 }
 
 bool SVGTRefTargetEventListener::operator==(const EventListener& listener)
index 1a10517..476d1e2 100644 (file)
@@ -1723,7 +1723,7 @@ void Internals::closeDummyInspectorFrontend()
     m_frontendChannel = nullptr;
 
     m_frontendWindow->close(m_frontendWindow->scriptExecutionContext());
-    m_frontendWindow.clear();
+    m_frontendWindow = nullptr;
 }
 
 void Internals::setJavaScriptProfilingEnabled(bool enabled, ExceptionCode& ec)
index f520c1d..0721076 100644 (file)
@@ -68,13 +68,13 @@ public:
             return;
         m_eventQueue.m_eventDispatcherMap.remove(m_event.get());
         m_event->target()->dispatchEvent(m_event);
-        m_event.clear();
+        m_event = nullptr;
     }
 
     void cancel()
     {
         m_isCancelled = true;
-        m_event.clear();
+        m_event = nullptr;
     }
 
 private:
index 7845b61..bd74eac 100644 (file)
@@ -264,7 +264,7 @@ void WorkerMessagingProxy::workerGlobalScopeDestroyedInternal()
     // WorkerGlobalScopeDestroyedTask is always the last to be performed, so the proxy is not needed for communication
     // in either side any more. However, the Worker object may still exist, and it assumes that the proxy exists, too.
     m_askedToTerminate = true;
-    m_workerThread = 0;
+    m_workerThread = nullptr;
 
     InspectorInstrumentation::workerGlobalScopeTerminated(m_scriptExecutionContext.get(), this);
 
index 873c829..e515552 100644 (file)
@@ -171,7 +171,7 @@ void WorkerThread::workerThread()
 
     // The below assignment will destroy the context, which will in turn notify messaging proxy.
     // We cannot let any objects survive past thread exit, because no other thread will run GC or otherwise destroy them.
-    m_workerGlobalScope = 0;
+    m_workerGlobalScope = nullptr;
 
     // Clean up WebCore::ThreadGlobalData before WTF::WTFThreadData goes away!
     threadGlobalData().destroy();
index c7e8c0b..475d6fb 100644 (file)
@@ -245,7 +245,7 @@ Blob* XMLHttpRequest::responseBlob()
             data.append(m_binaryResponseBuilder->data(), m_binaryResponseBuilder->size());
             String normalizedContentType = Blob::normalizedContentType(responseMIMEType()); // responseMIMEType defaults to text/xml which may be incorrect.
             m_responseBlob = Blob::create(WTF::move(data), normalizedContentType);
-            m_binaryResponseBuilder.clear();
+            m_binaryResponseBuilder = nullptr;
         } else {
             // If we errored out or got no data, we still return a blob, just an empty one.
             m_responseBlob = Blob::create();
@@ -265,7 +265,7 @@ ArrayBuffer* XMLHttpRequest::responseArrayBuffer()
             m_responseArrayBuffer = m_binaryResponseBuilder->createArrayBuffer();
         else
             m_responseArrayBuffer = ArrayBuffer::create(nullptr, 0);
-        m_binaryResponseBuilder.clear();
+        m_binaryResponseBuilder = nullptr;
     }
 
     return m_responseArrayBuffer.get();
@@ -868,8 +868,8 @@ void XMLHttpRequest::clearResponseBuffers()
     m_createdDocument = false;
     m_responseDocument = nullptr;
     m_responseBlob = nullptr;
-    m_binaryResponseBuilder.clear();
-    m_responseArrayBuffer.clear();
+    m_binaryResponseBuilder = nullptr;
+    m_responseArrayBuffer = nullptr;
     m_responseCacheIsValid = false;
 }
 
index 069ccec..491f12c 100644 (file)
@@ -162,8 +162,8 @@ void XSLTProcessor::removeParameter(const String& /*namespaceURI*/, const String
 
 void XSLTProcessor::reset()
 {
-    m_stylesheet.clear();
-    m_stylesheetRootNode.clear();
+    m_stylesheet = nullptr;
+    m_stylesheetRootNode = nullptr;
     m_parameters.clear();
 }
 
index 73ad500..c0091d6 100644 (file)
@@ -1,3 +1,12 @@
+2015-07-04  Chris Dumez  <cdumez@apple.com>
+
+        Drop RefPtr::clear() method
+        https://bugs.webkit.org/show_bug.cgi?id=146556
+
+        Reviewed by Brady Eidson.
+
+        Drop RefPtr::clear() method in favor of "= nullptr;" pattern.
+
 2015-07-03  Dan Bernstein  <mitz@apple.com>
 
         [Xcode] Update some build settings as recommended by Xcode 7
index d1eb32a..ba17f18 100644 (file)
@@ -88,7 +88,7 @@ void StorageAreaSync::scheduleFinalSync()
     ASSERT(isMainThread());
     // FIXME: We do this to avoid races, but it'd be better to make things safe without blocking.
     blockUntilImportComplete();
-    m_storageArea = 0;  // This is done in blockUntilImportComplete() but this is here as a form of documentation that we must be absolutely sure the ref count cycle is broken.
+    m_storageArea = nullptr; // This is done in blockUntilImportComplete() but this is here as a form of documentation that we must be absolutely sure the ref count cycle is broken.
 
     if (m_syncTimer.isActive())
         m_syncTimer.stop();
@@ -375,7 +375,7 @@ void StorageAreaSync::blockUntilImportComplete()
     MutexLocker locker(m_importLock);
     while (!m_importComplete)
         m_importCondition.wait(m_importLock);
-    m_storageArea = 0;
+    m_storageArea = nullptr;
 }
 
 void StorageAreaSync::sync(bool clearItems, const HashMap<String, String>& items)
index b1c08c4..e9f0822 100644 (file)
@@ -1,3 +1,12 @@
+2015-07-04  Chris Dumez  <cdumez@apple.com>
+
+        Drop RefPtr::clear() method
+        https://bugs.webkit.org/show_bug.cgi?id=146556
+
+        Reviewed by Brady Eidson.
+
+        Drop RefPtr::clear() method in favor of "= nullptr;" pattern.
+
 2015-07-03  Dan Bernstein  <mitz@apple.com>
 
         [Xcode] Update some build settings as recommended by Xcode 7
index 7600583..ac434ec 100644 (file)
@@ -281,7 +281,7 @@ extern "C" {
         }
         
         _proxy->destroy();
-        _proxy = 0;
+        _proxy = nullptr;
     }
     
     _pluginLayer = 0;
@@ -432,7 +432,7 @@ extern "C" {
         downcast<RenderEmbeddedObject>(*_element->renderer()).setPluginUnavailabilityReason(RenderEmbeddedObject::PluginCrashed);
 
     _pluginLayer = nil;
-    _proxy = 0;
+    _proxy = nullptr;
     
     // No need for us to be layer backed anymore
     self.wantsLayer = NO;
index 6ced194..c188b9d 100644 (file)
@@ -1143,7 +1143,7 @@ void WebEditorClient::didCheckSucceed(int sequence, NSArray* results)
 {
     ASSERT_UNUSED(sequence, sequence == m_textCheckingRequest->data().sequence());
     m_textCheckingRequest->didSucceed(core(results, m_textCheckingRequest->data().mask()));
-    m_textCheckingRequest.clear();
+    m_textCheckingRequest = nullptr;
 }
 #endif
 
index 622bd9d..17f1647 100644 (file)
@@ -142,7 +142,7 @@ void WebUserMediaClient::cancelRequest(UserMediaRequest& request)
     if (!_request)
         return;
 
-    _request = 0;
+    _request = nullptr;
 #endif
     
 }
index 1142858..a25941e 100644 (file)
@@ -1,3 +1,12 @@
+2015-07-04  Chris Dumez  <cdumez@apple.com>
+
+        Drop RefPtr::clear() method
+        https://bugs.webkit.org/show_bug.cgi?id=146556
+
+        Reviewed by Brady Eidson.
+
+        Drop RefPtr::clear() method in favor of "= nullptr;" pattern.
+
 2015-06-30  Simon Fraser  <simon.fraser@apple.com>
 
         Fix Mac and Windows builds.
index 180ed43..6d30cb5 100644 (file)
@@ -117,7 +117,7 @@ void PluginStream::stop()
 
     if (m_loader) {
         m_loader->cancel();
-        m_loader = 0;
+        m_loader = nullptr;
     }
 
     m_client = 0;
@@ -398,7 +398,7 @@ void PluginStream::sendJavaScriptStream(const URL& requestURL, const CString& re
             return;
     }
 
-    m_loader = 0;
+    m_loader = nullptr;
 
     destroyStream(resultString.isNull() ? NPRES_NETWORK_ERR : NPRES_DONE);
 }
@@ -451,7 +451,7 @@ void PluginStream::didFail(NetscapePlugInStreamLoader* loader, const ResourceErr
 
     destroyStream(NPRES_NETWORK_ERR);
 
-    m_loader = 0;
+    m_loader = nullptr;
 }
 
 void PluginStream::didFinishLoading(NetscapePlugInStreamLoader* loader)
@@ -464,7 +464,7 @@ void PluginStream::didFinishLoading(NetscapePlugInStreamLoader* loader)
 
     destroyStream(NPRES_DONE);
 
-    m_loader = 0;
+    m_loader = nullptr;
 }
 
 bool PluginStream::wantsAllStreams() const
@@ -472,11 +472,11 @@ bool PluginStream::wantsAllStreams() const
     if (!m_pluginFuncs->getvalue)
         return false;
 
-    void* result = 0;
+    void* result = nullptr;
     if (m_pluginFuncs->getvalue(m_instance, NPPVpluginWantsAllNetworkStreams, &result) != NPERR_NO_ERROR)
         return false;
 
-    return result != 0;
+    return !!result;
 }
 
 }
index 32672bc..05921c2 100644 (file)
@@ -187,7 +187,7 @@ void PluginView::init()
     LOG(Plugins, "PluginView::init(): Initializing plug-in '%s'", m_plugin->name().utf8().data());
 
     if (!m_plugin->load()) {
-        m_plugin = 0;
+        m_plugin = nullptr;
         m_status = PluginStatusCanNotLoadPlugin;
         return;
     }
index 06b0054..4828e4d 100644 (file)
@@ -126,6 +126,6 @@ void WebFramePolicyListener::receivedPolicyDecision(PolicyAction action)
 
 void WebFramePolicyListener::invalidate()
 {
-    m_frame = 0;
+    m_frame = nullptr;
 }
 
index 4375271..aa11282 100644 (file)
@@ -851,8 +851,8 @@ void WebView::deleteBackingStore()
         KillTimer(m_viewWindow, DeleteBackingStoreTimer);
         m_deleteBackingStoreTimerActive = false;
     }
-    m_backingStoreBitmap.clear();
-    m_backingStoreDirtyRegion.clear();
+    m_backingStoreBitmap = nullptr;
+    m_backingStoreDirtyRegion = nullptr;
     m_backingStoreSize.cx = m_backingStoreSize.cy = 0;
 }
 
@@ -1079,7 +1079,7 @@ void WebView::updateBackingStore(FrameView* frameView, HDC dc, bool backingStore
         if (m_uiDelegatePrivate)
             m_uiDelegatePrivate->webViewPainted(this);
 
-        m_backingStoreDirtyRegion.clear();
+        m_backingStoreDirtyRegion = nullptr;
     }
 
     if (!dc)
@@ -1732,7 +1732,7 @@ bool WebView::gesture(WPARAM wParam, LPARAM lParam)
 
         break;
     case GID_END:
-        m_gestureTargetNode = 0;
+        m_gestureTargetNode = nullptr;
         break;
     case GID_PAN: {
         if (gi.dwFlags & GF_BEGIN) {
@@ -6628,7 +6628,7 @@ void WebView::setAcceleratedCompositing(bool accelerated)
         ASSERT(m_layerTreeHost);
         m_layerTreeHost->setClient(0);
         m_layerTreeHost->setWindow(0);
-        m_layerTreeHost = 0;
+        m_layerTreeHost = nullptr;
         m_backingLayer = nullptr;
         m_isAcceleratedCompositing = false;
     }
index 8170ac8..f35aa6a 100644 (file)
@@ -1,3 +1,12 @@
+2015-07-04  Chris Dumez  <cdumez@apple.com>
+
+        Drop RefPtr::clear() method
+        https://bugs.webkit.org/show_bug.cgi?id=146556
+
+        Reviewed by Brady Eidson.
+
+        Drop RefPtr::clear() method in favor of "= nullptr;" pattern.        
+
 2015-07-03  Dan Bernstein  <mitz@apple.com>
 
         [Xcode] Update some build settings as recommended by Xcode 7
index 619805c..3d2ca5b 100644 (file)
@@ -147,7 +147,7 @@ void UniqueIDBDatabase::shutdownBackingStore(UniqueIDBDatabaseShutdownType type,
 {
     ASSERT(!RunLoop::isMain());
 
-    m_backingStore.clear();
+    m_backingStore = nullptr;
 
     if (type == UniqueIDBDatabaseShutdownType::DeleteShutdown) {
         String dbFilename = UniqueIDBDatabase::calculateAbsoluteDatabaseFilename(databaseDirectory);
index 88d6d17..4a9f467 100644 (file)
@@ -126,7 +126,7 @@ bool PluginControllerProxy::initialize(const PluginCreationParameters& creationP
         // used as an identifier so it's OK to just get a weak reference.
         Plugin* plugin = m_plugin.get();
         
-        m_plugin = 0;
+        m_plugin = nullptr;
 
         // This will delete the plug-in controller proxy object.
         m_connection->removePluginControllerProxy(this, plugin);
@@ -156,7 +156,7 @@ void PluginControllerProxy::destroy()
     Plugin* plugin = m_plugin.get();
 
     m_plugin->destroyPlugin();
-    m_plugin = 0;
+    m_plugin = nullptr;
 
     platformDestroy();
 
index 3930989..d9e07ef 100644 (file)
@@ -42,7 +42,7 @@ ChildProcessProxy::~ChildProcessProxy()
 
     if (m_processLauncher) {
         m_processLauncher->invalidate();
-        m_processLauncher = 0;
+        m_processLauncher = nullptr;
     }
 }
 
index b9a05f9..c66d28b 100644 (file)
@@ -55,7 +55,7 @@ void CoordinatedBackingStoreTile::swapBuffers(TextureMapper* textureMapper)
         texture->reset(m_tileRect.size(), m_surface->supportsAlpha());
 
     m_surface->copyToTexture(texture, m_sourceRect, m_surfaceOffset);
-    m_surface.clear();
+    m_surface = nullptr;
 }
 
 void CoordinatedBackingStoreTile::setBackBuffer(const IntRect& tileRect, const IntRect& sourceRect, PassRefPtr<CoordinatedSurface> buffer, const IntPoint& offset)
index c099f89..a2249ba 100644 (file)
@@ -257,7 +257,7 @@ void Download::platformInvalidate()
     if (m_resourceHandle) {
         m_resourceHandle->setClient(0);
         m_resourceHandle->cancel();
-        m_resourceHandle = 0;
+        m_resourceHandle = nullptr;
     }
 
     m_downloadClient = nullptr;
@@ -265,7 +265,7 @@ void Download::platformInvalidate()
 
 void Download::platformDidFinish()
 {
-    m_resourceHandle = 0;
+    m_resourceHandle = nullptr;
 }
 
 void Download::receivedCredential(const AuthenticationChallenge&, const Credential&)
index 77b501a..4588464 100644 (file)
@@ -940,7 +940,7 @@ void EwkView::hideContextMenu()
     if (sd->api->context_menu_hide)
         sd->api->context_menu_hide(sd);
 
-    m_contextMenu.clear();
+    m_contextMenu = nullptr;
 }
 
 void EwkView::requestPopupMenu(WKPopupMenuListenerRef popupMenuListener, const WKRect& rect, WKPopupItemTextDirection textDirection, double pageScaleFactor, WKArrayRef items, int32_t selectedIndex)
index d2d5e68..7af2aee 100644 (file)
@@ -96,7 +96,7 @@ GHashTable* webkit_form_submission_request_get_text_fields(WebKitFormSubmissionR
         return request->priv->values.get();
 
     if (!request->priv->webValues->size())
-        return 0;
+        return nullptr;
 
     request->priv->values = adoptGRef(g_hash_table_new_full(g_str_hash, g_str_equal, g_free, g_free));
 
@@ -107,7 +107,7 @@ GHashTable* webkit_form_submission_request_get_text_fields(WebKitFormSubmissionR
         g_hash_table_insert(request->priv->values.get(), g_strdup(it->key.utf8().data()), g_strdup(value->string().utf8().data()));
     }
 
-    request->priv->webValues = 0;
+    request->priv->webValues = nullptr;
 
     return request->priv->values.get();
 }
index d5515ae..e8d2449 100644 (file)
@@ -3520,7 +3520,7 @@ static bool matchesExtensionOrEquivalent(NSString *filename, NSString *extension
 
 - (void)pasteboardChangedOwner:(NSPasteboard *)pasteboard
 {
-    _data->_promisedImage = 0;
+    _data->_promisedImage = nullptr;
     _data->_promisedFilename = "";
     _data->_promisedURL = "";
 }
@@ -3531,7 +3531,7 @@ static bool matchesExtensionOrEquivalent(NSString *filename, NSString *extension
 
     if ([type isEqual:NSTIFFPboardType] && _data->_promisedImage) {
         [pasteboard setData:(NSData *)_data->_promisedImage->getTIFFRepresentation() forType:NSTIFFPboardType];
-        _data->_promisedImage = 0;
+        _data->_promisedImage = nullptr;
     }
 }
 
index 8097ac1..94a24c9 100644 (file)
@@ -126,7 +126,7 @@ void WebSocketServerConnection::sendRawData(const char* data, size_t length)
 void WebSocketServerConnection::didCloseSocketStream(SocketStreamHandle*)
 {
     // Destroy the SocketStreamHandle now to prevent closing an already closed socket later.
-    m_socket.clear();
+    m_socket = nullptr;
 
     // Web Socket Mode.
     if (m_mode == WebSocket)
index 9425299..513b1e0 100644 (file)
@@ -76,7 +76,7 @@ using namespace WebKit;
 
 - (void)dealloc
 {
-    _connection.clear();
+    _connection = nullptr;
     [_urlConnection cancel];
     [_urlConnection release];
     [super dealloc];
index c0ec48e..806e10f 100644 (file)
@@ -95,7 +95,7 @@ void StatisticsRequest::completedRequest(uint64_t requestID, const StatisticsDat
 
     if (m_outstandingRequests.isEmpty()) {
         m_callback->performCallbackWithReturnValue(m_responseDictionary.get());
-        m_callback = 0;
+        m_callback = nullptr;
     }
 }
 
index e8010e5..4061dd5 100644 (file)
@@ -42,7 +42,7 @@ WebFrameListenerProxy::~WebFrameListenerProxy()
 
 void WebFrameListenerProxy::invalidate()
 {
-    m_frame = 0;
+    m_frame = nullptr;
 }
 
 void WebFrameListenerProxy::receivedPolicyDecision(WebCore::PolicyAction action)
@@ -51,7 +51,7 @@ void WebFrameListenerProxy::receivedPolicyDecision(WebCore::PolicyAction action)
         return;
 
     m_frame->receivedPolicyDecision(action, m_listenerID, m_navigation.get());
-    m_frame = 0;
+    m_frame = nullptr;
 }
 
 } // namespace WebKit
index 41bb91a..6928afb 100644 (file)
@@ -57,11 +57,11 @@ WebFrameProxy::~WebFrameProxy()
 
 void WebFrameProxy::webProcessWillShutDown()
 {
-    m_page = 0;
+    m_page = nullptr;
 
     if (m_activeListener) {
         m_activeListener->invalidate();
-        m_activeListener = 0;
+        m_activeListener = nullptr;
     }
 }
 
index dd66671..a377e4a 100644 (file)
@@ -89,7 +89,7 @@ void WebOpenPanelResultListenerProxy::cancel()
 
 void WebOpenPanelResultListenerProxy::invalidate()
 {
-    m_page = 0;
+    m_page = nullptr;
 }
 
 } // namespace WebKit
index bd13a33..79d23e0 100644 (file)
@@ -3706,7 +3706,7 @@ void WebPageProxy::runOpenPanel(uint64_t frameID, const FileChooserSettings& set
 {
     if (m_openPanelResultListener) {
         m_openPanelResultListener->invalidate();
-        m_openPanelResultListener = 0;
+        m_openPanelResultListener = nullptr;
     }
 
     WebFrameProxy* frame = m_process->webFrame(frameID);
@@ -3808,7 +3808,7 @@ void WebPageProxy::showColorPicker(const WebCore::Color& initialColor, const Int
 #if ENABLE(INPUT_TYPE_COLOR_POPOVER)
     // A new popover color well needs to be created (and the previous one destroyed) for
     // each activation of a color element.
-    m_colorPicker = 0;
+    m_colorPicker = nullptr;
 #endif
     if (!m_colorPicker)
         m_colorPicker = m_pageClient.createColorPicker(this, initialColor, elementRect);
@@ -4300,7 +4300,7 @@ void WebPageProxy::didChooseFilesForOpenPanel(const Vector<String>& fileURLs)
     m_process->send(Messages::WebPage::DidChooseFilesForOpenPanel(fileURLs), m_pageID);
 
     m_openPanelResultListener->invalidate();
-    m_openPanelResultListener = 0;
+    m_openPanelResultListener = nullptr;
 }
 
 void WebPageProxy::didCancelForOpenPanel()
@@ -4311,7 +4311,7 @@ void WebPageProxy::didCancelForOpenPanel()
     m_process->send(Messages::WebPage::DidCancelForOpenPanel(), m_pageID);
     
     m_openPanelResultListener->invalidate();
-    m_openPanelResultListener = 0;
+    m_openPanelResultListener = nullptr;
 }
 
 void WebPageProxy::advanceToNextMisspelling(bool startBeforeSelection)
@@ -4767,7 +4767,7 @@ void WebPageProxy::printFinishedCallback(const ResourceError& printError, uint64
 void WebPageProxy::focusedFrameChanged(uint64_t frameID)
 {
     if (!frameID) {
-        m_focusedFrame = 0;
+        m_focusedFrame = nullptr;
         return;
     }
 
@@ -4780,7 +4780,7 @@ void WebPageProxy::focusedFrameChanged(uint64_t frameID)
 void WebPageProxy::frameSetLargestFrameChanged(uint64_t frameID)
 {
     if (!frameID) {
-        m_frameSetLargestFrame = 0;
+        m_frameSetLargestFrame = nullptr;
         return;
     }
 
index bb6bb60..bdd114a 100644 (file)
@@ -118,7 +118,7 @@ static void setEnableHighAccuracy(WKGeolocationManagerRef geolocationManager, bo
 {
     _isWebCoreGeolocationActive = NO;
     [_coreLocationProvider stop];
-    _lastActivePosition.clear();
+    _lastActivePosition = nullptr;
 }
 
 -(void)_setEnableHighAccuracy:(BOOL)enableHighAccuracy
index 4e86af1..28575ef 100644 (file)
@@ -55,7 +55,7 @@ bool InjectedBundle::initialize(const WebProcessCreationParameters& parameters,
             return false;
         }
 
-        m_sandboxExtension = 0;
+        m_sandboxExtension = nullptr;
     }
     
     RetainPtr<CFStringRef> injectedBundlePathStr = m_path.createCFString();
index 83d6ca8..25397c3 100644 (file)
@@ -76,7 +76,7 @@ void WebResourceLoader::cancelResourceLoader()
 
 void WebResourceLoader::detachFromCoreLoader()
 {
-    m_coreLoader = 0;
+    m_coreLoader = nullptr;
 }
 
 void WebResourceLoader::willSendRequest(const ResourceRequest& proposedRequest, const ResourceResponse& redirectResponse)
index 0a7b74d..a36e4b4 100644 (file)
@@ -277,7 +277,7 @@ void NetscapePlugin::cancelStreamLoad(NetscapePluginStream* pluginStream)
 void NetscapePlugin::removePluginStream(NetscapePluginStream* pluginStream)
 {
     if (pluginStream == m_manualStream) {
-        m_manualStream = 0;
+        m_manualStream = nullptr;
         return;
     }
 
index dccc541..47061d6 100644 (file)
@@ -1103,8 +1103,8 @@ void PDFPlugin::destroy()
             frameView->removeScrollableArea(this);
     }
 
-    m_activeAnnotation = 0;
-    m_annotationContainer = 0;
+    m_activeAnnotation = nullptr;
+    m_annotationContainer = nullptr;
 
     destroyScrollbar(HorizontalScrollbar);
     destroyScrollbar(VerticalScrollbar);
@@ -1604,14 +1604,14 @@ void PDFPlugin::setActiveAnnotation(PDFAnnotation *annotation)
 
     if (annotation) {
         if ([annotation isKindOfClass:pdfAnnotationTextWidgetClass()] && static_cast<PDFAnnotationTextWidget *>(annotation).isReadOnly) {
-            m_activeAnnotation = 0;
+            m_activeAnnotation = nullptr;
             return;
         }
 
         m_activeAnnotation = PDFPluginAnnotation::create(annotation, m_pdfLayerController.get(), this);
         m_activeAnnotation->attach(m_annotationContainer.get());
     } else
-        m_activeAnnotation = 0;
+        m_activeAnnotation = nullptr;
 }
 
 bool PDFPlugin::supportsForms()
index 6a833f1..2a0fd97 100644 (file)
@@ -166,7 +166,7 @@ void PluginView::Stream::cancel()
 
     m_streamWasCancelled = true;
     m_loader->cancel(m_loader->cancelledError());
-    m_loader = 0;
+    m_loader = nullptr;
 }
 
 static String buildHTTPHeaders(const ResourceResponse& response, long long& expectedContentLength)
@@ -604,7 +604,7 @@ void PluginView::initializePlugin()
 
 void PluginView::didFailToInitializePlugin()
 {
-    m_plugin = 0;
+    m_plugin = nullptr;
 
 #if ENABLE(NETSCAPE_PLUGIN_API)
     String frameURLString = frame()->loader().documentLoader()->responseURL().string();
index 1554dbc..7b8a926 100644 (file)
@@ -930,23 +930,23 @@ void WebPage::close()
     }
 
 #if ENABLE(FULLSCREEN_API)
-    m_fullScreenManager = 0;
+    m_fullScreenManager = nullptr;
 #endif
 
     if (m_activePopupMenu) {
         m_activePopupMenu->disconnectFromPage();
-        m_activePopupMenu = 0;
+        m_activePopupMenu = nullptr;
     }
 
     if (m_activeOpenPanelResultListener) {
         m_activeOpenPanelResultListener->disconnectFromPage();
-        m_activeOpenPanelResultListener = 0;
+        m_activeOpenPanelResultListener = nullptr;
     }
 
 #if ENABLE(INPUT_TYPE_COLOR)
     if (m_activeColorChooser) {
         m_activeColorChooser->disconnectFromPage();
-        m_activeColorChooser = 0;
+        m_activeColorChooser = nullptr;
     }
 #endif
 
@@ -3270,7 +3270,7 @@ void WebPage::countStringMatches(const String& string, uint32_t options, uint32_
 void WebPage::didChangeSelectedIndexForActivePopupMenu(int32_t newIndex)
 {
     changeSelectedIndex(newIndex);
-    m_activePopupMenu = 0;
+    m_activePopupMenu = nullptr;
 }
 
 void WebPage::changeSelectedIndex(int32_t index)
@@ -3306,12 +3306,12 @@ void WebPage::didChooseFilesForOpenPanel(const Vector<String>& files)
         return;
 
     m_activeOpenPanelResultListener->didChooseFiles(files);
-    m_activeOpenPanelResultListener = 0;
+    m_activeOpenPanelResultListener = nullptr;
 }
 
 void WebPage::didCancelForOpenPanel()
 {
-    m_activeOpenPanelResultListener = 0;
+    m_activeOpenPanelResultListener = nullptr;
 }
 
 #if ENABLE(SANDBOX_EXTENSIONS)
@@ -3415,7 +3415,7 @@ void WebPage::didSelectItemFromActiveContextMenu(const WebContextMenuItemData& i
         return;
 
     m_contextMenu->itemSelected(item);
-    m_contextMenu = 0;
+    m_contextMenu = nullptr;
 }
 #endif
 
index 55ecdb2..b6205db 100644 (file)
@@ -699,7 +699,7 @@ void WebPrintOperationGtk::printPagesDone()
 {
     m_printPagesIdleId = 0;
     endPrint();
-    m_cairoContext = 0;
+    m_cairoContext = nullptr;
 }
 
 void WebPrintOperationGtk::printDone(const WebCore::ResourceError& error)
index 23de83e..9f4cec7 100644 (file)
@@ -1978,7 +1978,7 @@ void WebPage::syncApplyAutocorrection(const String& correction, const String& or
         while (textForRange.length() && textForRange.length() > originalText.length() && loopCount < maxPositionsAttempts) {
             position = position.next();
             if (position.isNotNull() && position >= frame.selection().selection().start())
-                range = NULL;
+                range = nullptr;
             else
                 range = Range::create(*frame.document(), position, frame.selection().selection().start());
             textForRange = (range) ? plainTextReplacingNoBreakSpace(range.get()) : emptyString();
@@ -2458,7 +2458,7 @@ void WebPage::elementDidBlur(WebCore::Node* node)
                 send(Messages::WebPageProxy::StopAssistingNode());
             m_hasPendingBlurNotification = false;
         });
-        m_assistedNode = 0;
+        m_assistedNode = nullptr;
     }
 }
 
index a97bc21..b4c9e0e 100644 (file)
@@ -1034,7 +1034,7 @@ void WebProcess::networkProcessConnectionClosed(NetworkProcessConnection* connec
     ASSERT(m_networkProcessConnection);
     ASSERT_UNUSED(connection, m_networkProcessConnection == connection);
 
-    m_networkProcessConnection = 0;
+    m_networkProcessConnection = nullptr;
     
     m_webResourceLoadScheduler->networkProcessCrashed();
 }
@@ -1051,7 +1051,7 @@ void WebProcess::webToDatabaseProcessConnectionClosed(WebToDatabaseProcessConnec
     ASSERT(m_webToDatabaseProcessConnection);
     ASSERT(m_webToDatabaseProcessConnection == connection);
 
-    m_webToDatabaseProcessConnection = 0;
+    m_webToDatabaseProcessConnection = nullptr;
 }
 
 WebToDatabaseProcessConnection* WebProcess::webToDatabaseProcessConnection()
index 8d7acbb..09a827e 100644 (file)
@@ -1,3 +1,12 @@
+2015-07-04  Chris Dumez  <cdumez@apple.com>
+
+        Drop RefPtr::clear() method
+        https://bugs.webkit.org/show_bug.cgi?id=146556
+
+        Reviewed by Brady Eidson.
+
+        Drop RefPtr::clear() method in favor of "= nullptr;" pattern.
+
 2015-07-03  Dan Bernstein  <mitz@apple.com>
 
         Fixed the iOS build.
index bccd4a9..276df8b 100644 (file)
@@ -2064,7 +2064,7 @@ static void runTest(const string& inputLine)
     ASSERT(CFArrayGetCount(openWindowsRef) == 1);
     ASSERT(CFArrayGetValueAtIndex(openWindowsRef, 0) == [[mainFrame webView] window]);
 
-    gTestRunner.clear();
+    gTestRunner = nullptr;
 
     if (ignoreWebCoreNodeLeaks)
         [WebCoreStatistics stopIgnoringWebCoreNodeLeaks];
index d986cbf..c843547 100644 (file)
@@ -1183,7 +1183,7 @@ static void runTest(const string& inputLine)
 
 exit:
     removeFontFallbackIfPresent(fallbackPath);
-    ::gTestRunner.clear();
+    ::gTestRunner = nullptr;
 
     fputs("#EOF\n", stderr);
     fflush(stderr);
index fc92246..b726d7d 100644 (file)
@@ -102,7 +102,7 @@ TEST(WTF_RefPtr, Basic)
     {
         RefPtr<RefLogger> ptr(&a);
         ASSERT_EQ(&a, ptr.get());
-        ptr.clear();
+        ptr = nullptr;
         ASSERT_EQ(nullptr, ptr.get());
     }
     ASSERT_STREQ("ref(a) deref(a) ", takeLogStr().c_str());