Skip to content

createObjectURL() has been depreceated.#239

Merged
wesbos merged 2 commits intowesbos:masterfrom
prakhars17:patch-1
Sep 24, 2018
Merged

createObjectURL() has been depreceated.#239
wesbos merged 2 commits intowesbos:masterfrom
prakhars17:patch-1

Conversation

@prakhars17
Copy link
Copy Markdown
Contributor

Replaced createObjectURL() with newer syntax.

Well, if this PR is merged, it may be slightly different(only a single line) from the videos as createObjectURL() has been deprecated by major browsers. Just alerting @wesbos and others if not aware.

replaced createObjectURL() with newer syntax.
// DEPRECIATION :
// The following has been depreceated by major browsers.
// video.src = window.URL.createObjectURL(localMediaStream);
// Please refer to this:
Copy link
Copy Markdown
Collaborator

@palashmon palashmon Sep 16, 2018

Choose a reason for hiding this comment

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

Hi, thanks for your contribution. But I think you forgot to add the link in comments which one can refer to for further information about createObjectURL depreciation.

Regards!

@prakhars17
Copy link
Copy Markdown
Contributor Author

Thanks, @palashmon. Updated with the relevant links.

@palashmon
Copy link
Copy Markdown
Collaborator

Awesome!

@wesbos wesbos merged commit daa897a into wesbos:master Sep 24, 2018
@wesbos
Copy link
Copy Markdown
Owner

wesbos commented Sep 24, 2018

thank you - I like how you added the comment in so it doesn't kill the video :D

@prakhars17 prakhars17 deleted the patch-1 branch September 24, 2018 14:36
cbtsao47 pushed a commit to cbtsao47/JavaScript30 that referenced this pull request Nov 23, 2018
createObjectURL() has been depreceated.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants