blob hacking webcore style

Source/core/fileapi/Blob.cpp

 /*
  * Copyright (C) 2010 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 19 matching lines @@
 
 #include "config.h"
 #include "core/fileapi/Blob.h"
 
 #include "core/fileapi/BlobRegistry.h"
 #include "core/fileapi/BlobURL.h"
 #include "core/fileapi/File.h"
 
 namespace WebCore {
 
-class BlobURLRegistry : public URLRegistry {
+namespace {
+
+class PublicBlobURLRegistry : public URLRegistry {

[kinuko, 2013/09/30 11:31:53]

nit: Now all BlobURLs are public (and this class is private)... do we still want
to add this 'Public' prefix?

[michaeln, 2013/09/30 20:50:24]

I guess we don't need to. Maybe this was overzealous on my part about
eradicating the idea of using urls as the identifier for these things.

 public:
     virtual void registerURL(SecurityOrigin*, const KURL&, URLRegistrable*) OVERRIDE;
     virtual void unregisterURL(const KURL&) OVERRIDE;
 
     static URLRegistry& registry();
 };
 
-
-void BlobURLRegistry::registerURL(SecurityOrigin* origin, const KURL& publicURL, URLRegistrable* blob)
+void PublicBlobURLRegistry::registerURL(SecurityOrigin* origin, const KURL& publicURL, URLRegistrable* blob)
 {
     ASSERT(&blob->registry() == this);
-    BlobRegistry::registerBlobURL(origin, publicURL, static_cast<Blob*>(blob)->url());
+    BlobRegistry::registerPublicBlobURL(origin, publicURL, static_cast<Blob*>(blob)->blobDataHandle());
 }
 
-void BlobURLRegistry::unregisterURL(const KURL& url)
+void PublicBlobURLRegistry::unregisterURL(const KURL& publicURL)
 {
-    BlobRegistry::unregisterBlobURL(url);
+    BlobRegistry::revokePublicBlobURL(publicURL);
 }
 
-URLRegistry& BlobURLRegistry::registry()
+URLRegistry& PublicBlobURLRegistry::registry()
 {
-    DEFINE_STATIC_LOCAL(BlobURLRegistry, instance, ());
+    DEFINE_STATIC_LOCAL(PublicBlobURLRegistry, instance, ());
     return instance;
 }
 
+} // namespace
 
-Blob::Blob()
-    : m_size(0)
+Blob::Blob(PassRefPtr<BlobDataHandle> dataHandle)
+    : m_blobDataHandle(dataHandle)
 {
     ScriptWrappable::init(this);
-    OwnPtr<BlobData> blobData = BlobData::create();
-
-    // Create a new internal URL and register it with the provided blob data.
-    m_internalURL = BlobURL::createInternalURL();
-    BlobRegistry::registerBlobURL(m_internalURL, blobData.release());
-}
-
-Blob::Blob(PassOwnPtr<BlobData> blobData, long long size)
-    : m_type(blobData->contentType())
-    , m_size(size)
-{
-    ASSERT(blobData);
-    ScriptWrappable::init(this);
-
-    // Create a new internal URL and register it with the provided blob data.
-    m_internalURL = BlobURL::createInternalURL();
-    BlobRegistry::registerBlobURL(m_internalURL, blobData);
-}
-
-Blob::Blob(const KURL& srcURL, const String& type, long long size)
-    : m_type(type)
-    , m_size(size)
-{
-    ScriptWrappable::init(this);
-
-    // Create a new internal URL and register it with the same blob data as the source URL.
-    m_internalURL = BlobURL::createInternalURL();
-    BlobRegistry::registerBlobURL(0, m_internalURL, srcURL);
 }
 
 Blob::~Blob()
 {
-    BlobRegistry::unregisterBlobURL(m_internalURL);
 }
 
 PassRefPtr<Blob> Blob::slice(long long start, long long end, const String& contentType) const
 {
     // When we slice a file for the first time, we obtain a snapshot of the file by capturing its current size and modification time.
     // The modification time will be used to verify if the file has been changed or not, when the underlying data are accessed.
     long long size;
     double modificationTime;
     if (isFile()) {
         // FIXME: This involves synchronous file operation. We need to figure out how to make it asynchronous.
         toFile(this)->captureSnapshot(size, modificationTime);
     } else {
-        ASSERT(m_size != -1);
-        size = m_size;
+        size = this->size();
+        ASSERT(size != -1);
     }
 
     // Convert the negative value that is used to select from the end.
     if (start < 0)
         start = start + size;
     if (end < 0)
         end = end + size;
 
     // Clamp the range if it exceeds the size limit.
     if (start < 0)
         start = 0;
     if (end < 0)
         end = 0;
     if (start >= size) {
         start = 0;
         end = 0;
     } else if (end < start)
         end = start;
     else if (end > size)
         end = size;
 
     long long length = end - start;
     OwnPtr<BlobData> blobData = BlobData::create();
     blobData->setContentType(contentType);
     if (isFile()) {
         if (!toFile(this)->fileSystemURL().isEmpty())
-            blobData->appendURL(toFile(this)->fileSystemURL(), start, length, modificationTime);
+            blobData->appendFileSystemURL(toFile(this)->fileSystemURL(), start, length, modificationTime);
         else
             blobData->appendFile(toFile(this)->path(), start, length, modificationTime);
-    } else
-        blobData->appendBlob(m_internalURL, start, length);
-
-    return Blob::create(blobData.release(), length);
+    } else {
+        blobData->appendBlob(m_blobDataHandle, start, length);
+    }
+    return Blob::create(BlobDataHandle::create(blobData.release(), length));
 }
 
 URLRegistry& Blob::registry() const
 {
-    return BlobURLRegistry::registry();
+    return PublicBlobURLRegistry::registry();
 }
 
 
 } // namespace WebCore