Revert of Convert ppapi_unittests to run exclusively on Swarming

Revert of Convert ppapi_unittests to run exclusively on Swarming (patchset #1 id:1 of https://codereview.chromium.org/1224453003/)

Reason for revert:
Results in multiple data races reported on ppapi_unittests by ThreadSanitizer

Original issue's description:
> Convert ppapi_unittests to run exclusively on Swarming
>
> 0 configs already ran on Swarming
> 46 used to run locally and were converted:
> - chromium.chromiumos.json: Linux ChromiumOS Ozone Tests (1)
> - chromium.chromiumos.json: Linux ChromiumOS Tests (1)
> - chromium.chromiumos.json: Linux ChromiumOS Tests (dbg)(1)
> - chromium.fyi.json: ClangToTLinux tester
> - chromium.fyi.json: ClangToTLinuxASan tester
> - chromium.fyi.json: ClangToTMac tester
> - chromium.fyi.json: ClangToTMacASan tester
> - chromium.fyi.json: ClangToTWin tester
> - chromium.fyi.json: ClangToTWin(dbg) tester
> - chromium.fyi.json: ClangToTWin(dll) tester
> - chromium.fyi.json: ClangToTWin64 tester
> - chromium.fyi.json: ClangToTWin64(dbg) tester
> - chromium.fyi.json: ClangToTWin64(dll) tester
> - chromium.fyi.json: CrWinAsan tester
> - chromium.fyi.json: CrWinAsan(dll) tester
> - chromium.fyi.json: CrWinClang tester
> - chromium.fyi.json: CrWinClang(dbg) tester
> - chromium.fyi.json: CrWinClang(shared) tester
> - chromium.fyi.json: CrWinClang64 tester
> - chromium.fyi.json: CrWinClang64(dbg) tester
> - chromium.fyi.json: CrWinClang64(dll) tester
> - chromium.fyi.json: CrWinClangLLD tester
> - chromium.fyi.json: Linux Trusty
> - chromium.fyi.json: Linux Trusty (dbg)
> - chromium.linux.json: Cast Linux
> - chromium.linux.json: Linux Tests
> - chromium.linux.json: Linux Tests (dbg)(1)
> - chromium.linux.json: Linux Tests (dbg)(1)(32)
> - chromium.mac.json: Mac10.10 Tests
> - chromium.mac.json: Mac10.6 Tests
> - chromium.mac.json: Mac10.8 Tests
> - chromium.mac.json: Mac10.9 Tests
> - chromium.mac.json: Mac10.9 Tests (dbg)
> - chromium.memory.fyi.json: Linux ChromeOS MSan Tests
> - chromium.memory.fyi.json: Linux MSan Tests
> - chromium.memory.fyi.json: Linux TSan Tests
> - chromium.memory.json: Linux ASan LSan Tests (1)
> - chromium.memory.json: Linux Chromium OS ASan LSan Tests (1)
> - chromium.memory.json: Mac ASan 64 Tests (1)
> - chromium.webkit.json: Linux ChromiumOS Tests (1)
> - chromium.webkit.json: Linux ChromiumOS Tests (dbg)(1)
> - chromium.win.json: Vista Tests (1)
> - chromium.win.json: Win 7 Tests x64 (1)
> - chromium.win.json: Win7 Tests (1)
> - chromium.win.json: Win7 Tests (dbg)(1)
> - chromium.win.json: XP Tests (1)
>
> Ran:
>   ./manage.py --convert ppapi_unittests
>
> BUG=98637
>
> Committed: https://crrev.com/0e660cb50e66dc9d5e794d63b4bd72079acefc2e
> Cr-Commit-Position: refs/heads/master@{#337272}

TBR=raymes@chromium.org,maruel@chromium.org,thakis@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=98637
Committed: https://crrev.com/823b8242e43a402224364a3068a011056d4f371a
Cr-Commit-Position: refs/heads/master@{#337347}

[Derek Bruening, 2015-07-05 03:43:26]

Created Revert of Convert ppapi_unittests to run exclusively on Swarming

[commit-bot: I haz the power, 2015-07-05 03:43:34]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1218743005/1

[commit-bot: I haz the power, 2015-07-05 03:44:15]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-07-05 03:45:03]

Patchset 1 (id:??) landed as
https://crrev.com/823b8242e43a402224364a3068a011056d4f371a
Cr-Commit-Position: refs/heads/master@{#337347}

[Nico, 2015-07-05 05:44:39]

This has 0 impact on the tsan bot. It doesn't change code, and it doesn't change
the tsan bot config (i.e. the tsan bot won't run tests differently). Are you
sure that this CL is to blame?

[Nico, 2015-07-05 05:45:46]

Oh, actually it does impact the bot config. I'll take a look I guess.
(maruel: Huh??)

On Sat, Jul 4, 2015 at 10:44 PM, <thakis@chromium.org> wrote:

> This has 0 impact on the tsan bot. It doesn't change code, and it doesn't
> change
> the tsan bot config (i.e. the tsan bot won't run tests differently). Are
> you
> sure that this CL is to blame?
>
> https://codereview.chromium.org/1218743005/
>

To unsubscribe from this group and stop receiving emails from it, send an email
to chromium-reviews+unsubscribe@chromium.org.

[M-A Ruel, 2015-07-06 15:30:42]

Derek, thanks for the revert but can you include the builder name next time? I
found it back as:
http://build.chromium.org/p/chromium.memory.fyi/builders/Linux%20TSan%20Tests...


On 2015/07/05 05:45:46, Nico wrote:
> Oh, actually it does impact the bot config. I'll take a look I guess.
> (maruel: Huh??)

From the CL description:

> - chromium.memory.fyi.json: Linux TSan Tests

so yes it did impact.

I ran:
./swarming.py reproduce -S chromium-swarm.appspot.com 2873c3cd8f4aab10

and it did fail locally in the same way.

Location is global '<null>' of size 0 at 0x000000000000
(ppapi_unittests+0x000001b94f44)

makes me think it's a missing library (?)

[chromium-reviews, 2015-07-06 16:53:27]

On Mon, Jul 6, 2015 at 11:30 AM, <maruel@chromium.org> wrote:

> Derek, thanks for the revert but can you include the builder name next
> time? I
> found it back as:
>
>
http://build.chromium.org/p/chromium.memory.fyi/builders/Linux%20TSan%20Tests...
>

Sure.  I had already sent details to Nico in an earlier email.

To unsubscribe from this group and stop receiving emails from it, send an email
to chromium-reviews+unsubscribe@chromium.org.