Merge from master

Author: javafxports-github-bot

modules/javafx.media/src/main/native/gstreamer/gstreamer-lite/gst-plugins-bad/gst/aiff/aiffparse.c

@@ -1343,6 +1343,14 @@ gst_aiff_parse_stream_data (GstAiffParse * aiff)
   if (desired >= aiff->bytes_per_sample)
     desired -= (desired % aiff->bytes_per_sample);
 
+#ifdef GSTREAMER_LITE
+  if (desired == 0) {
+    GST_ELEMENT_ERROR (aiff, STREAM, DEMUX, (NULL),
+              ("Invalid stream"));
+    return GST_FLOW_ERROR;
+  }
+#endif // GSTREAMER_LITE
+
   GST_LOG_OBJECT (aiff, "Fetching %" G_GINT64_FORMAT " bytes of data "
       "from the sinkpad", desired);
 

modules/javafx.media/src/main/native/gstreamer/gstreamer-lite/gst-plugins-good/gst/wavparse/gstwavparse.c

@@ -2064,6 +2064,14 @@ gst_wavparse_stream_data (GstWavParse * wav)
   if (desired >= wav->blockalign && wav->blockalign > 0)
     desired -= (desired % wav->blockalign);
 
+#ifdef GSTREAMER_LITE
+  if (desired == 0) {
+    GST_ELEMENT_ERROR (wav, STREAM, DEMUX, (NULL),
+              ("Invalid stream"));
+    return GST_FLOW_ERROR;
+  }
+#endif // GSTREAMER_LITE
+
   GST_LOG_OBJECT (wav, "Fetching %" G_GINT64_FORMAT " bytes of data "
       "from the sinkpad", desired);
 

modules/javafx.web/src/main/java/com/sun/javafx/webkit/theme/RenderThemeImpl.java

@@ -313,26 +313,29 @@ protected Ref createWidget(
         if (ctrl.isManaged()) {
             ctrl.setManaged(false);
         }
-        if (type == WidgetType.SLIDER) {
-            Slider slider = (Slider)ctrl;
-            extParams.order(ByteOrder.nativeOrder());
-            slider.setOrientation(extParams.getInt()==0
-                ? Orientation.HORIZONTAL
-                : Orientation.VERTICAL);
-            slider.setMax(extParams.getFloat());
-            slider.setMin(extParams.getFloat());
-            slider.setValue(extParams.getFloat());
-        } else if (type == WidgetType.PROGRESSBAR) {
-            ProgressBar progress = (ProgressBar)ctrl;
-            extParams.order(ByteOrder.nativeOrder());
-            progress.setProgress(extParams.getInt() == 1
-                    ? extParams.getFloat()
-                    : progress.INDETERMINATE_PROGRESS);
-        } else if (type == WidgetType.METER) {
-            ProgressBar progress = (ProgressBar) ctrl;
-            extParams.order(ByteOrder.nativeOrder());
-            progress.setProgress(extParams.getFloat());
-            progress.setStyle(getMeterStyle(extParams.getInt()));
+
+        if (extParams != null) {
+            if (type == WidgetType.SLIDER) {
+                Slider slider = (Slider)ctrl;
+                extParams.order(ByteOrder.nativeOrder());
+                slider.setOrientation(extParams.getInt()==0
+                    ? Orientation.HORIZONTAL
+                    : Orientation.VERTICAL);
+                slider.setMax(extParams.getFloat());
+                slider.setMin(extParams.getFloat());
+                slider.setValue(extParams.getFloat());
+            } else if (type == WidgetType.PROGRESSBAR) {
+                ProgressBar progress = (ProgressBar)ctrl;
+                extParams.order(ByteOrder.nativeOrder());
+                progress.setProgress(extParams.getInt() == 1
+                        ? extParams.getFloat()
+                        : progress.INDETERMINATE_PROGRESS);
+            } else if (type == WidgetType.METER) {
+                ProgressBar progress = (ProgressBar) ctrl;
+                extParams.order(ByteOrder.nativeOrder());
+                progress.setProgress(extParams.getFloat());
+                progress.setStyle(getMeterStyle(extParams.getInt()));
+            }
         }
         return new FormControlRef(fc);
     }

modules/javafx.web/src/main/native/Source/WebCore/platform/graphics/java/ImageBufferJava.cpp

@@ -491,7 +491,7 @@ Vector<uint8_t> ImageBuffer::toData(const String& mimeType, std::optional<double
         static jmethodID midToData = env->GetMethodID(
                 PG_GetImageClass(env),
                 "toData",
-                "(Ljava/lang/String;)Ljava/lang/String;");
+                "(Ljava/lang/String;)[B");
         ASSERT(midToData);
 
         JLocalRef<jbyteArray> jdata((jbyteArray)env->CallObjectMethod(

modules/javafx.web/src/main/native/Source/WebCore/platform/java/RenderThemeJava.cpp

@@ -177,7 +177,7 @@ bool RenderThemeJava::paintWidget(
     JNIEnv* env = WebCore_GetJavaEnv();
 
     WTF::Vector<jbyte> extParams;
-    if (JNI_EXPAND(SLIDER) == widgetIndex) {
+    if (JNI_EXPAND(SLIDER) == widgetIndex && is<RenderSlider>(object)) {
         HTMLInputElement& input = downcast<RenderSlider>(object).element();
 
         extParams.grow(sizeof(jint) + 3 * sizeof(jfloat));
@@ -200,24 +200,26 @@ bool RenderThemeJava::paintWidget(
         memcpy(data, &valueAsNumber, sizeof(valueAsNumber));
     } else if (JNI_EXPAND(PROGRESS_BAR) == widgetIndex) {
 #if ENABLE(PROGRESS_ELEMENT)
-        RenderProgress& renderProgress = downcast<RenderProgress>(object);
+        if (is<RenderProgress>(object)) {
+            RenderProgress& renderProgress = downcast<RenderProgress>(object);
 
-        extParams.grow(sizeof(jint) + 3*sizeof(jfloat));
-        jbyte *data = extParams.data();
-        auto isDeterminate = jint(renderProgress.isDeterminate() ? 1 : 0);
-        memcpy(data, &isDeterminate, sizeof(isDeterminate));
-        data += sizeof(jint);
+            extParams.grow(sizeof(jint) + 3*sizeof(jfloat));
+            jbyte *data = extParams.data();
+            auto isDeterminate = jint(renderProgress.isDeterminate() ? 1 : 0);
+            memcpy(data, &isDeterminate, sizeof(isDeterminate));
+            data += sizeof(jint);
 
-        auto position = jfloat(renderProgress.position());
-        memcpy(data, &position, sizeof(position));
-        data += sizeof(jfloat);
+            auto position = jfloat(renderProgress.position());
+            memcpy(data, &position, sizeof(position));
+            data += sizeof(jfloat);
 
-        auto animationProgress = jfloat(renderProgress.animationProgress());
-        memcpy(data, &animationProgress, sizeof(animationProgress));
-        data += sizeof(jfloat);
+            auto animationProgress = jfloat(renderProgress.animationProgress());
+            memcpy(data, &animationProgress, sizeof(animationProgress));
+            data += sizeof(jfloat);
 
-        auto animationStartTime = jfloat(renderProgress.animationStartTime());
-        memcpy(data, &animationStartTime, sizeof(animationStartTime));
+            auto animationStartTime = jfloat(renderProgress.animationStartTime());
+            memcpy(data, &animationStartTime, sizeof(animationStartTime));
+        }
 #endif
 #if ENABLE(METER_ELEMENT)
     } else if (JNI_EXPAND(METER) == widgetIndex) {
@@ -228,7 +230,7 @@ bool RenderThemeJava::paintWidget(
             value = meter->valueRatio();
             region = meter->gaugeRegion();
 #if ENABLE(PROGRESS_ELEMENT)
-        } else if (object.isProgress()) {
+        } else if (is<RenderProgress>(object>)) {
             RenderProgress& renderProgress = downcast<RenderProgress>(object);
             value = jfloat(renderProgress.position());
 #endif

modules/javafx.web/src/shims/java/com/sun/javafx/webkit/prism/PrismInvokerShim.java

@@ -0,0 +1,33 @@
+/*
+ * Copyright (c) 2018, Oracle and/or its affiliates. All rights reserved.
+ * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
+ *
+ * This code is free software; you can redistribute it and/or modify it
+ * under the terms of the GNU General Public License version 2 only, as
+ * published by the Free Software Foundation.  Oracle designates this
+ * particular file as subject to the "Classpath" exception as provided
+ * by Oracle in the LICENSE file that accompanied this code.
+ *
+ * This code is distributed in the hope that it will be useful, but WITHOUT
+ * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
+ * FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License
+ * version 2 for more details (a copy is included in the LICENSE file that
+ * accompanied this code).
+ *
+ * You should have received a copy of the GNU General Public License version
+ * 2 along with this work; if not, write to the Free Software Foundation,
+ * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
+ *
+ * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
+ * or visit www.oracle.com if you need additional information or have any
+ * questions.
+ */
+
+package com.sun.javafx.webkit.prism;
+
+public final class PrismInvokerShim {
+    public static void runOnRenderThread(final Runnable r) {
+        PrismInvoker.runOnRenderThread(r);
+    }
+}
+