Add presubmit checks and watchlist for FFmpeg repo.

Add presubmit checks and watchlist for FFmpeg repo.

Presubmit does the following:
  - Ensures users don't commit to the Subversion repository.
  - Reminds users to update the README when they make changes.
  - Reminds users that the Windows DLLs, Subversion repository, and DEPS
    files need manual updating.

BUG=110775
TEST=Checked this commit.

[Ami GONE FROM CHROMIUM, 2012-02-22 01:22:00]

https://chromiumcodereview.appspot.com/9432013/diff/3001/PRESUBMIT.py
File PRESUBMIT.py (right):

https://chromiumcodereview.appspot.com/9432013/diff/3001/PRESUBMIT.py#newcode40
PRESUBMIT.py:40: 'Commits to the FFmpeg repository must be made through Git. The
easiest',
Get scherkus@' approval of this; his workflow is byzantine and he might have
objections to this.

https://chromiumcodereview.appspot.com/9432013/diff/3001/PRESUBMIT.py#newcode49
PRESUBMIT.py:49: print Color(BOLD_RED, '[ WARNING ]'.center(70, '*'))
Snazzy!

https://chromiumcodereview.appspot.com/9432013/diff/3001/PRESUBMIT.py#newcode49
PRESUBMIT.py:49: print Color(BOLD_RED, '[ WARNING ]'.center(70, '*'))
I don't like scripts that always WARN; it has a desensitizing effect.
s/WARNING/REMEMBER/

https://chromiumcodereview.appspot.com/9432013/diff/3001/PRESUBMIT.py#newcode57
PRESUBMIT.py:57: 'Please see README.chromium or contact videostack-eng for
help.']))
videostack-eng is a google-internal list, but this PRESUBMIT is going into a
public repo.

https://chromiumcodereview.appspot.com/9432013/diff/3001/PRESUBMIT.py#newcode68
PRESUBMIT.py:68: return [output_api.PresubmitPromptWarning('\n'.join([
Fine as long as this doesn't block CQ submission.

https://chromiumcodereview.appspot.com/9432013/diff/3001/codereview.settings
File codereview.settings (right):

https://chromiumcodereview.appspot.com/9432013/diff/3001/codereview.settings#...
codereview.settings:3: CC_LIST: chromium-reviews@chromium.org,
acolwell+watch@chromium.org, annacc+watch@chromium.org,
dalecurtis+watch@chromium.org, ddorwin+watch@chromium.org,
fischman+watch@chromium.org, hclam+watch@chromium.org, ihf+watch@chromium.org,
scherkus@chromium.org, vrk@chromium.org
Ugh.  How about this that I just created: feature-media-reviews@chromium.org ?

[DaleCurtis, 2012-02-22 20:15:13]

https://chromiumcodereview.appspot.com/9432013/diff/3001/PRESUBMIT.py
File PRESUBMIT.py (right):

https://chromiumcodereview.appspot.com/9432013/diff/3001/PRESUBMIT.py#newcode40
PRESUBMIT.py:40: 'Commits to the FFmpeg repository must be made through Git. The
easiest',
On 2012/02/22 01:22:00, Ami Fischman wrote:
> Get scherkus@' approval of this; his workflow is byzantine and he might have
> objections to this.

+scherkus.

https://chromiumcodereview.appspot.com/9432013/diff/3001/PRESUBMIT.py#newcode49
PRESUBMIT.py:49: print Color(BOLD_RED, '[ WARNING ]'.center(70, '*'))
On 2012/02/22 01:22:00, Ami Fischman wrote:
> I don't like scripts that always WARN; it has a desensitizing effect.
> s/WARNING/REMEMBER/

Done.

https://chromiumcodereview.appspot.com/9432013/diff/3001/PRESUBMIT.py#newcode57
PRESUBMIT.py:57: 'Please see README.chromium or contact videostack-eng for
help.']))
On 2012/02/22 01:22:00, Ami Fischman wrote:
> videostack-eng is a google-internal list, but this PRESUBMIT is going into a
> public repo.

Done.

https://chromiumcodereview.appspot.com/9432013/diff/3001/PRESUBMIT.py#newcode68
PRESUBMIT.py:68: return [output_api.PresubmitPromptWarning('\n'.join([
On 2012/02/22 01:22:00, Ami Fischman wrote:
> Fine as long as this doesn't block CQ submission.

These are OnUpload checks, so as far as I understand it they don't affect
OnCommit. I just confirmed this with maruel.

https://chromiumcodereview.appspot.com/9432013/diff/3001/codereview.settings
File codereview.settings (right):

https://chromiumcodereview.appspot.com/9432013/diff/3001/codereview.settings#...
codereview.settings:3: CC_LIST: chromium-reviews@chromium.org,
acolwell+watch@chromium.org, annacc+watch@chromium.org,
dalecurtis+watch@chromium.org, ddorwin+watch@chromium.org,
fischman+watch@chromium.org, hclam+watch@chromium.org, ihf+watch@chromium.org,
scherkus@chromium.org, vrk@chromium.org
On 2012/02/22 01:22:00, Ami Fischman wrote:
> Ugh.  How about this that I just created:
mailto:feature-media-reviews@chromium.org ?

Done.

[Ami GONE FROM CHROMIUM, 2012-02-22 20:57:26]

LGTM

https://chromiumcodereview.appspot.com/9432013/diff/6/codereview.settings
File codereview.settings (right):

https://chromiumcodereview.appspot.com/9432013/diff/6/codereview.settings#new...
codereview.settings:3: CC_LIST: chromium-reviews@chromium.org,
feature-media-reviews@chromium.org
I wonder if (a new) WATCHLISTS is a better place for f-m-r@c.o.  Either way.

[scherkus (not reviewing), 2012-02-23 21:46:56]

one comment but lgtm

https://chromiumcodereview.appspot.com/9432013/diff/6/codereview.settings
File codereview.settings (right):

https://chromiumcodereview.appspot.com/9432013/diff/6/codereview.settings#new...
codereview.settings:4: STATUS: http://chromium-status.appspot.com/status
AFAIK this will prevent us from committing if the main tree is closed -- may
consider removing in the future

[DaleCurtis, 2012-02-23 21:49:39]

https://chromiumcodereview.appspot.com/9432013/diff/6/codereview.settings
File codereview.settings (right):

https://chromiumcodereview.appspot.com/9432013/diff/6/codereview.settings#new...
codereview.settings:4: STATUS: http://chromium-status.appspot.com/status
On 2012/02/23 21:46:56, scherkus wrote:
> AFAIK this will prevent us from committing if the main tree is closed -- may
> consider removing in the future

Done.