Instead, log an error in the console.
Created attachment 319631 [details] Patch
Created attachment 319635 [details] Patch
Comment on attachment 319635 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=319635&action=review > Source/WebCore/Modules/cache/DOMCacheEngine.h:88 > + template<class Decoder> static bool decode(Decoder&, CacheIdentifierOperationResult&); This could return a std::optional<CacheIdentifierOperationResult>
Thanks for the review. (In reply to Alex Christensen from comment #3) > Comment on attachment 319635 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=319635&action=review > > > Source/WebCore/Modules/cache/DOMCacheEngine.h:88 > > + template<class Decoder> static bool decode(Decoder&, CacheIdentifierOperationResult&); > > This could return a std::optional<CacheIdentifierOperationResult> Yes, I plan to update these two decoders to the modern ones. Note though that in the case it might be end up in more lines of code.
Comment on attachment 319635 [details] Patch Rejecting attachment 319635 [details] from commit-queue. Failed to run "['/Volumes/Data/EWS/WebKit/Tools/Scripts/webkit-patch', '--status-host=webkit-queues.webkit.org', '--bot-id=webkit-cq-01', 'apply-attachment', '--no-update', '--non-interactive', 319635, '--port=mac']" exit_code: 2 cwd: /Volumes/Data/EWS/WebKit Last 500 characters of output: ebCore/Modules/cache/DOMCacheEngine.h patching file Source/WebCore/Modules/cache/WorkerCacheStorageConnection.cpp patching file Source/WebKit/NetworkProcess/cache/CacheStorageEngine.cpp patching file Source/WebKit/NetworkProcess/cache/CacheStorageEngineCaches.cpp patching file Source/WebKit/NetworkProcess/cache/CacheStorageEngineCaches.h Failed to run "[u'/Volumes/Data/EWS/WebKit/Tools/Scripts/svn-apply', '--force', '--reviewer', u'Alex Christensen']" exit_code: 1 cwd: /Volumes/Data/EWS/WebKit Full output: http://webkit-queues.webkit.org/results/4429024
Created attachment 319654 [details] Patch for landing
Created attachment 319659 [details] Patch for landing
Comment on attachment 319659 [details] Patch for landing Clearing flags on attachment: 319659 Committed r221506: <http://trac.webkit.org/changeset/221506>
All reviewed patches have been landed. Closing bug.
<rdar://problem/34693714>