Changeset 31300 in webkit

Timestamp:

Mar 25, 2008 4:43:29 PM (16 years ago)

Author:

kmccullough@apple.com

Message:

2008-03-25 Kevin McCullough <​kmccullough@apple.com>

Reviewed by Adam.

• Cleaned up the needless use of RefPtrs, which we may someday no longer use for InspectorResources.

• page/InspectorController.cpp: (WebCore::InspectorController::didFinishLoading): (WebCore::InspectorController::didFailLoading):

Location:

trunk/WebCore

Files:

• ChangeLog (modified) (1 diff)
• page/InspectorController.cpp (modified) (3 diffs)

trunk/WebCore/ChangeLog

@@ +1 @@
+2008-03-25  Kevin McCullough  <kmccullough@apple.com>
+
+        Reviewed by Adam.
+
+        - Cleaned up the needless use of RefPtrs, which we may someday no longer use for
+        InspectorResources.
+
+        * page/InspectorController.cpp:
+        (WebCore::InspectorController::didFinishLoading):
+        (WebCore::InspectorController::didFailLoading):
+
 2008-03-25  Kevin McCullough  <kmccullough@apple.com>
 

trunk/WebCore/page/InspectorController.cpp

@@ -1575 +1575 @@
         return;
 
-    RefPtr<InspectorResource> resource = m_resources.get(identifier);
+    InspectorResource* resource = m_resources.get(identifier).get();
     if (!resource)
         return;
 
-    removeResource(resource.get());
+    removeResource(resource);
 
     resource->finished = true;
     resource->endTime = currentTime();
 
-    addResource(resource.get());
+    addResource(resource);
 
     if (windowVisible() && resource->scriptObject) {
-        updateScriptResource(resource.get(), resource->startTime, resource->responseReceivedTime, resource->endTime);
-        updateScriptResource(resource.get(), resource->finished);
+        updateScriptResource(resource, resource->startTime, resource->responseReceivedTime, resource->endTime);
+        updateScriptResource(resource, resource->finished);
     }
 }
@@ -1597 +1597 @@
         return;
 
-    RefPtr<InspectorResource> resource = m_resources.get(identifier);
+    InspectorResource* resource = m_resources.get(identifier).get();
     if (!resource)
         return;
 
-    removeResource(resource.get());
+    removeResource(resource);
 
     resource->finished = true;
@@ -1607 +1607 @@
     resource->endTime = currentTime();
 
-    addResource(resource.get());
+    addResource(resource);
 
     if (windowVisible() && resource->scriptObject) {
-        updateScriptResource(resource.get(), resource->startTime, resource->responseReceivedTime, resource->endTime);
-        updateScriptResource(resource.get(), resource->finished, resource->failed);
+        updateScriptResource(resource, resource->startTime, resource->responseReceivedTime, resource->endTime);
+        updateScriptResource(resource, resource->finished, resource->failed);
     }
 }