Web Inspector: REGRESSION: [JSC] SourceProvider reuses IDs
authoroliver@apple.com <oliver@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 27 Feb 2013 00:38:14 +0000 (00:38 +0000)
committeroliver@apple.com <oliver@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 27 Feb 2013 00:38:14 +0000 (00:38 +0000)
https://bugs.webkit.org/show_bug.cgi?id=99674

Reviewed by Gavin Barraclough.

Simple incrementing counter for SourceProvider IDs.  Uses a
lock to incrementing the counter so we don't increment reuse
counter values or reassign the ID for a given SourceProvider.

* parser/SourceProvider.cpp:
(JSC::SourceProvider::SourceProvider):
(JSC):
(JSC::SourceProvider::getID):
* parser/SourceProvider.h:
(JSC::SourceProvider::asID):
(SourceProvider):

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

Source/JavaScriptCore/ChangeLog
Source/JavaScriptCore/parser/SourceProvider.cpp
Source/JavaScriptCore/parser/SourceProvider.h

index fd25e85..2ae2ff8 100644 (file)
@@ -1,3 +1,22 @@
+2013-02-26  Oliver Hunt  <oliver@apple.com>
+
+        Web Inspector: REGRESSION: [JSC] SourceProvider reuses IDs
+        https://bugs.webkit.org/show_bug.cgi?id=99674
+
+        Reviewed by Gavin Barraclough.
+
+        Simple incrementing counter for SourceProvider IDs.  Uses a
+        lock to incrementing the counter so we don't increment reuse
+        counter values or reassign the ID for a given SourceProvider.
+
+        * parser/SourceProvider.cpp:
+        (JSC::SourceProvider::SourceProvider):
+        (JSC):
+        (JSC::SourceProvider::getID):
+        * parser/SourceProvider.h:
+        (JSC::SourceProvider::asID):
+        (SourceProvider):
+
 2013-02-26  Sheriff Bot  <webkit.review.bot@gmail.com>
 
         Unreviewed, rolling out r144074.
index c998d12..1ec0ee4 100644 (file)
@@ -25,6 +25,7 @@
 
 #include "config.h"
 #include "SourceProvider.h"
+#include <wtf/TCSpinLock.h>
 
 namespace JSC {
 
@@ -32,6 +33,7 @@ SourceProvider::SourceProvider(const String& url, const TextPosition& startPosit
     : m_url(url)
     , m_startPosition(startPosition)
     , m_validated(false)
+    , m_id(0)
 {
 }
 
@@ -39,5 +41,16 @@ SourceProvider::~SourceProvider()
 {
 }
 
+static TCMalloc_SpinLock providerIdLock = SPINLOCK_INITIALIZER;
+
+void SourceProvider::getID()
+{
+    SpinLockHolder lock(&providerIdLock);
+    if (!m_id) {
+        static intptr_t nextProviderID = 0;
+        m_id = ++nextProviderID;
+    }
+}
+
 } // namespace JSC
 
index dec385a..929dc46 100644 (file)
@@ -58,7 +58,9 @@ namespace JSC {
             ASSERT(this);
             if (!this) // Be defensive in release mode.
                 return nullID;
-            return reinterpret_cast<intptr_t>(this);
+            if (!m_id)
+                getID();
+            return m_id;
         }
 
         bool isValid() const { return m_validated; }
@@ -66,9 +68,12 @@ namespace JSC {
 
     private:
 
+        JS_EXPORT_PRIVATE void getID();
+
         String m_url;
         TextPosition m_startPosition;
-        bool m_validated;
+        bool m_validated : 1;
+        uintptr_t m_id : sizeof(uintptr_t) * 8 - 1;
     };
 
     class StringSourceProvider : public SourceProvider {