totem does not prevent the screen to be blanked

I've looked at the code in src/plugins/totem-screensaver.c

static void totem_screensaver_update_from_state (TotemObject *totem,
                     TotemScreensaverPlugin *pi)
{
    if (totem_is_playing (totem) != FALSE
        && totem_is_fullscreen (totem) != FALSE) {
        totem_scrsaver_disable (pi->scr);
    } else {
        totem_scrsaver_enable (pi->scr);
    }
}

I really think that this condition in inadequate: why the check that totem is fullscreen? I mean, I can prefer watching the video in windowed mode, especially if the resolution is not hi...

the check should be if totem_is_playing && totem_is_playing_video_file. Remember that there is the problem of inhibit the suspend, which should be done also if totem is playing an audio file (don't know if this is done elsewhere).
the function totem_is_playing_video_file should be implemented

Thank you for your bug. That's known upstream http://bugzilla.gnome.org/show_bug.cgi?id=425473

Sorry. but can we apply the patch removing "&& totem_is_fullscreen" by ourselves? Upstream doesn't seem interested in this bug, and it is a simple fix for us!

I've sent a patch upstream, commited it to svn and uploaded a package update with it now

totem (2.19.90-0ubuntu3) gutsy; urgency=low

  * debian/patches/07_no_screensaver_activation_when_playing.dpatch:
    - unactivate the screensaver also when not playing in fullscreen
      (LP: #123124)

 -- Sebastien Bacher <email address hidden> Wed, 22 Aug 2007 12:37:55 +0200

Sebastien, in this way the screensaver is also disactivated when
playing music. Is there a way to check if a video is playing, instead
of music, to better handle this problem?

you might have graphical effects when playing music, that's only code and that's likely possible but we don't have the ressources to start coding on minor things like that at the moment and that's a different issue, you can try to open a new bug upstream about it though