Remove url requirement and checking about flutter_playout

[artths]

Also this check should be removed from didUpdateWidget

    if (widget.url == null || widget.url.isEmpty) {
      _disposePlatformView();
    }

Because it will crash player when I just want to pause it:

      _videoPlayer = VideoPlayout(
        desiredState: PlayerState.STOPPED,
      );

Want you do some PlayerController so we could control the video state without needing of recreate widget?

I will do PR when find all the errors.

[KhuramKhalid]

If we change the _onMediaChanged() method in lib/video.dart to following then this issue could be fixed.

void _onMediaChanged() {
    if (widget.url != null) {
      if (_methodChannel == null) {
        _setupPlayer();
      } else {
        _methodChannel.invokeMethod("onMediaChanged", {
          "autoPlay": widget.autoPlay,
          "url": widget.url,
          "title": widget.title,
          "subtitle": widget.subtitle,
          "isLiveStream": widget.isLiveStream,
        });
      }
    }
  }

I tested this by first passing null as URL which resulted in a plan Container() and then when I pass an actual URL and rebuild, it creates the player. Let me know your thoughts on this.

[artths]

Yes, it works if passing null, but will enter useless state if we supply some broken link to it:

  Video _videoPlayer = Video(
    url: "http://some_site.com/broken_link.mp4",
  );

Native view will not be created because of the isPlayable check but native channel will.

[KhuramKhalid]

That's true. The change you mentioned in #10 will fix this. Moving init code out of view() in native iOS. Then media change can call init again if player was never initialised. Would it be possible for you to create a PR for this? Otherwise I'll try working on this later.

[artths]

Sorry, was working on another project.
I created PR with the changes you propose in _onMediaChanged() and also in native part, because it crashes if I supply null url since last commit.

[KhuramKhalid]

Thanks for the PR. I've pushed a new version out which includes this. See v1.0.35