Title: [230616] trunk/Source/WebCore
Revision
230616
Author
dba...@webkit.org
Date
2018-04-12 17:11:06 -0700 (Thu, 12 Apr 2018)

Log Message

Attmept to fix the Windows build following <https://trac.webkit.org/changeset/230602/>
(https://bugs.webkit.org/show_bug.cgi?id=184386)

Visual Studio cannot prove that all cases in the switch are covered :(
So, add an ASSERT_NOT_REACHED() and a return a value outside of the switch block.

* loader/cache/CachedResourceLoader.cpp:
(WebCore::destinationForType):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (230615 => 230616)


--- trunk/Source/WebCore/ChangeLog	2018-04-13 00:02:48 UTC (rev 230615)
+++ trunk/Source/WebCore/ChangeLog	2018-04-13 00:11:06 UTC (rev 230616)
@@ -1,3 +1,14 @@
+2018-04-12  Daniel Bates  <daba...@apple.com>
+
+        Attmept to fix the Windows build following <https://trac.webkit.org/changeset/230602/>
+        (https://bugs.webkit.org/show_bug.cgi?id=184386)
+
+        Visual Studio cannot prove that all cases in the switch are covered :(
+        So, add an ASSERT_NOT_REACHED() and a return a value outside of the switch block. 
+
+        * loader/cache/CachedResourceLoader.cpp:
+        (WebCore::destinationForType):
+
 2018-04-12  Chris Dumez  <cdu...@apple.com>
 
         Introduce remote variants of Frame / DOMWindow classes

Modified: trunk/Source/WebCore/loader/cache/CachedResourceLoader.cpp (230615 => 230616)


--- trunk/Source/WebCore/loader/cache/CachedResourceLoader.cpp	2018-04-13 00:02:48 UTC (rev 230615)
+++ trunk/Source/WebCore/loader/cache/CachedResourceLoader.cpp	2018-04-13 00:11:06 UTC (rev 230616)
@@ -766,6 +766,8 @@
         // The caller is responsible for setting the appropriate destination.
         return FetchOptions::Destination::EmptyString;
     }
+    ASSERT_NOT_REACHED();
+    return FetchOptions::Destination::EmptyString;
 }
 
 ResourceErrorOr<CachedResourceHandle<CachedResource>> CachedResourceLoader::requestResource(CachedResource::Type type, CachedResourceRequest&& request, ForPreload forPreload, DeferOption defer)
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to