MD History: Remove unnecessary namespacing from tests

MD History: Remove unnecessary namespacing from tests (part 1)

These tests followed the (outdated) practice of defining tests
inside a dedicated `registerTests` function. This no longer has any
benefit for isolating tests, and just reduces the amount of horizontal
space available for test code.

This CL only removes these registerTests functions and deindents
the corresponding code -- there are no functional changes to any
tests.

BUG=647115
Committed: https://crrev.com/b3451f9f323fed4630432f989fa6c678089d96f8
Cr-Commit-Position: refs/heads/master@{#434433}

[tsergeant, 2016-11-16 00:14:28]

The CQ bit was checked by tsergeant@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-11-16 00:15:27]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-11-16 03:11:26]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-11-16 03:11:26]

Dry run: This issue passed the CQ dry run.

[tsergeant, 2016-11-21 04:28:56]

Description was changed from

==========
MD History: Remove unnecessary namespacing from tests

These tests followed the (outdated) practice of defining tests
inside a dedicated `registerTests` function. This no longer has any
benefit for isolating tests, and just reduces the amount of horizontal
space available for test code.

This CL only removes these registerTests functions and deindents
the corresponding code -- there are no functional changes to any
tests.

BUG=647115
==========

to

==========
MD History: Remove unnecessary namespacing from tests (part 1)

These tests followed the (outdated) practice of defining tests
inside a dedicated `registerTests` function. This no longer has any
benefit for isolating tests, and just reduces the amount of horizontal
space available for test code.

This CL only removes these registerTests functions and deindents
the corresponding code -- there are no functional changes to any
tests.

BUG=647115
==========

[tsergeant, 2016-11-21 04:29:26]

The CQ bit was checked by tsergeant@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-11-21 04:29:45]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[tsergeant, 2016-11-21 04:30:09]

tsergeant@chromium.org changed reviewers:
+ calamity@chromium.org

[tsergeant, 2016-11-21 04:30:10]

PTAL. Sorry this is a nasty thing to review, but like the description says,
there's no functional changes.

[commit-bot: I haz the power, 2016-11-21 06:21:01]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-11-21 06:21:02]

Dry run: This issue passed the CQ dry run.

[tsergeant, 2016-11-25 00:44:17]

On 2016/11/21 04:30:10, tsergeant wrote:
> PTAL. Sorry this is a nasty thing to review, but like the description says,
> there's no functional changes.

ping

[calamity, 2016-11-25 02:08:11]

lgtm

[tsergeant, 2016-11-25 02:45:25]

The CQ bit was checked by tsergeant@chromium.org

[commit-bot: I haz the power, 2016-11-25 02:45:39]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-11-25 03:26:03]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-11-25 03:26:04]

Try jobs failed on following builders:
  linux_chromium_clobber_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[tsergeant, 2016-11-25 03:37:57]

The CQ bit was checked by tsergeant@chromium.org

[commit-bot: I haz the power, 2016-11-25 03:38:07]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-11-25 04:35:51]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1480045077247220,
"parent_rev": "4f68408045509eab2325985338ea3f002fb78126", "commit_rev":
"2e58279c828c0536e94399ba435dcbb064682f09"}

[commit-bot: I haz the power, 2016-11-25 04:36:39]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-11-25 04:39:07]

Description was changed from

==========
MD History: Remove unnecessary namespacing from tests (part 1)

These tests followed the (outdated) practice of defining tests
inside a dedicated `registerTests` function. This no longer has any
benefit for isolating tests, and just reduces the amount of horizontal
space available for test code.

This CL only removes these registerTests functions and deindents
the corresponding code -- there are no functional changes to any
tests.

BUG=647115
==========

to

==========
MD History: Remove unnecessary namespacing from tests (part 1)

These tests followed the (outdated) practice of defining tests
inside a dedicated `registerTests` function. This no longer has any
benefit for isolating tests, and just reduces the amount of horizontal
space available for test code.

This CL only removes these registerTests functions and deindents
the corresponding code -- there are no functional changes to any
tests.

BUG=647115

Committed: https://crrev.com/b3451f9f323fed4630432f989fa6c678089d96f8
Cr-Commit-Position: refs/heads/master@{#434433}
==========

[commit-bot: I haz the power, 2016-11-25 04:39:07]

Patchset 3 (id:??) landed as
https://crrev.com/b3451f9f323fed4630432f989fa6c678089d96f8
Cr-Commit-Position: refs/heads/master@{#434433}