Added graph-video option

[daniellobrunello]

Added the graphVideo option for uploading video material to the graph-video.facebook.com endpoint.
Example:

FB.options({videoGraph: true});
FB.api(123456789 + '/videos', 'post', {
    title: 'Test Video',
    published: true,
    access_token: 'ABCDEFGHIJKLMOP12345!?',
    file_url: 'https://vids.test.com/test.mp4'
});

[dantman]

Not sure I like the videoGraph option. This seems like something that would be better as part of the path arg to the api function. So you don't need two entirely separate SDK instances to make video and non-video API requests.

[daniellobrunello]

You don't need two separate instances, you just have to switch the option before making the API request. Its not the most elegant way, I have to admit.
So you would prefer a check for a complete TLD for the path arg?

[dantman]

You don't need two separate instances, you just have to switch the option before making the API request. Its not the most elegant way, I have to admit.

API requests are async. So that is just begging for race condition based bugs.

So you would prefer a check for a complete TLD for the path arg?

Yeah, I'd prefer some way of sending a request to the video api using the first arg to the method. Maybe a video: prefix, an object arg like { video: true, path: ... } or { api: 'video', path: ... }. Or perhaps we should go with FB.videoApi. A video option might be acceptable if it's a hidden prop where you essentially do FB.video.api(...) (FB.video would internally extend the instance and set the api to 'video').

Thinking about it FB.video.api sounds like it might be the nicest API.