Skip to content
This repository was archived by the owner on Apr 3, 2020. It is now read-only.

[Android] Support third part media player on Crosswalk#3736

Open
fujunwei wants to merge 1 commit intocrosswalk-project:masterfrom
fujunwei:new_exo_media_player
Open

[Android] Support third part media player on Crosswalk#3736
fujunwei wants to merge 1 commit intocrosswalk-project:masterfrom
fujunwei:new_exo_media_player

Conversation

@fujunwei
Copy link
Copy Markdown
Contributor

@fujunwei fujunwei commented Jun 3, 2016

A requirement from an important customer who want to forward web resources with proxy
on Crosswalk, but android system MediaPlayer can't set a proxy with a standard API.
The ExoMediaPlayer is playing videos and music is a popular activity on Android devices,
and it can be configured with proxy.
https://developer.android.com/guide/topics/media/exoplayer.html

BUG=XWALK-6770

@crosswalk-trybot
Copy link
Copy Markdown

crosswalk-trybot commented Jun 3, 2016

Testing patch series with fujunwei/crosswalk@024bae0 as its head.

Bot Status
Crosswalk Android-X86 [FAILED 💔](https://build.crosswalk-project.org/try/builders/Crosswalk Android-X86/builds/4531)
Crosswalk Android x86-64 [FAILED 💔](https://build.crosswalk-project.org/try/builders/Crosswalk Android x86-64/builds/1579)
Crosswalk Linux [SUCCESS 💚](https://build.crosswalk-project.org/try/builders/Crosswalk Linux/builds/4521)

}

@XWalkAPI(reservable = true)
public void setExMediaPlayer(XWalkExMediaPlayerInternal mediaPlayer) {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Add the comment for this API?

@fujunwei
Copy link
Copy Markdown
Contributor Author

fujunwei commented Jun 3, 2016

Thanks @wuhengzhi Those proposals will be modified one by one.

@@ -9,11 +9,13 @@
import android.media.MediaPlayer;
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Remove it?

@sunlin-link
Copy link
Copy Markdown
Contributor

One thing, I'm still confused by the name ExMediaPlayer. Can we use a more intuitive name instead of some abbreviations?

@fujunwei fujunwei force-pushed the new_exo_media_player branch from 024bae0 to 9baa6c1 Compare June 27, 2016 05:39
@crosswalk-trybot
Copy link
Copy Markdown

crosswalk-trybot commented Jun 27, 2016

Testing patch series with fujunwei/crosswalk@9baa6c1 as its head.

Bot Status
Crosswalk Android-X86 [FAILED 💔](https://build.crosswalk-project.org/try/builders/Crosswalk Android-X86/builds/4564)
Crosswalk Android x86-64 [FAILED 💔](https://build.crosswalk-project.org/try/builders/Crosswalk Android x86-64/builds/1612)
Crosswalk Linux [SUCCESS 💚](https://build.crosswalk-project.org/try/builders/Crosswalk Linux/builds/4554)

@fujunwei
Copy link
Copy Markdown
Contributor Author

Changed ExMediaPlayer to ExternalMediaPlayer that is used in native code. Developer will use XWalkMediaPlayer to implement third party Media Player.
@lincsoon @xuxin @wuhengzhi PTAL.

}

/**
* Sets the data source (FileDescriptor) to use.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Lack of first parameter

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@param fd the FileDescriptor for the file you want to play.

@fujunwei fujunwei force-pushed the new_exo_media_player branch from 9baa6c1 to 18f4196 Compare June 28, 2016 07:29
@crosswalk-trybot
Copy link
Copy Markdown

crosswalk-trybot commented Jun 28, 2016

Testing patch series with fujunwei/crosswalk@18f4196 as its head.

Bot Status
Crosswalk Android-X86 [FAILED 💔](https://build.crosswalk-project.org/try/builders/Crosswalk Android-X86/builds/4568)
Crosswalk Android x86-64 [FAILED 💔](https://build.crosswalk-project.org/try/builders/Crosswalk Android x86-64/builds/1616)
Crosswalk Linux [SUCCESS 💚](https://build.crosswalk-project.org/try/builders/Crosswalk Linux/builds/4558)


/**
* Sets the volume on this player.
* @since 7.0
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Missing the explanation for parameters.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants