WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
148433
[GStreamer] "method" property for the webkitwebsrc element
https://bugs.webkit.org/show_bug.cgi?id=148433
Summary
[GStreamer] "method" property for the webkitwebsrc element
Philippe Normand
Reported
2015-08-25 08:52:43 PDT
The GStreamer uridownloader element makes use of this property in some cases for DASH. See also
https://bugzilla.gnome.org/show_bug.cgi?id=752413
So we should also add this property in webkitwebsrc, the patch is trivial.
Attachments
patch
(4.02 KB, patch)
2015-08-25 09:00 PDT
,
Philippe Normand
cgarcia
: review+
Details
Formatted Diff
Diff
View All
Add attachment
proposed patch, testcase, etc.
Philippe Normand
Comment 1
2015-08-25 09:00:39 PDT
Created
attachment 259847
[details]
patch
Carlos Garcia Campos
Comment 2
2015-08-26 00:27:05 PDT
Comment on
attachment 259847
[details]
patch View in context:
https://bugs.webkit.org/attachment.cgi?id=259847&action=review
> Source/WebCore/platform/graphics/gstreamer/WebKitWebSourceGStreamer.cpp:116 > + GUniquePtr<gchar> method;
method is quite confusing name, we can't change the property name, of course, but at least the variable could be httpMethod or something like that, I didn't know what this was about until I read the full patch.
> Source/WebCore/platform/graphics/gstreamer/WebKitWebSourceGStreamer.cpp:269 > + 0, static_cast<GParamFlags>(G_PARAM_READWRITE | G_PARAM_STATIC_STRINGS)));
nullptr.
Philippe Normand
Comment 3
2015-08-26 02:51:44 PDT
Committed
r188961
: <
http://trac.webkit.org/changeset/188961
>
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug