Diff
Modified: releases/WebKitGTK/webkit-2.18/Source/_javascript_Core/ChangeLog (220661 => 220662)
--- releases/WebKitGTK/webkit-2.18/Source/_javascript_Core/ChangeLog 2017-08-14 09:00:28 UTC (rev 220661)
+++ releases/WebKitGTK/webkit-2.18/Source/_javascript_Core/ChangeLog 2017-08-14 09:03:43 UTC (rev 220662)
@@ -1,3 +1,14 @@
+2017-08-10 Tim Horton <timothy_hor...@apple.com>
+
+ Remove some unused lambda captures so that WebKit builds with -Wunused-lambda-capture
+ https://bugs.webkit.org/show_bug.cgi?id=175436
+ <rdar://problem/33667497>
+
+ Reviewed by Simon Fraser.
+
+ * interpreter/Interpreter.cpp:
+ (JSC::Interpreter::Interpreter):
+
2017-08-09 Caio Lima <ticaiol...@gmail.com>
[JSC] Create JSSet constructor that accepts it's size as parameter
Modified: releases/WebKitGTK/webkit-2.18/Source/_javascript_Core/interpreter/Interpreter.cpp (220661 => 220662)
--- releases/WebKitGTK/webkit-2.18/Source/_javascript_Core/interpreter/Interpreter.cpp 2017-08-14 09:00:28 UTC (rev 220661)
+++ releases/WebKitGTK/webkit-2.18/Source/_javascript_Core/interpreter/Interpreter.cpp 2017-08-14 09:03:43 UTC (rev 220662)
@@ -322,7 +322,7 @@
{
#if !ASSERT_DISABLED
static std::once_flag assertOnceKey;
- std::call_once(assertOnceKey, [this] {
+ std::call_once(assertOnceKey, [] {
for (unsigned i = 0; i < NUMBER_OF_BYTECODE_IDS; ++i) {
OpcodeID opcodeID = static_cast<OpcodeID>(i);
RELEASE_ASSERT(getOpcodeID(getOpcode(opcodeID)) == opcodeID);
Modified: releases/WebKitGTK/webkit-2.18/Source/WebCore/ChangeLog (220661 => 220662)
--- releases/WebKitGTK/webkit-2.18/Source/WebCore/ChangeLog 2017-08-14 09:00:28 UTC (rev 220661)
+++ releases/WebKitGTK/webkit-2.18/Source/WebCore/ChangeLog 2017-08-14 09:03:43 UTC (rev 220662)
@@ -1,3 +1,16 @@
+2017-08-10 Tim Horton <timothy_hor...@apple.com>
+
+ Remove some unused lambda captures so that WebKit builds with -Wunused-lambda-capture
+ https://bugs.webkit.org/show_bug.cgi?id=175436
+ <rdar://problem/33667497>
+
+ Reviewed by Simon Fraser.
+
+ * platform/audio/cocoa/MediaSessionManagerCocoa.cpp:
+ (PlatformMediaSessionManager::updateSessionState):
+ * platform/graphics/cocoa/WebCoreDecompressionSession.mm:
+ (WebCore::WebCoreDecompressionSession::handleDecompressionOutput):
+
2017-08-10 Myles C. Maxfield <mmaxfi...@apple.com>
Addressing post-review comments after r219173.
Modified: releases/WebKitGTK/webkit-2.18/Source/WebCore/platform/audio/cocoa/MediaSessionManagerCocoa.cpp (220661 => 220662)
--- releases/WebKitGTK/webkit-2.18/Source/WebCore/platform/audio/cocoa/MediaSessionManagerCocoa.cpp 2017-08-14 09:00:28 UTC (rev 220661)
+++ releases/WebKitGTK/webkit-2.18/Source/WebCore/platform/audio/cocoa/MediaSessionManagerCocoa.cpp 2017-08-14 09:03:43 UTC (rev 220662)
@@ -62,7 +62,7 @@
bool hasWebAudioType = false;
bool hasAudibleAudioOrVideoMediaType = false;
- bool hasAudioCapture = anyOfSessions([this, &hasWebAudioType, &hasAudibleAudioOrVideoMediaType] (PlatformMediaSession& session, size_t) mutable {
+ bool hasAudioCapture = anyOfSessions([&hasWebAudioType, &hasAudibleAudioOrVideoMediaType] (PlatformMediaSession& session, size_t) mutable {
auto type = session.mediaType();
if (type == PlatformMediaSession::WebAudio)
hasWebAudioType = true;
Modified: releases/WebKitGTK/webkit-2.18/Source/WebCore/platform/graphics/cocoa/WebCoreDecompressionSession.mm (220661 => 220662)
--- releases/WebKitGTK/webkit-2.18/Source/WebCore/platform/graphics/cocoa/WebCoreDecompressionSession.mm 2017-08-14 09:00:28 UTC (rev 220661)
+++ releases/WebKitGTK/webkit-2.18/Source/WebCore/platform/graphics/cocoa/WebCoreDecompressionSession.mm 2017-08-14 09:03:43 UTC (rev 220662)
@@ -224,6 +224,9 @@
void WebCoreDecompressionSession::handleDecompressionOutput(bool displaying, OSStatus status, VTDecodeInfoFlags infoFlags, CVImageBufferRef rawImageBuffer, CMTime presentationTimeStamp, CMTime presentationDuration)
{
+ UNUSED_PARAM(status);
+ UNUSED_PARAM(infoFlags);
+
CMVideoFormatDescriptionRef rawImageBufferDescription = nullptr;
if (noErr != CMVideoFormatDescriptionCreateForImageBuffer(kCFAllocatorDefault, rawImageBuffer, &rawImageBufferDescription))
return;
@@ -240,8 +243,7 @@
return;
RefPtr<WebCoreDecompressionSession> protectedThis { this };
RetainPtr<CMSampleBufferRef> imageSampleBuffer = adoptCF(rawImageSampleBuffer);
- dispatch_async(m_enqueingQueue.get(), [protectedThis, status, imageSampleBuffer, infoFlags, displaying] {
- UNUSED_PARAM(infoFlags);
+ dispatch_async(m_enqueingQueue.get(), [protectedThis, imageSampleBuffer, displaying] {
protectedThis->enqueueDecodedSample(imageSampleBuffer.get(), displaying);
});
}
Modified: releases/WebKitGTK/webkit-2.18/Source/WebKit/ChangeLog (220661 => 220662)
--- releases/WebKitGTK/webkit-2.18/Source/WebKit/ChangeLog 2017-08-14 09:00:28 UTC (rev 220661)
+++ releases/WebKitGTK/webkit-2.18/Source/WebKit/ChangeLog 2017-08-14 09:03:43 UTC (rev 220662)
@@ -1,3 +1,16 @@
+2017-08-10 Tim Horton <timothy_hor...@apple.com>
+
+ Remove some unused lambda captures so that WebKit builds with -Wunused-lambda-capture
+ https://bugs.webkit.org/show_bug.cgi?id=175436
+ <rdar://problem/33667497>
+
+ Reviewed by Simon Fraser.
+
+ * UIProcess/UserMediaPermissionRequestManagerProxy.cpp:
+ (WebKit::UserMediaPermissionRequestManagerProxy::requestUserMediaPermissionForFrame):
+ * UIProcess/WebsiteData/WebsiteDataStore.cpp:
+ (WebKit::WebsiteDataStore::topPrivatelyControlledDomainsWithWebsiteData):
+
2017-08-10 Zan Dobersek <zdober...@igalia.com>
[GTK] Don't use --whole-archive linking flags for the WebKit2 target libraries
Modified: releases/WebKitGTK/webkit-2.18/Source/WebKit/UIProcess/UserMediaPermissionRequestManagerProxy.cpp (220661 => 220662)
--- releases/WebKitGTK/webkit-2.18/Source/WebKit/UIProcess/UserMediaPermissionRequestManagerProxy.cpp 2017-08-14 09:00:28 UTC (rev 220661)
+++ releases/WebKitGTK/webkit-2.18/Source/WebKit/UIProcess/UserMediaPermissionRequestManagerProxy.cpp 2017-08-14 09:03:43 UTC (rev 220662)
@@ -305,7 +305,7 @@
userMediaAccessWasDenied(userMediaID, UserMediaPermissionRequestProxy::UserMediaAccessDenialReason::UserMediaDisabled);
};
- auto haveDeviceSaltHandler = [this, userMediaID, validHandler = WTFMove(validHandler), invalidHandler = WTFMove(invalidHandler), audioConstraints = WebCore::MediaConstraints(audioConstraints), videoConstraints = WebCore::MediaConstraints(videoConstraints)](uint64_t userMediaID, String&& deviceIdentifierHashSalt, bool originHasPersistentAccess) mutable {
+ auto haveDeviceSaltHandler = [this, validHandler = WTFMove(validHandler), invalidHandler = WTFMove(invalidHandler), audioConstraints = WebCore::MediaConstraints(audioConstraints), videoConstraints = WebCore::MediaConstraints(videoConstraints)](uint64_t userMediaID, String&& deviceIdentifierHashSalt, bool originHasPersistentAccess) mutable {
auto request = m_pendingDeviceRequests.take(userMediaID);
if (!request)
Modified: releases/WebKitGTK/webkit-2.18/Source/WebKit/UIProcess/WebsiteData/WebsiteDataStore.cpp (220661 => 220662)
--- releases/WebKitGTK/webkit-2.18/Source/WebKit/UIProcess/WebsiteData/WebsiteDataStore.cpp 2017-08-14 09:00:28 UTC (rev 220661)
+++ releases/WebKitGTK/webkit-2.18/Source/WebKit/UIProcess/WebsiteData/WebsiteDataStore.cpp 2017-08-14 09:03:43 UTC (rev 220662)
@@ -538,7 +538,7 @@
void WebsiteDataStore::topPrivatelyControlledDomainsWithWebsiteData(OptionSet<WebsiteDataType> dataTypes, OptionSet<WebsiteDataFetchOption> fetchOptions, Function<void(HashSet<String>&&)>&& completionHandler)
{
- fetchData(dataTypes, fetchOptions, [completionHandler = WTFMove(completionHandler), this, protectedThis = makeRef(*this)](auto&& existingDataRecords) {
+ fetchData(dataTypes, fetchOptions, [completionHandler = WTFMove(completionHandler), protectedThis = makeRef(*this)](auto&& existingDataRecords) {
HashSet<String> domainsWithDataRecords;
for (auto&& dataRecord : existingDataRecords) {
String domain = dataRecord.topPrivatelyControlledDomain();
Modified: releases/WebKitGTK/webkit-2.18/Tools/ChangeLog (220661 => 220662)
--- releases/WebKitGTK/webkit-2.18/Tools/ChangeLog 2017-08-14 09:00:28 UTC (rev 220661)
+++ releases/WebKitGTK/webkit-2.18/Tools/ChangeLog 2017-08-14 09:03:43 UTC (rev 220662)
@@ -1,3 +1,14 @@
+2017-08-10 Tim Horton <timothy_hor...@apple.com>
+
+ Remove some unused lambda captures so that WebKit builds with -Wunused-lambda-capture
+ https://bugs.webkit.org/show_bug.cgi?id=175436
+ <rdar://problem/33667497>
+
+ Reviewed by Simon Fraser.
+
+ * TestWebKitAPI/Tests/WebCore/FileMonitor.cpp:
+ (TestWebKitAPI::TEST_F):
+
2017-08-09 Don Olmstead <don.olmst...@sony.com>
[WTF] Move TextStream into WTF
Modified: releases/WebKitGTK/webkit-2.18/Tools/TestWebKitAPI/Tests/WebCore/FileMonitor.cpp (220661 => 220662)
--- releases/WebKitGTK/webkit-2.18/Tools/TestWebKitAPI/Tests/WebCore/FileMonitor.cpp 2017-08-14 09:00:28 UTC (rev 220661)
+++ releases/WebKitGTK/webkit-2.18/Tools/TestWebKitAPI/Tests/WebCore/FileMonitor.cpp 2017-08-14 09:03:43 UTC (rev 220662)
@@ -149,7 +149,7 @@
auto testQueue = WorkQueue::create("Test Work Queue");
- auto monitor = std::make_unique<FileMonitor>(tempFilePath(), testQueue.copyRef(), [this] (FileMonitor::FileChangeType type) {
+ auto monitor = std::make_unique<FileMonitor>(tempFilePath(), testQueue.copyRef(), [] (FileMonitor::FileChangeType type) {
ASSERT(!RunLoop::isMain());
switch (type) {
case FileMonitor::FileChangeType::Modification:
@@ -191,7 +191,7 @@
auto testQueue = WorkQueue::create("Test Work Queue");
- auto monitor = std::make_unique<FileMonitor>(tempFilePath(), testQueue.copyRef(), [this] (FileMonitor::FileChangeType type) {
+ auto monitor = std::make_unique<FileMonitor>(tempFilePath(), testQueue.copyRef(), [] (FileMonitor::FileChangeType type) {
ASSERT(!RunLoop::isMain());
switch (type) {
case FileMonitor::FileChangeType::Modification:
@@ -251,7 +251,7 @@
auto testQueue = WorkQueue::create("Test Work Queue");
- auto monitor = std::make_unique<FileMonitor>(tempFilePath(), testQueue.copyRef(), [this] (FileMonitor::FileChangeType type) {
+ auto monitor = std::make_unique<FileMonitor>(tempFilePath(), testQueue.copyRef(), [] (FileMonitor::FileChangeType type) {
ASSERT(!RunLoop::isMain());
switch (type) {
case FileMonitor::FileChangeType::Modification:
@@ -290,7 +290,7 @@
auto testQueue = WorkQueue::create("Test Work Queue");
- auto monitor = std::make_unique<FileMonitor>(tempFilePath(), testQueue.copyRef(), [this] (FileMonitor::FileChangeType type) {
+ auto monitor = std::make_unique<FileMonitor>(tempFilePath(), testQueue.copyRef(), [] (FileMonitor::FileChangeType type) {
ASSERT(!RunLoop::isMain());
switch (type) {
case FileMonitor::FileChangeType::Modification:
@@ -347,7 +347,7 @@
auto testQueue = WorkQueue::create("Test Work Queue");
- auto monitor = std::make_unique<FileMonitor>(tempFilePath(), testQueue.copyRef(), [this] (FileMonitor::FileChangeType type) {
+ auto monitor = std::make_unique<FileMonitor>(tempFilePath(), testQueue.copyRef(), [] (FileMonitor::FileChangeType type) {
ASSERT(!RunLoop::isMain());
switch (type) {
case FileMonitor::FileChangeType::Modification: