Attachment 340497 Details for Bug 185685 – Patch

[patch] Patch

bug-185685-20180516100418.patch (text/plain), 2.24 KB, created by Andy Estes on 2018-05-16 10:04:19 PDT

(hide)

Description:

Filename:

MIME Type:

Creator: Andy Estes

Created: 2018-05-16 10:04:19 PDT

Size: 2.24 KB

patch

obsolete

>Subversion Revision: 231770
>diff --git a/Source/WebKit/ChangeLog b/Source/WebKit/ChangeLog
>index 2ef3654515e3df3c92b1872a91b4517875795f82..9c61b2d7b9856d002f4b9ab42da44815432f04d1 100644
>--- a/Source/WebKit/ChangeLog
>+++ b/Source/WebKit/ChangeLog
>@@ -1,3 +1,16 @@
>+2018-05-16  Andy Estes  <aestes@apple.com>
>+
>+        [Wi-Fi Assertions] Adopt WiFiAssertionHolderAdditions
>+        https://bugs.webkit.org/show_bug.cgi?id=185685
>+        <rdar://problem/40136681>
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        * NetworkProcess/cocoa/WiFiAssertionHolder.cpp:
>+        (WebKit::WiFiAssertionHolder::WiFiAssertionHolder):
>+        (WebKit::WiFiAssertionHolder::~WiFiAssertionHolder):
>+        (WebKit::ensureWiFiManagerClient): Deleted.
>+
> 2018-05-14  Andy Estes  <aestes@apple.com>
> 
>         [Wi-Fi Assertions] Allow clients to specify a context identifier
>diff --git a/Source/WebKit/NetworkProcess/cocoa/WiFiAssertionHolder.cpp b/Source/WebKit/NetworkProcess/cocoa/WiFiAssertionHolder.cpp
>index 423be70e608f4cd1127b7284be5de21a541a003f..9def018ba7a10b8290dfc491e986ac1d9d9b4a65 100644
>--- a/Source/WebKit/NetworkProcess/cocoa/WiFiAssertionHolder.cpp
>+++ b/Source/WebKit/NetworkProcess/cocoa/WiFiAssertionHolder.cpp
>@@ -28,36 +28,20 @@
> 
> #if HAVE(MOBILE_WIFI)
> 
>-#include "Logging.h"
>-#include "MobileWiFiSPI.h"
>+#if USE(APPLE_INTERNAL_SDK)
>+#include <WebKitAdditions/WiFiAssertionHolderAdditions.cpp>
>+#endif
> 
> namespace WebKit {
> 
>-static WiFiManagerClientRef ensureWiFiManagerClient()
>-{
>-    static WiFiManagerClientRef wiFiManagerClient = WiFiManagerClientCreate(kCFAllocatorDefault, kWiFiClientTypeNormal);
>-    return wiFiManagerClient;
>-}
>-
>-static uint64_t wiFiAssertionCount;
>-
> WiFiAssertionHolder::WiFiAssertionHolder()
> {
>-    if (wiFiAssertionCount++)
>-        return;
>-
>-    RELEASE_LOG(WiFiAssertions, "Acquiring Wi-Fi assertion.");
>-    WiFiManagerClientSetType(ensureWiFiManagerClient(), kWiFiClientTypeBackground);
>+    holdAssertion();
> }
> 
> WiFiAssertionHolder::~WiFiAssertionHolder()
> {
>-    ASSERT(wiFiAssertionCount);
>-    if (--wiFiAssertionCount)
>-        return;
>-
>-    RELEASE_LOG(WiFiAssertions, "Releasing Wi-Fi assertion.");
>-    WiFiManagerClientSetType(ensureWiFiManagerClient(), kWiFiClientTypeNormal);
>+    releaseAssertion();
> }
> 
> } // namespace WebKit

Actions: View | Formatted Diff | Diff

Attachments on bug 185685: 340497 | 340513