Skip to content
This repository was archived by the owner on Feb 22, 2023. It is now read-only.

[video_player] Fix player initialization and other warnings #1552

Merged
merged 2 commits into from
Apr 29, 2019
Merged

[video_player] Fix player initialization and other warnings #1552

merged 2 commits into from
Apr 29, 2019

Conversation

iskakaushik
Copy link
Contributor

  1. When a player has not initialized it's presentationSize is
    returned as CGSizeZero.

  2. The enum type must be AVPlayerItemStatusFailed not AVPlayerStatusFailed

  3. Fix a few other IDE warnings.

Related Issues

Fixes flutter/flutter#19197

1. When a player has not initialized it's presentationSize is
   returned as CGSizeZero.

2. The enum type must be AVPlayerItemStatusFailed not AVPlayerStatusFailed

3. Fix a few other IDE warnings.
Copy link
Contributor

@cyanglaz cyanglaz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@iskakaushik iskakaushik merged commit 29d3a5b into flutter:master Apr 29, 2019
WoodyGuo pushed a commit to liuwei1130/plugins that referenced this pull request Jun 27, 2019
…1552)

* [video_player] Fix player initialization and other warnings

1. When a player has not initialized it's presentationSize is
   returned as CGSizeZero.

2. The enum type must be AVPlayerItemStatusFailed not AVPlayerStatusFailed

3. Fix a few other IDE warnings.

* update pubspec and changelog
@quentinleguennec
Copy link
Contributor

Hi @iskakaushik @cyanglaz ,
This part of the PR is breaking our app:

// The player has not yet initialized.
    if (height == CGSizeZero.height && width == CGSizeZero.width) {
      return;
    }

We are using the video_player to play mp3 files (since there is no well maintained audio plugin). It was working perfectly but since the file have a width and height of 0 this if breaks the plugin for us on iOS. Could we remove this limitation?

@khietlam
Copy link

I'm using this video_player to play mp3 files as @quentinleguennec . Now it breaks my app.

Please help with this issue. Thanks.

julianscheel pushed a commit to jusst-engineering/plugins that referenced this pull request Mar 11, 2020
* Update README to mention web support
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

video_player plugin: 0.6.4 - Future already completed
5 participants