[MSVC] Remove experimental lambda processor usage
authordon.olmstead@sony.com <don.olmstead@sony.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 27 Mar 2020 04:31:02 +0000 (04:31 +0000)
committerdon.olmstead@sony.com <don.olmstead@sony.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 27 Mar 2020 04:31:02 +0000 (04:31 +0000)
https://bugs.webkit.org/show_bug.cgi?id=209358

Reviewed by Fujii Hironori.

.:

Remove /experimental:newLambdaProcessor since WebKit is no longer able to build
with this setting as of Visual Studio 16.5.0.

* Source/cmake/OptionsMSVC.cmake:

Source/WebCore:

Fix build for Visual Studio scoping issue for lambdas. The experimental lambada
processor did build this code but is now failing to build WebKit at all.

* dom/DocumentStorageAccess.cpp:
(WebCore::DocumentStorageAccess::requestStorageAccess):

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

ChangeLog
Source/WebCore/ChangeLog
Source/WebCore/dom/DocumentStorageAccess.cpp
Source/cmake/OptionsMSVC.cmake

index 9db14a1..401100e 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,15 @@
+2020-03-26  Don Olmstead  <don.olmstead@sony.com>
+
+        [MSVC] Remove experimental lambda processor usage
+        https://bugs.webkit.org/show_bug.cgi?id=209358
+
+        Reviewed by Fujii Hironori.
+
+        Remove /experimental:newLambdaProcessor since WebKit is no longer able to build
+        with this setting as of Visual Studio 16.5.0.
+
+        * Source/cmake/OptionsMSVC.cmake:
+
 2020-03-26  Fujii Hironori  <Hironori.Fujii@sony.com>
 
         [Win] lld-link: error: /manifestdependency: is not allowed in .drectve
index 598628f..b34489e 100644 (file)
@@ -1,3 +1,16 @@
+2020-03-26  Don Olmstead  <don.olmstead@sony.com>
+
+        [MSVC] Remove experimental lambda processor usage
+        https://bugs.webkit.org/show_bug.cgi?id=209358
+
+        Reviewed by Fujii Hironori.
+
+        Fix build for Visual Studio scoping issue for lambdas. The experimental lambada
+        processor did build this code but is now failing to build WebKit at all.
+
+        * dom/DocumentStorageAccess.cpp:
+        (WebCore::DocumentStorageAccess::requestStorageAccess):
+
 2020-03-26  Peng Liu  <peng.liu6@apple.com>
 
         Swipe down gestures cause the video layer to stick for a moment before bouncing back into place
index be6944d..80a4db7 100644 (file)
@@ -182,7 +182,7 @@ void DocumentStorageAccess::requestStorageAccess(Ref<DeferredPromise>&& promise)
         bool shouldPreserveUserGesture = wasGranted == StorageAccessWasGranted::Yes || promptWasShown == StorageAccessPromptWasShown::No;
 
         if (shouldPreserveUserGesture) {
-            m_document.eventLoop().queueMicrotask([this, weakThis = makeWeakPtr(*this)] {
+            m_document.eventLoop().queueMicrotask([this, weakThis] {
                 if (weakThis)
                     enableTemporaryTimeUserGesture();
             });
@@ -197,7 +197,7 @@ void DocumentStorageAccess::requestStorageAccess(Ref<DeferredPromise>&& promise)
         }
 
         if (shouldPreserveUserGesture) {
-            m_document.eventLoop().queueMicrotask([this, weakThis = makeWeakPtr(*this)] {
+            m_document.eventLoop().queueMicrotask([this, weakThis] {
                 if (weakThis)
                     consumeTemporaryTimeUserGesture();
             });
index 2ab23e4..9f2d755 100644 (file)
@@ -29,11 +29,6 @@ endif ()
 # Specify the source code encoding
 add_compile_options(/utf-8 /validate-charset)
 
-# Enable the new lambda processor for better C++ conformance with /std:c++17
-if (NOT COMPILER_IS_CLANG_CL)
-    add_compile_options(/experimental:newLambdaProcessor)
-endif ()
-
 if (${CMAKE_BUILD_TYPE} MATCHES "Debug")
     set(CMAKE_SHARED_LINKER_FLAGS "${CMAKE_SHARED_LINKER_FLAGS} /OPT:NOREF /OPT:NOICF")
     set(CMAKE_EXE_LINKER_FLAGS "${CMAKE_EXE_LINKER_FLAGS} /OPT:NOREF /OPT:NOICF")