rg3/youtube-dl — pull request #422 by kevinamadeus (2012-09-25 08:33:39+00:00)
Title: Add InfoExtractor for Google Plus video (updated)

**I closed the old pull request https://github.com/rg3/youtube-dl/pull/416 and open this one again, because I am such a lousy git user that my commits were started to get very messy. I deleted my old fork so I couldn't rebase or whatever. In the end I forked and commit again hoping that this will not make the main repo history ugly.**

This is an InfoExtractor for Google Plus video. Please note this is targeted for posts which users uploaded the video directly to G+, but NOT posting Youtube links onto a G+ entry.

It is important for the user to know how to use this extractor.
They should paste the absolute link of that post containing the video link this:
```
youtube-dl https://plus.google.com/[userid]/posts/[postid]
```
The link can be obtained by copying the link of the timestamp of a post.
The video box and the video itself popups by javascript. There is no easy way to catch the URL for them.
So, the InfoExtractor does all the actions. Just paste the URL of the post itself.

**rbrito commented:**
>It looks good, apart from some small things:

>* While you are removing spurious trailing whitespace (and this is a great thing), this could gratuitously make other patches fail to apply. It is also a good practice to separate functional from cosmetic issues and a sequence of commits would be welcome here.

>* It would be nice if the information extrator used the same naming convention as the other extractors. In particular, it should end with IE. Also you misspelled Google as Goole.

>I have not tried to review the whole patch, but those points stood out.

I have addressed to rbrito's comments in this commit.