Run Oilpan GC in blink_platform_unittests.

Run Oilpan GC in blink_platform_unittests.

This fixes the issue that caused this revert:
  https://codereview.chromium.org/1450953002/

Essentially, this becomes the first platform unit test to have a heap-allocated
mock, which we have to clean up after. This is similar to the logic in
RunAllTests.cpp for webkit_unit_tests and heap_unittests.

OriginAccessEntryTest is adjusted to restore the old platform so that it doesn't
break subsequent tests (or in this case, the heap's attempt to clean itself up).
Currently it nulls out the platform in many of its tests, and this causes code
in the heap that attempts to record a histogram to crash. I've copied how
PlatformTestingSupport does platform init/restore.

Committed: https://crrev.com/dd85fe85c3986fd3cf647d27e23a10244126c212
Cr-Commit-Position: refs/heads/master@{#360126}

[jbroman, 2015-11-16 19:06:22]

Description was changed from

==========
Run Oilpan GC in blink_platform_unittests.

This fixes the issue that caused this revert:
  https://codereview.chromium.org/1450953002/

Essentially, this becomes the first platform unit test to have a heap-allocated
mock, which we have to clean up after. This is similar to the logic in
RunAllTests.cpp for webkit_unit_tests and heap_unittests.
==========

to

==========
Run Oilpan GC in blink_platform_unittests.

This fixes the issue that caused this revert:
  https://codereview.chromium.org/1450953002/

Essentially, this becomes the first platform unit test to have a heap-allocated
mock, which we have to clean up after. This is similar to the logic in
RunAllTests.cpp for webkit_unit_tests and heap_unittests.

OriginAccessEntryTest is adjusted to restore the old platform so that it doesn't
break subsequent tests (or in this case, the heap's attempt to clean itself up).
Currently it nulls out the platform in many of its tests, and this causes code
in the heap that attempts to record a histogram to crash. I've copied how
PlatformTestingSupport does platform init/restore.
==========

[jbroman, 2015-11-16 19:14:30]

jbroman@chromium.org changed reviewers:
+ haraken@chromium.org, mkwst@chromium.org

[jbroman, 2015-11-16 19:14:30]

haraken for Oilpan expertise
mkwst for Source/platform/weborigin/

[haraken, 2015-11-16 23:30:13]

LGTM

https://codereview.chromium.org/1448833003/diff/20001/third_party/WebKit/Sour...
File third_party/WebKit/Source/platform/testing/RunAllTests.cpp (right):

https://codereview.chromium.org/1448833003/diff/20001/third_party/WebKit/Sour...
third_party/WebKit/Source/platform/testing/RunAllTests.cpp:58:
blink::ThreadState::current()->registerTraceDOMWrappers(nullptr, nullptr);

This doesn't need to be called for each test. You can call it in the main
function.

[jbroman, 2015-11-17 16:16:52]

https://codereview.chromium.org/1448833003/diff/20001/third_party/WebKit/Sour...
File third_party/WebKit/Source/platform/testing/RunAllTests.cpp (right):

https://codereview.chromium.org/1448833003/diff/20001/third_party/WebKit/Sour...
third_party/WebKit/Source/platform/testing/RunAllTests.cpp:58:
blink::ThreadState::current()->registerTraceDOMWrappers(nullptr, nullptr);
On 2015/11/16 at 23:30:13, haraken wrote:
> This doesn't need to be called for each test. You can call it in the main
function.

Moved at your request. (I'd put it here to match blink_heap_unittests.)

[Mike West, 2015-11-17 18:47:02]

LGTM!

[jbroman, 2015-11-17 19:02:40]

The CQ bit was checked by jbroman@chromium.org

[jbroman, 2015-11-17 19:02:40]

The patchset sent to the CQ was uploaded after l-g-t-m from haraken@chromium.org
Link to the patchset: https://codereview.chromium.org/1448833003/#ps40001
(title: "move registerTraceDOMWrappers call")

[commit-bot: I haz the power, 2015-11-17 19:04:37]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1448833003/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1448833003/40001

[commit-bot: I haz the power, 2015-11-17 19:10:45]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2015-11-17 19:11:44]

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