From f5e900eedd1cc0b55fa8ffe7752a391084d3408a Mon Sep 17 00:00:00 2001 From: Ray Greenwell Date: Wed, 12 Nov 2008 05:26:58 +0000 Subject: [PATCH] - Broke out creating the video UI into a method. - Bank on the video UI having sane width/height. git-svn-id: svn+ssh://src.earth.threerings.net/nenya/trunk@697 ed5b42cb-e716-0410-a449-f6a68f950b19 --- src/as/com/threerings/flash/MediaContainer.as | 17 ++++++++++++----- .../flash/video/SimpleVideoDisplay.as | 10 ++++++++++ 2 files changed, 22 insertions(+), 5 deletions(-) diff --git a/src/as/com/threerings/flash/MediaContainer.as b/src/as/com/threerings/flash/MediaContainer.as index e9b6bde2..bb9e4fdb 100644 --- a/src/as/com/threerings/flash/MediaContainer.as +++ b/src/as/com/threerings/flash/MediaContainer.as @@ -64,6 +64,7 @@ import com.threerings.util.Util; import com.threerings.flash.video.FlvVideoPlayer; import com.threerings.flash.video.SimpleVideoDisplay; +import com.threerings.flash.video.VideoPlayer; import com.threerings.flash.video.VideoPlayerCodes; /** @@ -250,12 +251,18 @@ public class MediaContainer extends Sprite protected function setupVideo (url :String) :void { var player :FlvVideoPlayer = new FlvVideoPlayer(); - var vid :SimpleVideoDisplay = new SimpleVideoDisplay(player); -// vid.addEventListener(VideoPlayerCodes.SIZE, handleVideoSizeKnown); - _media = vid; - addChildAt(vid, 0); + _media = createVideoUI(player); + addChildAt(_media, 0); player.load(url); - updateContentDimensions(SimpleVideoDisplay.NATIVE_WIDTH, SimpleVideoDisplay.NATIVE_HEIGHT); + updateContentDimensions(_media.width, _media.height); + } + + /** + * Create the actual display for the VideoPlayer. + */ + protected function createVideoUI (player :VideoPlayer) :DisplayObject + { + return new SimpleVideoDisplay(player); } /** diff --git a/src/as/com/threerings/flash/video/SimpleVideoDisplay.as b/src/as/com/threerings/flash/video/SimpleVideoDisplay.as index 2f365c33..663513cd 100644 --- a/src/as/com/threerings/flash/video/SimpleVideoDisplay.as +++ b/src/as/com/threerings/flash/video/SimpleVideoDisplay.as @@ -103,6 +103,16 @@ public class SimpleVideoDisplay extends Sprite _knob.addEventListener(MouseEvent.MOUSE_DOWN, handleKnobDown); } + override public function get width () :Number + { + return NATIVE_WIDTH; + } + + override public function get height () :Number + { + return NATIVE_HEIGHT; + } + /** * Stop playing our video. */