CR: Add ResizeWindow for Helix dbus-server

+    enum VideoSizeMode
{VSZ_FULL_WINDOW,VSZ_IDEAL_SIZE,VSZ_DOUBLE_SIZE,VSZ_FIT_WIND
OW,VSZ_FIT_SCREEN,VSZ_FULL_SCREEN,VSZ_INVALID};

Not a big deal, but can you change that line so each enum is
on
a seperate line? It is easier to read and we want to keep
lines
under 80chars long if we can at all.

The rest looks good.

--greg.




Zhao, Halley wrote:
> Synopsis: 
> 
>     improve ResizeWindow() for Helix dbus-server. the
video will center
> aligned 
> 
>     inside the gdk window and provides 6 playback
modes:
> 
>         VSZ_FULL_WINDOW: video playback fullfill the
gdk window
> 
>         VSZ_FIT_WINDOW:  video playback keep aspect
(width:height) and
> fullfill the gdk window
> 
>         VSZ_IDEAL_SIZE:  video playback use the
original size
> 
>         VSZ_DOUBLE_SIZE: video playback use double
original size
> 
>         VSZ_FIT_SCREEN:  video playback keep aspect
(width:height) and
> fullfill the screen
> 
>         VSZ_FULL_SCREEN: video playback fullfill the
screen
> 
>     
> 
>     
> 
> Overview: 
> 
>     improve ResizeWindow() for Helix dbus-server. the
video will center
> aligned 
> 
>     in the gdk window and provides 6 playback modes:
> 
>         VSZ_FULL_WINDOW: video playback fullfill the
gdk window
> 
>         VSZ_FIT_WINDOW:  video playback keep aspect
(width:height) and
> fullfill the gdk window
> 
>         VSZ_IDEAL_SIZE:  video playback use the
original size
> 
>         VSZ_DOUBLE_SIZE: video playback use double
original size
> 
>         VSZ_FIT_SCREEN:  video playback keep aspect
(width:height) and
> fullfill the screen
> 
>         VSZ_FULL_SCREEN: video playback fullfill the
screen
> 
>         
> 
> Files Added:
> 
>        No file added
> 
> Files Modified:
> 
>     hxplayer.h:
(player/kit/dbus-server/pub/hxplayer.h)
> 
>         add a parameter of  playback mode for
hxplayer_resize_window
> 
>     helix-dbus-server.cpp:
> (player/kit/dbus-server/src/helix-dbus-server.cpp)
> 
>         add different playback mode support, add center
aligned
> 
>     hxplayer.cpp:
(player/kit/dbus-server/src/hxplayer.cpp)
> 
>         add different playback mode support
> 
>         
> 
> Image Size and Heap Use impact (Client -Only):
> 
>     little
> 
> Platforms and Profiles Affected:
> 
>     platform:   linux-2.2-libc6-gcc32-i586
> 
>     profile:    helix-client-all-defines
> 
>  
> 
> Distribution Libraries Affected:
> 
>     <helix-dbus-server.bin>
> 
> Distribution library impact and planned action:
> 
>     <None>
> 
> Platforms and Profiles Build Verified:
> 
>     Set BIF branch  -> hxdbus_3_1_0_atlas
> 
>     Set Target(s)   -> dbus_server_with_video
> 
>     Set Profile     -> helix-client-all-defines
> 
>     System ID       -> linux-2.2-libc6-gcc32-i586
> 
> Branch: 
> 
>     HEAD, hxclient_3_1_0_atlas
> 
> Copyright assignment: <MUST be one of the following
statements >
> 
>    2.      Intel has signed and delivered a Joint
Copyright Assignment
> 
>            to RealNetworks, and received acknowledgment
that the 
> 
>            agreement was received. 
> 
> Files Attached:
> 
>     helix-dbus-server-resize.diff
> 
>  
> 
> ZHAO, Halley (Aihua)
> 
> Email: halley.zhaointel.com <mailto:aihua.zhaointel.com> 
> 
> Tel: +86(21)61166476
> 
> iNet: 8821-6476
> 
> SSG/OTC/UMD
> 
>  
> 
> 
> 
>
------------------------------------------------------------
------------
> 
> This body part will be downloaded on demand.


_______________________________________________
Player-dev mailing list
Player-devhelixcommunity.org
http://lists.helixcommunity.org/mailman/listinfo/play
er-dev

　　Update the patch.
　　1> split Video Size Mode to separated lines
　　2> add pos_x, pos_y for UpdateSiteOrigin()
　　
　　
　　>-----Original Message-----
　　>From: Greg Wright [mailto:gwrightreal.com]
　　>Sent: 2007定12埖8晩 2:18
　　>To: Zhao, Halley
　　>Cc: helix-client-devhelixcommunity.org; Shen,
Cathy;
　　>player-devhelixcommunity.org
　　>Subject: Re: [Player-dev] CR: Add ResizeWindow for
Helix dbus-server
　　>
　　>+    enum VideoSizeMode
　　>{VSZ_FULL_WINDOW,VSZ_IDEAL_SIZE,VSZ_DOUBLE_SIZE,VSZ_
FIT_WINDOW,VSZ_FIT_
　　>SCREEN,VSZ_FULL_SCREEN,VSZ_INVALID};
　　>
　　>Not a big deal, but can you change that line so each
enum is on
　　>a seperate line? It is easier to read and we want to
keep lines
　　>under 80chars long if we can at all.
　　>
　　>The rest looks good.
　　>
　　>--greg.
　　>
　　>
　　>
　　>
　　>Zhao, Halley wrote:
　　>> Synopsis:
　　>>
　　>>     improve ResizeWindow() for Helix
dbus-server. the video will center
　　>> aligned
　　>>
　　>>     inside the gdk window and provides 6
playback modes:
　　>>
　　>>         VSZ_FULL_WINDOW: video playback
fullfill the gdk window
　　>>
　　>>         VSZ_FIT_WINDOW:  video playback keep
aspect (width:height) and
　　>> fullfill the gdk window
　　>>
　　>>         VSZ_IDEAL_SIZE:  video playback use the
original size
　　>>
　　>>         VSZ_DOUBLE_SIZE: video playback use
double original size
　　>>
　　>>         VSZ_FIT_SCREEN:  video playback keep
aspect (width:height) and
　　>> fullfill the screen
　　>>
　　>>         VSZ_FULL_SCREEN: video playback
fullfill the screen
　　>>
　　>>
　　>>
　　>>
　　>>
　　>> Overview:
　　>>
　　>>     improve ResizeWindow() for Helix
dbus-server. the video will center
　　>> aligned
　　>>
　　>>     in the gdk window and provides 6 playback
modes:
　　>>
　　>>         VSZ_FULL_WINDOW: video playback
fullfill the gdk window
　　>>
　　>>         VSZ_FIT_WINDOW:  video playback keep
aspect (width:height) and
　　>> fullfill the gdk window
　　>>
　　>>         VSZ_IDEAL_SIZE:  video playback use the
original size
　　>>
　　>>         VSZ_DOUBLE_SIZE: video playback use
double original size
　　>>
　　>>         VSZ_FIT_SCREEN:  video playback keep
aspect (width:height) and
　　>> fullfill the screen
　　>>
　　>>         VSZ_FULL_SCREEN: video playback
fullfill the screen
　　>>
　　>>
　　>>
　　>> Files Added:
　　>>
　　>>        No file added
　　>>
　　>> Files Modified:
　　>>
　　>>     hxplayer.h:
(player/kit/dbus-server/pub/hxplayer.h)
　　>>
　　>>         add a parameter of  playback mode for
hxplayer_resize_window
　　>>
　　>>     helix-dbus-server.cpp:
　　>>
(player/kit/dbus-server/src/helix-dbus-server.cpp)
　　>>
　　>>         add different playback mode support,
add center aligned
　　>>
　　>>     hxplayer.cpp:
(player/kit/dbus-server/src/hxplayer.cpp)
　　>>
　　>>         add different playback mode support
　　>>
　　>>
　　>>
　　>> Image Size and Heap Use impact (Client -Only):
　　>>
　　>>     little
　　>>
　　>> Platforms and Profiles Affected:
　　>>
　　>>     platform:   linux-2.2-libc6-gcc32-i586
　　>>
　　>>     profile:    helix-client-all-defines
　　>>
　　>>
　　>>
　　>> Distribution Libraries Affected:
　　>>
　　>>     <helix-dbus-server.bin>
　　>>
　　>> Distribution library impact and planned
action:
　　>>
　　>>     <None>
　　>>
　　>> Platforms and Profiles Build Verified:
　　>>
　　>>     Set BIF branch  -> hxdbus_3_1_0_atlas
　　>>
　　>>     Set Target(s)   ->
dbus_server_with_video
　　>>
　　>>     Set Profile     ->
helix-client-all-defines
　　>>
　　>>     System ID       ->
linux-2.2-libc6-gcc32-i586
　　>>
　　>> Branch:
　　>>
　　>>     HEAD, hxclient_3_1_0_atlas
　　>>
　　>> Copyright assignment: <MUST be one of the
following statements >
　　>>
　　>>    2.      Intel has signed and delivered a
Joint Copyright Assignment
　　>>
　　>>            to RealNetworks, and received
acknowledgment that the
　　>>
　　>>            agreement was received.
　　>>
　　>> Files Attached:
　　>>
　　>>     helix-dbus-server-resize.diff
　　>>
　　>>
　　>>
　　>> ZHAO, Halley (Aihua)
　　>>
　　>> Email: halley.zhaointel.com
<mailto:aihua.zhaointel.com>
　　>>
　　>> Tel: +86(21)61166476
　　>>
　　>> iNet: 8821-6476
　　>>
　　>> SSG/OTC/UMD
　　>>
　　>>
　　>>
　　>>
　　>>
　　>>
　　>----------------------------------------------------
-------------------
　　>-
　　>>
　　>> This body part will be downloaded on demand.

_______________________________________________
Player-dev mailing list
Player-devhelixcommunity.org
http://lists.helixcommunity.org/mailman/listinfo/play
er-dev

Zhao, Halley wrote:
> 　　Update the patch.
> 　　1> split Video Size Mode to separated lines

dbus changes look good.

--greg.


> 　　2> add pos_x, pos_y for UpdateSiteOrigin()

> 　　
> 　　
> 　　>-----Original Message-----
> 　　>From: Greg Wright [mailto:gwrightreal.com]
> 　　>Sent: 2007定12埖8晩 2:18
> 　　>To: Zhao, Halley
> 　　>Cc: helix-client-devhelixcommunity.org; Shen,
Cathy;
> 　　>player-devhelixcommunity.org
> 　　>Subject: Re: [Player-dev] CR: Add ResizeWindow
for Helix dbus-server
> 　　>
> 　　>+    enum VideoSizeMode
>
　　>{VSZ_FULL_WINDOW,VSZ_IDEAL_SIZE,VSZ_DOUBLE_SIZE,VSZ_
FIT_WINDOW,VSZ_FIT_
> 　　>SCREEN,VSZ_FULL_SCREEN,VSZ_INVALID};
> 　　>
> 　　>Not a big deal, but can you change that line so
each enum is on
> 　　>a seperate line? It is easier to read and we
want to keep lines
> 　　>under 80chars long if we can at all.
> 　　>
> 　　>The rest looks good.
> 　　>
> 　　>--greg.
> 　　>
> 　　>
> 　　>
> 　　>
> 　　>Zhao, Halley wrote:
> 　　>> Synopsis:
> 　　>>
> 　　>>     improve ResizeWindow() for Helix
dbus-server. the video will center
> 　　>> aligned
> 　　>>
> 　　>>     inside the gdk window and provides 6
playback modes:
> 　　>>
> 　　>>         VSZ_FULL_WINDOW: video playback
fullfill the gdk window
> 　　>>
> 　　>>         VSZ_FIT_WINDOW:  video playback
keep aspect (width:height) and
> 　　>> fullfill the gdk window
> 　　>>
> 　　>>         VSZ_IDEAL_SIZE:  video playback
use the original size
> 　　>>
> 　　>>         VSZ_DOUBLE_SIZE: video playback
use double original size
> 　　>>
> 　　>>         VSZ_FIT_SCREEN:  video playback
keep aspect (width:height) and
> 　　>> fullfill the screen
> 　　>>
> 　　>>         VSZ_FULL_SCREEN: video playback
fullfill the screen
> 　　>>
> 　　>>
> 　　>>
> 　　>>
> 　　>>
> 　　>> Overview:
> 　　>>
> 　　>>     improve ResizeWindow() for Helix
dbus-server. the video will center
> 　　>> aligned
> 　　>>
> 　　>>     in the gdk window and provides 6
playback modes:
> 　　>>
> 　　>>         VSZ_FULL_WINDOW: video playback
fullfill the gdk window
> 　　>>
> 　　>>         VSZ_FIT_WINDOW:  video playback
keep aspect (width:height) and
> 　　>> fullfill the gdk window
> 　　>>
> 　　>>         VSZ_IDEAL_SIZE:  video playback
use the original size
> 　　>>
> 　　>>         VSZ_DOUBLE_SIZE: video playback
use double original size
> 　　>>
> 　　>>         VSZ_FIT_SCREEN:  video playback
keep aspect (width:height) and
> 　　>> fullfill the screen
> 　　>>
> 　　>>         VSZ_FULL_SCREEN: video playback
fullfill the screen
> 　　>>
> 　　>>
> 　　>>
> 　　>> Files Added:
> 　　>>
> 　　>>        No file added
> 　　>>
> 　　>> Files Modified:
> 　　>>
> 　　>>     hxplayer.h:
(player/kit/dbus-server/pub/hxplayer.h)
> 　　>>
> 　　>>         add a parameter of  playback mode
for hxplayer_resize_window
> 　　>>
> 　　>>     helix-dbus-server.cpp:
> 　　>>
(player/kit/dbus-server/src/helix-dbus-server.cpp)
> 　　>>
> 　　>>         add different playback mode
support, add center aligned
> 　　>>
> 　　>>     hxplayer.cpp:
(player/kit/dbus-server/src/hxplayer.cpp)
> 　　>>
> 　　>>         add different playback mode
support
> 　　>>
> 　　>>
> 　　>>
> 　　>> Image Size and Heap Use impact (Client
-Only):
> 　　>>
> 　　>>     little
> 　　>>
> 　　>> Platforms and Profiles Affected:
> 　　>>
> 　　>>     platform:  
linux-2.2-libc6-gcc32-i586
> 　　>>
> 　　>>     profile:    helix-client-all-defines
> 　　>>
> 　　>>
> 　　>>
> 　　>> Distribution Libraries Affected:
> 　　>>
> 　　>>     <helix-dbus-server.bin>
> 　　>>
> 　　>> Distribution library impact and planned
action:
> 　　>>
> 　　>>     <None>
> 　　>>
> 　　>> Platforms and Profiles Build Verified:
> 　　>>
> 　　>>     Set BIF branch  ->
hxdbus_3_1_0_atlas
> 　　>>
> 　　>>     Set Target(s)   ->
dbus_server_with_video
> 　　>>
> 　　>>     Set Profile     ->
helix-client-all-defines
> 　　>>
> 　　>>     System ID       ->
linux-2.2-libc6-gcc32-i586
> 　　>>
> 　　>> Branch:
> 　　>>
> 　　>>     HEAD, hxclient_3_1_0_atlas
> 　　>>
> 　　>> Copyright assignment: <MUST be one of
the following statements >
> 　　>>
> 　　>>    2.      Intel has signed and delivered
a Joint Copyright Assignment
> 　　>>
> 　　>>            to RealNetworks, and received
acknowledgment that the
> 　　>>
> 　　>>            agreement was received.
> 　　>>
> 　　>> Files Attached:
> 　　>>
> 　　>>     helix-dbus-server-resize.diff
> 　　>>
> 　　>>
> 　　>>
> 　　>> ZHAO, Halley (Aihua)
> 　　>>
> 　　>> Email: halley.zhaointel.com
<mailto:aihua.zhaointel.com>
> 　　>>
> 　　>> Tel: +86(21)61166476
> 　　>>
> 　　>> iNet: 8821-6476
> 　　>>
> 　　>> SSG/OTC/UMD
> 　　>>
> 　　>>
> 　　>>
> 　　>>
> 　　>>
> 　　>>
>
　　>----------------------------------------------------
-------------------
> 　　>-
> 　　>>
> 　　>> This body part will be downloaded on
demand.


_______________________________________________
Player-dev mailing list
Player-devhelixcommunity.org
http://lists.helixcommunity.org/mailman/listinfo/play
er-dev

　　Commit to Atlas310 & HEAD
　　BR.
　　
　　
　　>-----Original Message-----
　　>From: Greg Wright [mailto:gwrightreal.com]
　　>Sent: 2007定12埖11晩 3:44
　　>To: Zhao, Halley
　　>Cc: helix-client-devhelixcommunity.org; Shen,
Cathy;
　　>player-devhelixcommunity.org
　　>Subject: Re: [Player-dev] CR: Add ResizeWindow for
Helix dbus-server
　　>
　　>Zhao, Halley wrote:
　　>> 　　Update the patch.
　　>> 　　1> split Video Size Mode to separated
lines
　　>
　　>dbus changes look good.
　　>
　　>--greg.
　　>
　　>
　　>> 　　2> add pos_x, pos_y for
UpdateSiteOrigin()
　　>
　　>>
　　>>
　　>> 　　>-----Original Message-----
　　>> 　　>From: Greg Wright [mailto:gwrightreal.com]
　　>> 　　>Sent: 2007定12埖8晩 2:18
　　>> 　　>To: Zhao, Halley
　　>> 　　>Cc: helix-client-devhelixcommunity.org; Shen, Cathy;
　　>> 　　>player-devhelixcommunity.org
　　>> 　　>Subject: Re: [Player-dev] CR: Add
ResizeWindow for Helix dbus-server
　　>> 　　>
　　>> 　　>+    enum VideoSizeMode
　　>>
　　>>{VSZ_FULL_WINDOW,VSZ_IDEAL_SIZE,VSZ_DOUBLE_SIZE,
VSZ_FIT_WINDOW,VSZ_FIT
　　>_
　　>> 　　>SCREEN,VSZ_FULL_SCREEN,VSZ_INVALID};
　　>> 　　>
　　>> 　　>Not a big deal, but can you change that
line so each enum is on
　　>> 　　>a seperate line? It is easier to read
and we want to keep lines
　　>> 　　>under 80chars long if we can at all.
　　>> 　　>
　　>> 　　>The rest looks good.
　　>> 　　>
　　>> 　　>--greg.
　　>> 　　>
　　>> 　　>
　　>> 　　>
　　>> 　　>
　　>> 　　>Zhao, Halley wrote:
　　>> 　　>> Synopsis:
　　>> 　　>>
　　>> 　　>>     improve ResizeWindow() for
Helix dbus-server. the video will
　　>center
　　>> 　　>> aligned
　　>> 　　>>
　　>> 　　>>     inside the gdk window and
provides 6 playback modes:
　　>> 　　>>
　　>> 　　>>         VSZ_FULL_WINDOW: video
playback fullfill the gdk window
　　>> 　　>>
　　>> 　　>>         VSZ_FIT_WINDOW:  video
playback keep aspect
　　>(width:height) and
　　>> 　　>> fullfill the gdk window
　　>> 　　>>
　　>> 　　>>         VSZ_IDEAL_SIZE:  video
playback use the original size
　　>> 　　>>
　　>> 　　>>         VSZ_DOUBLE_SIZE: video
playback use double original size
　　>> 　　>>
　　>> 　　>>         VSZ_FIT_SCREEN:  video
playback keep aspect
　　>(width:height) and
　　>> 　　>> fullfill the screen
　　>> 　　>>
　　>> 　　>>         VSZ_FULL_SCREEN: video
playback fullfill the screen
　　>> 　　>>
　　>> 　　>>
　　>> 　　>>
　　>> 　　>>
　　>> 　　>>
　　>> 　　>> Overview:
　　>> 　　>>
　　>> 　　>>     improve ResizeWindow() for
Helix dbus-server. the video will
　　>center
　　>> 　　>> aligned
　　>> 　　>>
　　>> 　　>>     in the gdk window and provides
6 playback modes:
　　>> 　　>>
　　>> 　　>>         VSZ_FULL_WINDOW: video
playback fullfill the gdk window
　　>> 　　>>
　　>> 　　>>         VSZ_FIT_WINDOW:  video
playback keep aspect
　　>(width:height) and
　　>> 　　>> fullfill the gdk window
　　>> 　　>>
　　>> 　　>>         VSZ_IDEAL_SIZE:  video
playback use the original size
　　>> 　　>>
　　>> 　　>>         VSZ_DOUBLE_SIZE: video
playback use double original size
　　>> 　　>>
　　>> 　　>>         VSZ_FIT_SCREEN:  video
playback keep aspect
　　>(width:height) and
　　>> 　　>> fullfill the screen
　　>> 　　>>
　　>> 　　>>         VSZ_FULL_SCREEN: video
playback fullfill the screen
　　>> 　　>>
　　>> 　　>>
　　>> 　　>>
　　>> 　　>> Files Added:
　　>> 　　>>
　　>> 　　>>        No file added
　　>> 　　>>
　　>> 　　>> Files Modified:
　　>> 　　>>
　　>> 　　>>     hxplayer.h:
(player/kit/dbus-server/pub/hxplayer.h)
　　>> 　　>>
　　>> 　　>>         add a parameter of 
playback mode for
　　>hxplayer_resize_window
　　>> 　　>>
　　>> 　　>>     helix-dbus-server.cpp:
　　>> 　　>>
(player/kit/dbus-server/src/helix-dbus-server.cpp)
　　>> 　　>>
　　>> 　　>>         add different playback
mode support, add center aligned
　　>> 　　>>
　　>> 　　>>     hxplayer.cpp:
(player/kit/dbus-server/src/hxplayer.cpp)
　　>> 　　>>
　　>> 　　>>         add different playback
mode support
　　>> 　　>>
　　>> 　　>>
　　>> 　　>>
　　>> 　　>> Image Size and Heap Use impact
(Client -Only):
　　>> 　　>>
　　>> 　　>>     little
　　>> 　　>>
　　>> 　　>> Platforms and Profiles Affected:
　　>> 　　>>
　　>> 　　>>     platform:  
linux-2.2-libc6-gcc32-i586
　　>> 　　>>
　　>> 　　>>     profile:   
helix-client-all-defines
　　>> 　　>>
　　>> 　　>>
　　>> 　　>>
　　>> 　　>> Distribution Libraries Affected:
　　>> 　　>>
　　>> 　　>>     <helix-dbus-server.bin>
　　>> 　　>>
　　>> 　　>> Distribution library impact and
planned action:
　　>> 　　>>
　　>> 　　>>     <None>
　　>> 　　>>
　　>> 　　>> Platforms and Profiles Build
Verified:
　　>> 　　>>
　　>> 　　>>     Set BIF branch  ->
hxdbus_3_1_0_atlas
　　>> 　　>>
　　>> 　　>>     Set Target(s)   ->
dbus_server_with_video
　　>> 　　>>
　　>> 　　>>     Set Profile     ->
helix-client-all-defines
　　>> 　　>>
　　>> 　　>>     System ID       ->
linux-2.2-libc6-gcc32-i586
　　>> 　　>>
　　>> 　　>> Branch:
　　>> 　　>>
　　>> 　　>>     HEAD, hxclient_3_1_0_atlas
　　>> 　　>>
　　>> 　　>> Copyright assignment: <MUST be
one of the following statements >
　　>> 　　>>
　　>> 　　>>    2.      Intel has signed and
delivered a Joint Copyright
　　>Assignment
　　>> 　　>>
　　>> 　　>>            to RealNetworks, and
received acknowledgment that the
　　>> 　　>>
　　>> 　　>>            agreement was
received.
　　>> 　　>>
　　>> 　　>> Files Attached:
　　>> 　　>>
　　>> 　　>>     helix-dbus-server-resize.diff
　　>> 　　>>
　　>> 　　>>
　　>> 　　>>
　　>> 　　>> ZHAO, Halley (Aihua)
　　>> 　　>>
　　>> 　　>> Email: halley.zhaointel.com
<mailto:aihua.zhaointel.com>
　　>> 　　>>
　　>> 　　>> Tel: +86(21)61166476
　　>> 　　>>
　　>> 　　>> iNet: 8821-6476
　　>> 　　>>
　　>> 　　>> SSG/OTC/UMD
　　>> 　　>>
　　>> 　　>>
　　>> 　　>>
　　>> 　　>>
　　>> 　　>>
　　>> 　　>>
　　>>
　　>>------------------------------------------------
----------------------
　　>-
　　>> 　　>-
　　>> 　　>>
　　>> 　　>> This body part will be downloaded
on demand.

_______________________________________________
Player-dev mailing list
Player-devhelixcommunity.org
http://lists.helixcommunity.org/mailman/listinfo/play
er-dev