HeapCompact doesn't need to support multiple threads

HeapCompact doesn't need to support multiple threads

Now that the per-thread heaps have been shipped, HeapCompact no longer needs to
support multiple threads. This CL removes that code.

BUG=671856
Review-Url: https://codereview.chromium.org/2633463006
Cr-Commit-Position: refs/heads/master@{#443542}
Committed: https://chromium.googlesource.com/chromium/src/+/6422f3b2391675817cc28ed311776067853f5757

[haraken, 2017-01-13 09:26:23]

Description was changed from

==========
HeapCompact doesn't need to support multiple threads

Now that the per-thread heaps have been shipped, HeapCompact no longer needs to
support multiple threads. This CL removes that code.

BUG=
==========

to

==========
HeapCompact doesn't need to support multiple threads

Now that the per-thread heaps have been shipped, HeapCompact no longer needs to
support multiple threads. This CL removes that code.

BUG=671856
==========

[haraken, 2017-01-13 09:26:34]

haraken@chromium.org changed reviewers:
+ sigbjornf@opera.com

[haraken, 2017-01-13 09:26:34]

PTAL

[sof, 2017-01-13 09:36:28]

lgtm

https://codereview.chromium.org/2633463006/diff/1/third_party/WebKit/Source/p...
File third_party/WebKit/Source/platform/heap/HeapCompact.cpp (right):

https://codereview.chromium.org/2633463006/diff/1/third_party/WebKit/Source/p...
third_party/WebKit/Source/platform/heap/HeapCompact.cpp:454: // Compaction has
been completed by all participating threads, unblock
just remove?

[haraken, 2017-01-13 09:49:27]

The CQ bit was checked by haraken@chromium.org

[haraken, 2017-01-13 09:49:27]

The patchset sent to the CQ was uploaded after l-g-t-m from sigbjornf@opera.com
Link to the patchset: https://codereview.chromium.org/2633463006/#ps20001
(title: "temp")

[commit-bot: I haz the power, 2017-01-13 09:49:48]

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, 2017-01-13 12:45:35]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1484300967225770,
"parent_rev": "989dace104cd4bc1aa304c9c30c54e6cff0d894d", "commit_rev":
"6422f3b2391675817cc28ed311776067853f5757"}

[commit-bot: I haz the power, 2017-01-13 12:46:15]

Description was changed from

==========
HeapCompact doesn't need to support multiple threads

Now that the per-thread heaps have been shipped, HeapCompact no longer needs to
support multiple threads. This CL removes that code.

BUG=671856
==========

to

==========
HeapCompact doesn't need to support multiple threads

Now that the per-thread heaps have been shipped, HeapCompact no longer needs to
support multiple threads. This CL removes that code.

BUG=671856

Review-Url: https://codereview.chromium.org/2633463006
Cr-Commit-Position: refs/heads/master@{#443542}
Committed:
https://chromium.googlesource.com/chromium/src/+/6422f3b2391675817cc28ed31177...
==========

[commit-bot: I haz the power, 2017-01-13 12:46:16]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/6422f3b2391675817cc28ed31177...