Commit Graph

42 Commits

Author SHA1 Message Date
Rick Eyre
ce6c727cbe 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
a980ca643a Bug 950308 - TextTrack, TextTrackCue, TextTrackList should BindToOwner. r=smaug 2014-04-07 13:58:38 -04:00
Boris Zbarsky
35fca5eeeb 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
0c16d60594 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
54d409dfca 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
d8bc92185a Bug 985484 - Remove TextTrackCue::CueChange. r=cpearce 2014-03-19 11:01:38 -04:00
Rick Eyre
d192be9622 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
ce8424f247 Bug 967157 - Setting TextTrackCue::DisplayState should set TextTrackCue::HasBeenReset to false. r=rillian 2014-02-03 15:27:51 -05:00
Rick Eyre
e19074b0f0 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
74638c74c5 Bug 882299 - Implement VTTCue::Line r=smaug,rillian,bz 2014-01-15 08:04:00 -08:00
Rick Eyre
fcea6068e0 Bug 865407 - Part 6: Expose VTTCue::Reset to Chrome JS. r=bz 2013-12-12 13:13:38 -05:00
Rick Eyre
e7dca1a38d 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
67d71baa8f Bug 865407 - Part 4: Expose TextTrackCue::DisplayState to chrome JS. r=rillian, r=bz 2013-12-12 11:27:30 -05:00
Ryan VanderMeulen
9fff18dc96 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
6a851ca394 Bug 865407 - Part 6: Expose VTTCue::Reset to Chrome JS. r=bz 2013-12-12 13:13:38 -05:00
Rick Eyre
f984065401 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
d1d4dafa39 Bug 865407 - Part 4: Expose TextTrackCue::DisplayState to chrome JS. r=rillian, r=bz 2013-12-12 11:27:30 -05:00
Rick Eyre
b83b790cb9 Bug 949643 - Implement VTTCue::PositionAlign. r=rillian 2013-12-13 12:57:48 -05:00
Rick Eyre
e49f238506 Bug 950830 - Clean up TextTrackCue style and TODO comments. r=rillian 2013-12-16 14:49:22 -05:00
Rick Eyre
ff8e7b94f1 Bug 949642 - Implement VTTCue::LineAlign. r=rillian 2013-12-13 09:29:15 -05:00
Wes Kocher
7dadb7ffc4 Backed out changeset df0855f26e4d (bug 949642) for introducing a new intermittent failure 2013-12-20 18:33:55 -08:00
Wes Kocher
d578056d58 Backed out changeset 6532bf066bb4 (bug 882299) 2013-12-20 18:33:04 -08:00
Wes Kocher
901b762675 Backed out changeset 00aa14a1f461 (bug 950830) 2013-12-20 18:32:20 -08:00
Rick Eyre
fa0a0a1b17 Bug 950830 - Clean up TextTrackCue style and TODO comments. r=rillian 2013-12-16 14:49:22 -05:00
Rick Eyre
42f6a15b8f Bug 882299 - Implement VTTCue::Line. r=smaug, r=rillian 2013-12-16 14:35:01 -05:00
Rick Eyre
47bef1e163 Bug 949642 - Implement VTTCue::LineAlign. r=rillian 2013-12-13 09:29:15 -05:00
Rick Eyre
50aa5e5556 Bug 950049 - Rename VTTCue::TextTrackCueAlign to AlignSetting. r=rillian 2013-12-13 09:44:50 -05:00
Ehsan Akhgari
180798444f 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
fc4a14a980 Bug 917945 - Part 2: Add RegionId attribute to VTTCue. r=rillian 2013-09-18 13:29:20 -04:00
Rick Eyre
51719c804f 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
8dd8551b49 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
2276d0a676 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
73c2f74ef5 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
de0ee30e74 Bug 903772: Part 3 - Refactor GlobalObject. r=peterv 2013-08-22 22:17:08 -07:00
Rick Eyre
67da9a7c47 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
8b4f8baa85 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
87be796147 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
5dd627ae07 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
d187fe7323 Bug 887463 - remove webvtt parser library. r=cpearce. 2013-07-09 17:02:00 -07:00
Rick Eyre
ecb691da3b 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
73e8300de5 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
d98efa09c2 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