[Windows] Build fix after r155621
authorbfulgham@apple.com <bfulgham@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 12 Sep 2013 17:46:39 +0000 (17:46 +0000)
committerbfulgham@apple.com <bfulgham@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 12 Sep 2013 17:46:39 +0000 (17:46 +0000)
Rubber-stamped by Anders Carlsson.

See also https://bugs.webkit.org/show_bug.cgi?id=121235 for
why this was necessary.

* css/MediaQueryEvaluator.cpp: Work around VS compiler bug.
* dom/make_names.pl: Ditto.
* loader/archive/ArchiveFactory.cpp: Ditto.
* rendering/svg/SVGPathData.cpp: Ditto.

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

Source/WebCore/ChangeLog
Source/WebCore/css/MediaQueryEvaluator.cpp
Source/WebCore/dom/make_names.pl
Source/WebCore/loader/archive/ArchiveFactory.cpp
Source/WebCore/rendering/svg/SVGPathData.cpp

index 9f32cf5..0bf7df5 100644 (file)
@@ -1,3 +1,17 @@
+2013-09-12  Brent Fulgham  <bfulgham@apple.com>
+
+        [Windows] Build fix after r155621
+
+        Rubber-stamped by Anders Carlsson.
+
+        See also https://bugs.webkit.org/show_bug.cgi?id=121235 for
+        why this was necessary.
+
+        * css/MediaQueryEvaluator.cpp: Work around VS compiler bug.
+        * dom/make_names.pl: Ditto.
+        * loader/archive/ArchiveFactory.cpp: Ditto.
+        * rendering/svg/SVGPathData.cpp: Ditto.
+
 2013-09-12  Anders Carlsson  <andersca@apple.com>
 
         SharedBuffer::createCFData should return RetainPtr<CFDataRef>
index 9758560..7c766c4 100644 (file)
@@ -684,12 +684,14 @@ static bool pointerMediaFeatureEval(CSSValue* value, RenderStyle*, Frame* frame,
         || (pointer == MousePointer && id == CSSValueFine);
 }
 
+// FIXME: Remove unnecessary '&' from the following 'ADD_TO_FUNCTIONMAP' definition
+// once we switch to a non-broken Visual Studio compiler.  https://bugs.webkit.org/show_bug.cgi?id=121235
 static void createFunctionMap()
 {
     // Create the table.
     gFunctionMap = new FunctionMap;
 #define ADD_TO_FUNCTIONMAP(name, str)  \
-    gFunctionMap->set(name##MediaFeature.impl(), name##MediaFeatureEval);
+    gFunctionMap->set(name##MediaFeature.impl(), &name##MediaFeatureEval);
     CSS_MEDIAQUERY_NAMES_FOR_EACH_MEDIAFEATURE(ADD_TO_FUNCTIONMAP);
 #undef ADD_TO_FUNCTIONMAP
 }
index 8032394..18ef090 100755 (executable)
@@ -1,6 +1,6 @@
 #!/usr/bin/perl -w
 
-# Copyright (C) 2005, 2006, 2007, 2009 Apple Inc. All rights reserved.
+# Copyright (C) 2005, 2006, 2007, 2009, 2013 Apple Inc. All rights reserved.
 # Copyright (C) 2009, Julien Chaffraix <jchaffraix@webkit.org>
 # Copyright (C) 2009 Torch Mobile Inc. All rights reserved. (http://www.torchmobile.com/)
 # Copyright (C) 2011 Ericsson AB. All rights reserved.
@@ -1230,7 +1230,9 @@ END
         }
 
         my $ucTag = $enabledTags{$tag}{JSInterfaceName};
-        print F "       map.set(${tag}Tag.localName().impl(), create${ucTag}Wrapper);\n";
+        # FIXME Remove unnecessary '&' from the following (print) line once we switch to a non-broken Visual Studio compiler.
+        # https://bugs.webkit.org/show_bug.cgi?id=121235:
+        print F "       map.set(${tag}Tag.localName().impl(), &create${ucTag}Wrapper);\n";
 
         if ($conditional) {
             print F "#endif\n";
index fbbbaa1..14c81fd 100644 (file)
@@ -1,5 +1,5 @@
 /*
- * Copyright (C) 2008 Apple Inc. All rights reserved.
+ * Copyright (C) 2008, 2013 Apple Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
@@ -63,12 +63,14 @@ static ArchiveMIMETypesMap& archiveMIMETypes()
     if (initialized)
         return mimeTypes;
 
+    // FIXME: Remove unnecessary 'static_cast<RawDataCreationFunction*>' from the following 'mimeTypes.set' operations
+    // once we switch to a non-broken Visual Studio compiler.  https://bugs.webkit.org/show_bug.cgi?id=121235
 #if ENABLE(WEB_ARCHIVE) && USE(CF)
-    mimeTypes.set("application/x-webarchive", archiveFactoryCreate<LegacyWebArchive>);
+    mimeTypes.set("application/x-webarchive", static_cast<RawDataCreationFunction*>(&archiveFactoryCreate<LegacyWebArchive>));
 #endif
 #if ENABLE(MHTML)
-    mimeTypes.set("multipart/related", archiveFactoryCreate<MHTMLArchive>);
-    mimeTypes.set("application/x-mimearchive", archiveFactoryCreate<MHTMLArchive>);
+    mimeTypes.set("multipart/related", static_cast<RawDataCreationFunction*>(&archiveFactoryCreate<MHTMLArchive>));
+    mimeTypes.set("application/x-mimearchive", static_cast<RawDataCreationFunction*>(&archiveFactoryCreate<MHTMLArchive>));
 #endif
 
     initialized = true;
index 97b5deb..a33fff4 100644 (file)
@@ -151,13 +151,13 @@ void updatePathFromGraphicsElement(SVGElement* element, Path& path)
     static HashMap<AtomicStringImpl*, PathUpdateFunction>* map = 0;
     if (!map) {
         map = new HashMap<AtomicStringImpl*, PathUpdateFunction>;
-        map->set(SVGNames::circleTag.localName().impl(), updatePathFromCircleElement);
-        map->set(SVGNames::ellipseTag.localName().impl(), updatePathFromEllipseElement);
-        map->set(SVGNames::lineTag.localName().impl(), updatePathFromLineElement);
-        map->set(SVGNames::pathTag.localName().impl(), updatePathFromPathElement);
-        map->set(SVGNames::polygonTag.localName().impl(), updatePathFromPolygonElement);
-        map->set(SVGNames::polylineTag.localName().impl(), updatePathFromPolylineElement);
-        map->set(SVGNames::rectTag.localName().impl(), updatePathFromRectElement);
+        map->set(SVGNames::circleTag.localName().impl(), &updatePathFromCircleElement);
+        map->set(SVGNames::ellipseTag.localName().impl(), &updatePathFromEllipseElement);
+        map->set(SVGNames::lineTag.localName().impl(), &updatePathFromLineElement);
+        map->set(SVGNames::pathTag.localName().impl(), &updatePathFromPathElement);
+        map->set(SVGNames::polygonTag.localName().impl(), &updatePathFromPolygonElement);
+        map->set(SVGNames::polylineTag.localName().impl(), &updatePathFromPolylineElement);
+        map->set(SVGNames::rectTag.localName().impl(), &updatePathFromRectElement);
     }
 
     if (PathUpdateFunction pathUpdateFunction = map->get(element->localName().impl()))