Commit Graph

44 Commits

Author SHA1 Message Date
Benoit Jacob
b41662d39d Bug 1035394 - Fix dangerous public destructors in content/media - r=cpearce 2014-07-08 17:23:16 -04:00
Ehsan Akhgari
ac692ed2ca Bug 1024463 - Define an out of line destructor for TextTrackCue so that we can forward delcare HTMLTrackElement in TextTrackCue.h; r=bzbarsky
This is required because the destructor for the nsRefPtr<HTMLTrackElement>
member requires the full definition of the type.
2014-06-12 11:31:18 -04:00
Rick Eyre
5433bf6d01 Bug 976580 - Set cue state to dirty when video resize happens. r=rillian, r=bz
This way when the video is resized the rendering algorithm for the cues
will run again and hence, the caption size will be increased/decreased
for the new size of the video.
2014-04-07 15:42:33 -04:00
Rick Eyre
47dcbf2ce9 Bug 950308 - TextTrack, TextTrackCue, TextTrackList should BindToOwner. r=smaug 2014-04-07 13:58:38 -04:00
Boris Zbarsky
fdbb455d4f Bug 991742 part 8. Remove the "aScope" argument of WebIDL/nsWrapperCache WrapObject() methods. r=bholley
This patch was mostly generated with the following command:

find . -name "*.h" -o -name "*.cpp" | xargs sed -e '/WrapObject(JSContext/ {; N; s/\(WrapObject(JSContext *\* *a\{0,1\}[Cc]x\),\n\{0,1\} *JS::Handle<JSObject\*> a\{0,1\}[sS]cope/\1/ ; }' -i ""

and then reverting the changes that made to
dom/bindings/BindingUtils.h, since those WrapObject methods are not
the ones we're trying to change here, plus a bunch of manual fixups
for cases that this command did not catch (including all the callsites
of WrapObject()).
2014-04-08 18:27:18 -04:00
Rick Eyre
41a90c6dc2 Bug 879431 - Set VTTCue's TextTrack from TextTrack::AddCue r=rillian
We were having an issue where the cue's track property was undefined
because it was never set when adding the cue to a text track's list
of cues.
2014-04-02 18:16:00 +02:00
Masayuki Nakano
b185105992 Bug 984269 Rename nsDOMEventTargetHelper to mozilla::DOMEventTargetHelper r=smaug
--HG--
rename : dom/events/nsDOMEventTargetHelper.cpp => dom/events/DOMEventTargetHelper.cpp
rename : dom/events/nsDOMEventTargetHelper.h => dom/events/DOMEventTargetHelper.h
2014-04-01 15:13:50 +09:00
Rick Eyre
ca6fa01a7e Bug 985484 - Remove TextTrackCue::CueChange. r=cpearce 2014-03-19 11:01:38 -04:00
Rick Eyre
5b292b121e Bug 978163 - Part 2: VTTCue should have Region property. r=rillian, r=bz
See spec bug: https://www.w3.org/Bugs/Public/show_bug.cgi?id=24380
2014-03-11 13:33:58 -04:00
Rick Eyre
477050afba Bug 967157 - Setting TextTrackCue::DisplayState should set TextTrackCue::HasBeenReset to false. r=rillian 2014-02-03 15:27:51 -05:00
Rick Eyre
dbde66be6a Bug 965336 - Setting TextTrackCue's Start or End times should set its state to dirty. r=rillian 2014-01-29 12:57:43 -05:00
Rick Eyre
1fe94a73b3 Bug 882299 - Implement VTTCue::Line r=smaug,rillian,bz 2014-01-15 08:04:00 -08:00
Rick Eyre
4e420f1c25 Bug 865407 - Part 6: Expose VTTCue::Reset to Chrome JS. r=bz 2013-12-12 13:13:38 -05:00
Rick Eyre
78e1d23095 Bug 865407 - Part 5: Move update display of cues code to TextTrackManager. r=bz, r=rillian 2013-12-12 12:02:17 -05:00
Rick Eyre
4739efe22b Bug 865407 - Part 4: Expose TextTrackCue::DisplayState to chrome JS. r=rillian, r=bz 2013-12-12 11:27:30 -05:00
Ryan VanderMeulen
d65ecca981 Backed out 5 changesets (bug 865407) for Windows crashes.
Backed out changeset d3e7f6392670 (bug 865407)
Backed out changeset 7ba94f08b7a5 (bug 865407)
Backed out changeset 3de6e32a4346 (bug 865407)
Backed out changeset 5c0572c6727b (bug 865407)
Backed out changeset 5db43cac79f5 (bug 865407)
2014-01-10 11:50:49 -05:00
Rick Eyre
d6dc5e74a0 Bug 865407 - Part 6: Expose VTTCue::Reset to Chrome JS. r=bz 2013-12-12 13:13:38 -05:00
Rick Eyre
7b7b323893 Bug 865407 - Part 5: Move update display of cues code to TextTrackManager. r=bz, r=rillian 2013-12-12 12:02:17 -05:00
Rick Eyre
57a293b49e Bug 865407 - Part 4: Expose TextTrackCue::DisplayState to chrome JS. r=rillian, r=bz 2013-12-12 11:27:30 -05:00
Rick Eyre
af35c2dfc6 Bug 949643 - Implement VTTCue::PositionAlign. r=rillian 2013-12-13 12:57:48 -05:00
Rick Eyre
86640b66f3 Bug 950830 - Clean up TextTrackCue style and TODO comments. r=rillian 2013-12-16 14:49:22 -05:00
Rick Eyre
c4a5a3c5a6 Bug 949642 - Implement VTTCue::LineAlign. r=rillian 2013-12-13 09:29:15 -05:00
Wes Kocher
19e7b8c082 Backed out changeset df0855f26e4d (bug 949642) for introducing a new intermittent failure 2013-12-20 18:33:55 -08:00
Wes Kocher
58a13fe0cb Backed out changeset 6532bf066bb4 (bug 882299) 2013-12-20 18:33:04 -08:00
Wes Kocher
9c61949d24 Backed out changeset 00aa14a1f461 (bug 950830) 2013-12-20 18:32:20 -08:00
Rick Eyre
5655b03c49 Bug 950830 - Clean up TextTrackCue style and TODO comments. r=rillian 2013-12-16 14:49:22 -05:00
Rick Eyre
68a00515c1 Bug 882299 - Implement VTTCue::Line. r=smaug, r=rillian 2013-12-16 14:35:01 -05:00
Rick Eyre
4fdaf8fb73 Bug 949642 - Implement VTTCue::LineAlign. r=rillian 2013-12-13 09:29:15 -05:00
Rick Eyre
2837cf469c Bug 950049 - Rename VTTCue::TextTrackCueAlign to AlignSetting. r=rillian 2013-12-13 09:44:50 -05:00
Ehsan Akhgari
65f992afec Bug 922933 - Don't #include nsIDocument.h in nsIContent.h; r=jst
--HG--
extra : rebase_source : 51f56fa27604224eacc2ba384bb826c49f4f917f
2013-10-02 07:40:07 -04:00
Rick Eyre
d6b2116792 Bug 917945 - Part 2: Add RegionId attribute to VTTCue. r=rillian 2013-09-18 13:29:20 -04:00
Rick Eyre
aa35aad13b Bug 895091 - Part 2: Add WebVTT parser and wrapper IDls r=rillian, ted, gwagner, khuey
- Added vtt.js, a JS WebVTT parser, to Gecko.
- Added a wrapper to interface between it and Gecko.
- Updated TextTrackCue and WebVTTListener to work with it.

--HG--
rename : content/media/WebVTTLoadListener.cpp => content/media/WebVTTListener.cpp
rename : content/media/WebVTTLoadListener.h => content/media/WebVTTListener.h
2013-09-19 08:26:00 -07:00
Rick Eyre
d59dfb9494 Bug 895091 - Part 1: Remove references to C WebVTT parser and unused code r=rillian
- Remove left over references to WebVTT C parser in TextTrackCue and
WebVTTListener classes.
- Remove miscellaneous unused code as well and included libraries as
well.
2013-09-10 06:53:00 -07:00
Marcus Saad
90fe20b7ad Bug 903425 - Update TextTrackCue::Vertical to be a DirectionSetting enum. r=rillian
Following up with the changes made in http://dev.w3.org/html5/webvtt/#dfn-dom-vttcue-vertical, this patch comes to create a new enum to validate and control the possible values for TextTrackCue::Vertical. Throwing behavior was removed for spec conformance.
2013-09-12 11:07:14 -03:00
Rick Eyre
d91df96fac Bug 868509 - Rename TextTrackCue to VTTCue r=rillian
This way we can ensure that JS code written for WebVTT is spec
compliant while at the same time not having to abstract VTTCue
from TextTrackCue.

--HG--
rename : dom/webidl/TextTrackCue.webidl => dom/webidl/VTTCue.webidl
2013-08-27 13:07:00 -07:00
Kyle Huey
bf31d78a83 Bug 903772: Part 3 - Refactor GlobalObject. r=peterv 2013-08-22 22:17:08 -07:00
Rick Eyre
7b90a3a973 Bug 905320 - Fix TextTrack classes to utilize cycle collection correctly. r=khuey
We were incorrectly using the wrapper cache cycle collection macros when that
functionality was already implemented by the base classes. Therefore, we
should instead use the inherited version of the cycle collection macros.
2013-08-15 11:11:15 -04:00
Marcus Saad
664e93ad34 Bug 882743 - Make TextTrackCue setters throw. r=rillian, r=bz
From 8f5dccf7fd5f5d2a74341ead60586346e4ab3ead Mon Sep 17 00:00:00 2001
Now throwing javascript exceptions from attribute setters for TextTrack-related DOM classes.
As can be seen in http://dev.w3.org/html5/webvtt/#webvtt-api, some of the attributes require validation and throwing.
There is no validation implemented to the align property because there is a bug opened against the spec to change it to an Enum.
When cleared, it should have the IDL changed to remove the throwing.
Validation comments were removed from the properties that didn't have proper validation defined by the spec.
2013-07-24 14:18:54 -04:00
Ralph Giles
05565a4994 Bug 894104 - Part 3: remove TextTrackCue's mGlobal. r=bz
From 095c76c72dd5915b2fa8ec74f4e2b0b45de244f1 Mon Sep 17 00:00:00 2001
Since we keep a document reference we no longer need the
separate global. It was only used in GetParentObject() and
we can return nullptr there (meaning 'use whatever magic global')
or return the document instead, which is what this patch does.
2013-07-18 15:01:48 -07:00
Ralph Giles
ac4de9f961 Bug 894104 - Part 1: Retain a document reference for GetCueAsHTML.
From 447985000db919e2f054012ca10f03d626c401b1 Mon Sep 17 00:00:00 2001
 r=bz
Save a copy of the owning document at creation time instead
of looking it up from mGlobal in each call. This ensures it
is available even if we're called after our window is unlinked
at teardown.

Make the constructors throw if a window is not available,
so we can return an INVALID_STATE_ERR if we're called
in an incomplete environment.
2013-07-16 22:14:40 -07:00
Ralph Giles
2ec3570c84 Bug 887463 - remove webvtt parser library. r=cpearce. 2013-07-09 17:02:00 -07:00
Rick Eyre
41e11bfdcb Bug 882714 - Implement TextTrackCue::DisplayState r=rillian
- Renamed mCueDiv to mDisplayState to match spec.
- Introduced mReset which is set whenever TextTrackCue changes in
a way that will mean we have to rebuild the computed cuetext of the
TextTrackCue.
2013-07-04 23:24:43 +02:00
Rick Eyre
3800bbb044 Bug 833382 - Implement WebVTTLoadListener r=cpearce,bz
- Implemented WebVTTLoadListener to manage the webvtt parser.
- TextTrackCue now handles the conversion of webvtt nodes to
  anonymous content which will be displayed on the video div
  overlay.
- HTMLTrackElement manages the lifetime of the WebVTTLoadListener.
2013-06-10 08:30:00 -07:00
Ralph Giles
3d52d52549 Bug 833385 - Implement HTMLTrackElement and TextTrack. r=bz,Ms2ger
Add webidl interfaces and implementations of the HTML <track>
element and related TextTrack, TextTrackList, TextTrackCue,
and TextTrackCueList dom objects.

Visibility is controlled by the media.webvtt.enabled pref,
which defaults to false.

HTMLMediaElement:NewURIFromString() is hoisted to
nsGenericHTMLElement so it's available to the track
element as well.

This patch is primarily work by Dale Karp, David Humphrey
and others as Seneca College.
2013-05-22 00:14:00 +08:00