Title: [167875] trunk/Source/WebCore
Revision
167875
Author
carlo...@webkit.org
Date
2014-04-28 01:03:00 -0700 (Mon, 28 Apr 2014)

Log Message

[GTK] TextTrack::addCue can raise an exception since r163974
https://bugs.webkit.org/show_bug.cgi?id=132227

Reviewed by Martin Robinson.

webkit_dom_text_track_add_cue() now receives a GError paramater
which is an API break. Add
webkit_dom_text_track_add_cue_with_error and keep
webkit_dom_text_track_add_cue as deprecated to keep API
compatibility.

* bindings/gobject/WebKitDOMDeprecated.cpp:
(webkit_dom_text_track_add_cue):
* bindings/gobject/WebKitDOMDeprecated.h:
* bindings/gobject/WebKitDOMDeprecated.symbols:
* bindings/gobject/webkitdom.symbols:
* bindings/scripts/CodeGeneratorGObject.pm:
(GetEffectiveFunctionName):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (167874 => 167875)


--- trunk/Source/WebCore/ChangeLog	2014-04-28 07:57:18 UTC (rev 167874)
+++ trunk/Source/WebCore/ChangeLog	2014-04-28 08:03:00 UTC (rev 167875)
@@ -1,5 +1,26 @@
 2014-04-28  Carlos Garcia Campos  <cgar...@igalia.com>
 
+        [GTK] TextTrack::addCue can raise an exception since r163974
+        https://bugs.webkit.org/show_bug.cgi?id=132227
+
+        Reviewed by Martin Robinson.
+
+        webkit_dom_text_track_add_cue() now receives a GError paramater
+        which is an API break. Add
+        webkit_dom_text_track_add_cue_with_error and keep
+        webkit_dom_text_track_add_cue as deprecated to keep API
+        compatibility.
+
+        * bindings/gobject/WebKitDOMDeprecated.cpp:
+        (webkit_dom_text_track_add_cue):
+        * bindings/gobject/WebKitDOMDeprecated.h:
+        * bindings/gobject/WebKitDOMDeprecated.symbols:
+        * bindings/gobject/webkitdom.symbols:
+        * bindings/scripts/CodeGeneratorGObject.pm:
+        (GetEffectiveFunctionName):
+
+2014-04-28  Carlos Garcia Campos  <cgar...@igalia.com>
+
         [GTK] TextTrackCue API changed in r163649
         https://bugs.webkit.org/show_bug.cgi?id=132226
 

Modified: trunk/Source/WebCore/bindings/gobject/WebKitDOMDeprecated.cpp (167874 => 167875)


--- trunk/Source/WebCore/bindings/gobject/WebKitDOMDeprecated.cpp	2014-04-28 07:57:18 UTC (rev 167874)
+++ trunk/Source/WebCore/bindings/gobject/WebKitDOMDeprecated.cpp	2014-04-28 08:03:00 UTC (rev 167875)
@@ -29,6 +29,7 @@
 #include "WebKitDOMObject.h"
 #include "WebKitDOMPrivate.h"
 #include "WebKitDOMProcessingInstruction.h"
+#include "WebKitDOMTextTrack.h"
 #include "WebKitDOMVTTCue.h"
 #include "WebKitDOMWebKitNamedFlow.h"
 
@@ -795,3 +796,10 @@
     g_return_if_fail(WEBKIT_DOM_IS_VTT_CUE(self));
     webkit_dom_vtt_cue_set_text(WEBKIT_DOM_VTT_CUE(self), value);
 }
+
+// WebKitDOMTextTrack
+
+void webkit_dom_text_track_add_cue(WebKitDOMTextTrack* self, WebKitDOMTextTrackCue* cue)
+{
+    return webkit_dom_text_track_add_cue_with_error(self, cue, nullptr);
+}

Modified: trunk/Source/WebCore/bindings/gobject/WebKitDOMDeprecated.h (167874 => 167875)


--- trunk/Source/WebCore/bindings/gobject/WebKitDOMDeprecated.h	2014-04-28 07:57:18 UTC (rev 167874)
+++ trunk/Source/WebCore/bindings/gobject/WebKitDOMDeprecated.h	2014-04-28 08:03:00 UTC (rev 167875)
@@ -1013,6 +1013,16 @@
 WEBKIT_DEPRECATED_FOR(webkit_dom_vtt_cue_set_text) void
 webkit_dom_text_track_cue_set_text(WebKitDOMTextTrackCue* self, const gchar* value);
 
+/**
+ * webkit_dom_text_track_add_cue:
+ * @self: A #WebKitDOMTextTrack
+ * @cue: A #WebKitDOMTextTrackCue
+ *
+ * Deprecated: 2.6: Use webkit_dom_text_track_add_cue_with_error() instead.
+ */
+WEBKIT_DEPRECATED_FOR(webkit_dom_text_track_add_cue_with_error) void
+webkit_dom_text_track_add_cue(WebKitDOMTextTrack* self, WebKitDOMTextTrackCue* cue);
+
 G_END_DECLS
 
 #endif /* WEBKIT_DISABLE_DEPRECATED */

Modified: trunk/Source/WebCore/bindings/gobject/WebKitDOMDeprecated.symbols (167874 => 167875)


--- trunk/Source/WebCore/bindings/gobject/WebKitDOMDeprecated.symbols	2014-04-28 07:57:18 UTC (rev 167874)
+++ trunk/Source/WebCore/bindings/gobject/WebKitDOMDeprecated.symbols	2014-04-28 08:03:00 UTC (rev 167875)
@@ -82,3 +82,4 @@
 void webkit_dom_text_track_cue_set_align(WebKitDOMTextTrackCue*, const gchar*, GError**)
 gchar* webkit_dom_text_track_cue_get_text(WebKitDOMTextTrackCue*)
 void webkit_dom_text_track_cue_set_text(WebKitDOMTextTrackCue*, const gchar*)
+void webkit_dom_text_track_add_cue(WebKitDOMTextTrack*, WebKitDOMTextTrackCue*)

Modified: trunk/Source/WebCore/bindings/gobject/webkitdom.symbols (167874 => 167875)


--- trunk/Source/WebCore/bindings/gobject/webkitdom.symbols	2014-04-28 07:57:18 UTC (rev 167874)
+++ trunk/Source/WebCore/bindings/gobject/webkitdom.symbols	2014-04-28 08:03:00 UTC (rev 167875)
@@ -1704,6 +1704,7 @@
 gulong webkit_dom_html_video_element_get_webkit_decoded_frame_count(WebKitDOMHTMLVideoElement*)
 gulong webkit_dom_html_video_element_get_webkit_dropped_frame_count(WebKitDOMHTMLVideoElement*)
 void webkit_dom_text_track_add_cue(WebKitDOMTextTrack*, WebKitDOMTextTrackCue*)
+void webkit_dom_text_track_add_cue_with_error(WebKitDOMTextTrack*, WebKitDOMTextTrackCue*, GError**)
 void webkit_dom_text_track_remove_cue(WebKitDOMTextTrack*, WebKitDOMTextTrackCue*, GError**)
 gboolean webkit_dom_text_track_dispatch_event(WebKitDOMTextTrack*, WebKitDOMEvent*, GError**)
 gchar* webkit_dom_text_track_get_id(WebKitDOMTextTrack*)

Modified: trunk/Source/WebCore/bindings/scripts/CodeGeneratorGObject.pm (167874 => 167875)


--- trunk/Source/WebCore/bindings/scripts/CodeGeneratorGObject.pm	2014-04-28 07:57:18 UTC (rev 167874)
+++ trunk/Source/WebCore/bindings/scripts/CodeGeneratorGObject.pm	2014-04-28 08:03:00 UTC (rev 167875)
@@ -942,6 +942,12 @@
         return "webkit_dom_html_input_element_get_capture_enabled";
     }
 
+    # webkit_dom_text_track_add_cue raises an exception since r163974. We need to add a with_error version to
+    # keep API backwards compatibility.
+    if ($functionName eq "webkit_dom_text_track_add_cue") {
+        return "webkit_dom_text_track_add_cue_with_error";
+    }
+
     return $functionName;
 }
 
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to