Attachment 373430 Details for Bug 199477 – Patch

[patch] Patch

bug-199477-20190703162316.patch (text/plain), 1.79 KB, created by youenn fablet on 2019-07-03 16:23:17 PDT

(hide)

Description:

Filename:

MIME Type:

Creator: youenn fablet

Created: 2019-07-03 16:23:17 PDT

Size: 1.79 KB

patch

obsolete

>Subversion Revision: 247073
>diff --git a/Source/WebKit/ChangeLog b/Source/WebKit/ChangeLog
>index e06391055158f232a8054d018543d5a190a774b1..5ca2c2f2564c9b06b36655f7680c8bbe5ab8a902 100644
>--- a/Source/WebKit/ChangeLog
>+++ b/Source/WebKit/ChangeLog
>@@ -1,3 +1,13 @@
>+2019-07-03  Youenn Fablet  <youenn@apple.com>
>+
>+        Remove unneeded variable in LocalStorageNamespace::getOrCreateStorageArea
>+        https://bugs.webkit.org/show_bug.cgi?id=199477
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        * NetworkProcess/WebStorage/StorageManager.cpp:
>+        (WebKit::StorageManager::LocalStorageNamespace::getOrCreateStorageArea):
>+
> 2019-07-03  Youenn Fablet  <youenn@apple.com>
> 
>         Make CacheStorage::Engine directory listing operations in a background thread
>diff --git a/Source/WebKit/NetworkProcess/WebStorage/StorageManager.cpp b/Source/WebKit/NetworkProcess/WebStorage/StorageManager.cpp
>index 59f749a753ab2e3678d0ae284572787f180302f3..3ef53117d525dc5a217ff7c96f0ce02651868dd0 100644
>--- a/Source/WebKit/NetworkProcess/WebStorage/StorageManager.cpp
>+++ b/Source/WebKit/NetworkProcess/WebStorage/StorageManager.cpp
>@@ -356,10 +356,8 @@ StorageManager::LocalStorageNamespace::~LocalStorageNamespace()
> 
> auto StorageManager::LocalStorageNamespace::getOrCreateStorageArea(SecurityOriginData&& securityOrigin, IsEphemeral isEphemeral) -> Ref<StorageArea>
> {
>-    RefPtr<StorageArea> protectedStorageArea;
>     return *m_storageAreaMap.ensure(securityOrigin, [&]() mutable {
>-        protectedStorageArea = StorageArea::create(isEphemeral == IsEphemeral::Yes ? nullptr : this, WTFMove(securityOrigin), m_quotaInBytes);
>-        return protectedStorageArea.get();
>+        return StorageArea::create(isEphemeral == IsEphemeral::Yes ? nullptr : this, WTFMove(securityOrigin), m_quotaInBytes);
>     }).iterator->value;
> }
>

Actions: View | Formatted Diff | Diff

Attachments on bug 199477: 373430