Update PRESUBMIT scripts to use PresubmitPromptOrNotify helper.

Update PRESUBMIT scripts to use PresubmitPromptOrNotify helper.

NOTRY=true

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=191863

[Wez, 2013-03-26 18:02:45]

PTAL

(FYI I got an error about missing base files _after_ the base files had uploaded
for this patch)

[M-A Ruel, 2013-03-26 18:05:00]

https://chromiumcodereview.appspot.com/13093006/diff/1/PRESUBMIT.py
File PRESUBMIT.py (right):

https://chromiumcodereview.appspot.com/13093006/diff/1/PRESUBMIT.py#newcode226
PRESUBMIT.py:226: # We don't warn on commit, to avoid stopping commits going
through CQ.
I think you can remove all the comments too, here and lines 440, 619, etc.

https://chromiumcodereview.appspot.com/13093006/diff/1/PRESUBMIT_test.py
File PRESUBMIT_test.py (right):

https://chromiumcodereview.appspot.com/13093006/diff/1/PRESUBMIT_test.py#newc...
PRESUBMIT_test.py:50: 
add another line.

[Wez, 2013-03-26 20:08:02]

https://chromiumcodereview.appspot.com/13093006/diff/1/PRESUBMIT.py
File PRESUBMIT.py (right):

https://chromiumcodereview.appspot.com/13093006/diff/1/PRESUBMIT.py#newcode226
PRESUBMIT.py:226: # We don't warn on commit, to avoid stopping commits going
through CQ.
On 2013/03/26 18:05:00, Marc-Antoine Ruel wrote:
> I think you can remove all the comments too, here and lines 440, 619, etc.

Done.

https://chromiumcodereview.appspot.com/13093006/diff/1/PRESUBMIT_test.py
File PRESUBMIT_test.py (right):

https://chromiumcodereview.appspot.com/13093006/diff/1/PRESUBMIT_test.py#newc...
PRESUBMIT_test.py:50: 
On 2013/03/26 18:05:00, Marc-Antoine Ruel wrote:
> add another line.

Done.

[Wez, 2013-04-01 20:34:12]

Le ping.

[M-A Ruel, 2013-04-02 15:58:21]

lgtm

[commit-bot: I haz the power, 2013-04-02 15:58:52]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/wez@chromium.org/13093006/5001

[commit-bot: I haz the power, 2013-04-02 15:58:58]

Presubmit check for 13093006-5001 failed and returned exit status 1.

INFO:root:Found 3 file(s).

testBadOrigFile (__main__.BadExtensionsTest) ... ok
testBadRejFile (__main__.BadExtensionsTest) ... ok
testGoodFiles (__main__.BadExtensionsTest) ... ok
testOnlyOwnersFiles (__main__.BadExtensionsTest) ... ok
testAlphabeticalOrderMismatch (__main__.IncludeOrderTest) ... ok
testCheckOnlyCFiles (__main__.IncludeOrderTest) ... ok
testConflictAdded1 (__main__.IncludeOrderTest) ... ok
testConflictAdded2 (__main__.IncludeOrderTest) ... ok
testIfElifElseEndif (__main__.IncludeOrderTest) ... ok
testOrderAlreadyWrong (__main__.IncludeOrderTest) ... ok
testSpecialFirstInclude1 (__main__.IncludeOrderTest) ... ok
testSpecialFirstInclude2 (__main__.IncludeOrderTest) ... ok
testSpecialFirstInclude3 (__main__.IncludeOrderTest) ... ok
testSpecialFirstInclude4 (__main__.IncludeOrderTest) ... ok
testSpecialFirstInclude5 (__main__.IncludeOrderTest) ... ok
testSysIncludes (__main__.IncludeOrderTest) ... ok
testSystemHeaderOrder (__main__.IncludeOrderTest) ... ok
testSystemHeaderOrderMismatch1 (__main__.IncludeOrderTest) ... ok
testSystemHeaderOrderMismatch2 (__main__.IncludeOrderTest) ... ok
testSystemHeaderOrderMismatch3 (__main__.IncludeOrderTest) ... ok
testUncheckableIncludes (__main__.IncludeOrderTest) ... ok
testInvalidOSMacroNames (__main__.InvalidOSMacroNamesTest) ... ok
testValidOSMacroNames (__main__.InvalidOSMacroNamesTest) ... ok
testTypicalConflict (__main__.VersionControlConflictsTest) ... ok

----------------------------------------------------------------------
Ran 24 tests in 0.006s

OK
Running presubmit commit checks ...
Running /b/commit-queue/workdir/chromium/PRESUBMIT.py
Running /b/commit-queue/workdir/chromium/PRESUBMIT_test.py
Running /b/commit-queue/workdir/chromium/ppapi/PRESUBMIT.py

** Presubmit Messages **
If this change has an associated bug, add BUG=[bug number].

** Presubmit ERRORS **
Missing LGTM from an OWNER for these files:
    ppapi/PRESUBMIT.py

[Wez, 2013-04-02 18:26:13]

dmichael: PTAL ppapi/PRESUBMIT.py

[dmichael (off chromium), 2013-04-02 18:27:49]

ppapi/* lgtm

[commit-bot: I haz the power, 2013-04-02 18:29:34]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/wez@chromium.org/13093006/5001

[commit-bot: I haz the power, 2013-04-02 18:31:44]

Change committed as 191863