New issue
Advanced search Search tips

Issue 591606 link

Starred by 3 users

Issue metadata

Status: Fixed
Owner:
Closed: Jan 2017
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug

Blocking:
issue 671856



Sign in to add a comment

Implement oilpan per thread heap

Project Member Reported by keishi@chromium.org, Mar 3 2016

Issue description

Implement per thread heap so that oilpan gc won't need to stop all threads.
 
Components: -Blink>Oilpan Blink>MemoryAllocator>GarbageCollection
Project Member

Comment 2 by bugdroid1@chromium.org, Mar 3 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/dabce1df8fba05b62fddc37ecf76a32eae560214

commit dabce1df8fba05b62fddc37ecf76a32eae560214
Author: keishi <keishi@chromium.org>
Date: Thu Mar 03 06:28:16 2016

Rename BaseHeap to BaseArena

Rename BaseHeap to BaseArena so the distiction from Heap is clear.

BUG= 591606 

Review URL: https://codereview.chromium.org/1754183002

Cr-Commit-Position: refs/heads/master@{#378971}

[modify] https://crrev.com/dabce1df8fba05b62fddc37ecf76a32eae560214/third_party/WebKit/Source/core/css/CSSValue.h
[modify] https://crrev.com/dabce1df8fba05b62fddc37ecf76a32eae560214/third_party/WebKit/Source/core/dom/Node.h
[modify] https://crrev.com/dabce1df8fba05b62fddc37ecf76a32eae560214/third_party/WebKit/Source/platform/heap/BlinkGC.h
[modify] https://crrev.com/dabce1df8fba05b62fddc37ecf76a32eae560214/third_party/WebKit/Source/platform/heap/BlinkGCDesign.md
[modify] https://crrev.com/dabce1df8fba05b62fddc37ecf76a32eae560214/third_party/WebKit/Source/platform/heap/Heap.cpp
[modify] https://crrev.com/dabce1df8fba05b62fddc37ecf76a32eae560214/third_party/WebKit/Source/platform/heap/Heap.h
[modify] https://crrev.com/dabce1df8fba05b62fddc37ecf76a32eae560214/third_party/WebKit/Source/platform/heap/HeapAllocator.cpp
[modify] https://crrev.com/dabce1df8fba05b62fddc37ecf76a32eae560214/third_party/WebKit/Source/platform/heap/HeapAllocator.h
[modify] https://crrev.com/dabce1df8fba05b62fddc37ecf76a32eae560214/third_party/WebKit/Source/platform/heap/HeapPage.cpp
[modify] https://crrev.com/dabce1df8fba05b62fddc37ecf76a32eae560214/third_party/WebKit/Source/platform/heap/HeapPage.h
[modify] https://crrev.com/dabce1df8fba05b62fddc37ecf76a32eae560214/third_party/WebKit/Source/platform/heap/HeapTest.cpp
[modify] https://crrev.com/dabce1df8fba05b62fddc37ecf76a32eae560214/third_party/WebKit/Source/platform/heap/MarkingVisitorImpl.h
[modify] https://crrev.com/dabce1df8fba05b62fddc37ecf76a32eae560214/third_party/WebKit/Source/platform/heap/PagePool.cpp
[modify] https://crrev.com/dabce1df8fba05b62fddc37ecf76a32eae560214/third_party/WebKit/Source/platform/heap/PagePool.h
[modify] https://crrev.com/dabce1df8fba05b62fddc37ecf76a32eae560214/third_party/WebKit/Source/platform/heap/PersistentNode.cpp
[modify] https://crrev.com/dabce1df8fba05b62fddc37ecf76a32eae560214/third_party/WebKit/Source/platform/heap/ThreadState.cpp
[modify] https://crrev.com/dabce1df8fba05b62fddc37ecf76a32eae560214/third_party/WebKit/Source/platform/heap/ThreadState.h
[modify] https://crrev.com/dabce1df8fba05b62fddc37ecf76a32eae560214/third_party/WebKit/Source/platform/heap/TraceTraits.h

Project Member

Comment 3 by bugdroid1@chromium.org, Mar 3 2016

Labels: merge-merged-2666
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/dabce1df8fba05b62fddc37ecf76a32eae560214

commit dabce1df8fba05b62fddc37ecf76a32eae560214
Author: keishi <keishi@chromium.org>
Date: Thu Mar 03 06:28:16 2016

Rename BaseHeap to BaseArena

Rename BaseHeap to BaseArena so the distiction from Heap is clear.

BUG= 591606 

Review URL: https://codereview.chromium.org/1754183002

Cr-Commit-Position: refs/heads/master@{#378971}

[modify] https://crrev.com/dabce1df8fba05b62fddc37ecf76a32eae560214/third_party/WebKit/Source/core/css/CSSValue.h
[modify] https://crrev.com/dabce1df8fba05b62fddc37ecf76a32eae560214/third_party/WebKit/Source/core/dom/Node.h
[modify] https://crrev.com/dabce1df8fba05b62fddc37ecf76a32eae560214/third_party/WebKit/Source/platform/heap/BlinkGC.h
[modify] https://crrev.com/dabce1df8fba05b62fddc37ecf76a32eae560214/third_party/WebKit/Source/platform/heap/BlinkGCDesign.md
[modify] https://crrev.com/dabce1df8fba05b62fddc37ecf76a32eae560214/third_party/WebKit/Source/platform/heap/Heap.cpp
[modify] https://crrev.com/dabce1df8fba05b62fddc37ecf76a32eae560214/third_party/WebKit/Source/platform/heap/Heap.h
[modify] https://crrev.com/dabce1df8fba05b62fddc37ecf76a32eae560214/third_party/WebKit/Source/platform/heap/HeapAllocator.cpp
[modify] https://crrev.com/dabce1df8fba05b62fddc37ecf76a32eae560214/third_party/WebKit/Source/platform/heap/HeapAllocator.h
[modify] https://crrev.com/dabce1df8fba05b62fddc37ecf76a32eae560214/third_party/WebKit/Source/platform/heap/HeapPage.cpp
[modify] https://crrev.com/dabce1df8fba05b62fddc37ecf76a32eae560214/third_party/WebKit/Source/platform/heap/HeapPage.h
[modify] https://crrev.com/dabce1df8fba05b62fddc37ecf76a32eae560214/third_party/WebKit/Source/platform/heap/HeapTest.cpp
[modify] https://crrev.com/dabce1df8fba05b62fddc37ecf76a32eae560214/third_party/WebKit/Source/platform/heap/MarkingVisitorImpl.h
[modify] https://crrev.com/dabce1df8fba05b62fddc37ecf76a32eae560214/third_party/WebKit/Source/platform/heap/PagePool.cpp
[modify] https://crrev.com/dabce1df8fba05b62fddc37ecf76a32eae560214/third_party/WebKit/Source/platform/heap/PagePool.h
[modify] https://crrev.com/dabce1df8fba05b62fddc37ecf76a32eae560214/third_party/WebKit/Source/platform/heap/PersistentNode.cpp
[modify] https://crrev.com/dabce1df8fba05b62fddc37ecf76a32eae560214/third_party/WebKit/Source/platform/heap/ThreadState.cpp
[modify] https://crrev.com/dabce1df8fba05b62fddc37ecf76a32eae560214/third_party/WebKit/Source/platform/heap/ThreadState.h
[modify] https://crrev.com/dabce1df8fba05b62fddc37ecf76a32eae560214/third_party/WebKit/Source/platform/heap/TraceTraits.h

Project Member

Comment 9 by bugdroid1@chromium.org, Apr 14 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/191c640cc25f76c0c0f8a0b448de33cd059f4a8e

commit 191c640cc25f76c0c0f8a0b448de33cd059f4a8e
Author: keishi <keishi@chromium.org>
Date: Thu Apr 14 04:04:13 2016

Rename Heap to ThreadHeap

BUG= 591606 
CQ_EXTRA_TRYBOTS=tryserver.chromium.perf:winx64_10_perf_cq;tryserver.chromium.perf:mac_retina_perf_cq;tryserver.chromium.perf:linux_perf_cq

Review URL: https://codereview.chromium.org/1845543002

Cr-Commit-Position: refs/heads/master@{#387233}

[modify] https://crrev.com/191c640cc25f76c0c0f8a0b448de33cd059f4a8e/third_party/WebKit/Source/bindings/core/v8/ScriptPromiseResolverTest.cpp
[modify] https://crrev.com/191c640cc25f76c0c0f8a0b448de33cd059f4a8e/third_party/WebKit/Source/bindings/core/v8/V8GCController.cpp
[modify] https://crrev.com/191c640cc25f76c0c0f8a0b448de33cd059f4a8e/third_party/WebKit/Source/bindings/core/v8/WrapperTypeInfo.h
[modify] https://crrev.com/191c640cc25f76c0c0f8a0b448de33cd059f4a8e/third_party/WebKit/Source/core/animation/AnimationInputHelpersTest.cpp
[modify] https://crrev.com/191c640cc25f76c0c0f8a0b448de33cd059f4a8e/third_party/WebKit/Source/core/animation/AnimationStackTest.cpp
[modify] https://crrev.com/191c640cc25f76c0c0f8a0b448de33cd059f4a8e/third_party/WebKit/Source/core/animation/AnimationTest.cpp
[modify] https://crrev.com/191c640cc25f76c0c0f8a0b448de33cd059f4a8e/third_party/WebKit/Source/core/animation/AnimationTimelineTest.cpp
[modify] https://crrev.com/191c640cc25f76c0c0f8a0b448de33cd059f4a8e/third_party/WebKit/Source/core/animation/CompositorAnimationsTest.cpp
[modify] https://crrev.com/191c640cc25f76c0c0f8a0b448de33cd059f4a8e/third_party/WebKit/Source/core/css/CSSStyleSheetResourceTest.cpp
[modify] https://crrev.com/191c640cc25f76c0c0f8a0b448de33cd059f4a8e/third_party/WebKit/Source/core/css/CSSValue.h
[modify] https://crrev.com/191c640cc25f76c0c0f8a0b448de33cd059f4a8e/third_party/WebKit/Source/core/css/StylePropertySet.cpp
[modify] https://crrev.com/191c640cc25f76c0c0f8a0b448de33cd059f4a8e/third_party/WebKit/Source/core/css/resolver/MatchedPropertiesCache.h
[modify] https://crrev.com/191c640cc25f76c0c0f8a0b448de33cd059f4a8e/third_party/WebKit/Source/core/dom/CrossThreadTaskTest.cpp
[modify] https://crrev.com/191c640cc25f76c0c0f8a0b448de33cd059f4a8e/third_party/WebKit/Source/core/dom/DocumentStatisticsCollectorTest.cpp
[modify] https://crrev.com/191c640cc25f76c0c0f8a0b448de33cd059f4a8e/third_party/WebKit/Source/core/dom/DocumentTest.cpp
[modify] https://crrev.com/191c640cc25f76c0c0f8a0b448de33cd059f4a8e/third_party/WebKit/Source/core/dom/ElementData.cpp
[modify] https://crrev.com/191c640cc25f76c0c0f8a0b448de33cd059f4a8e/third_party/WebKit/Source/core/dom/ElementData.h
[modify] https://crrev.com/191c640cc25f76c0c0f8a0b448de33cd059f4a8e/third_party/WebKit/Source/core/dom/IntersectionObserver.cpp
[modify] https://crrev.com/191c640cc25f76c0c0f8a0b448de33cd059f4a8e/third_party/WebKit/Source/core/dom/Node.h
[modify] https://crrev.com/191c640cc25f76c0c0f8a0b448de33cd059f4a8e/third_party/WebKit/Source/core/dom/ScriptRunnerTest.cpp
[modify] https://crrev.com/191c640cc25f76c0c0f8a0b448de33cd059f4a8e/third_party/WebKit/Source/core/editing/markers/DocumentMarkerControllerTest.cpp
[modify] https://crrev.com/191c640cc25f76c0c0f8a0b448de33cd059f4a8e/third_party/WebKit/Source/core/frame/EventHandlerRegistry.cpp
[modify] https://crrev.com/191c640cc25f76c0c0f8a0b448de33cd059f4a8e/third_party/WebKit/Source/core/frame/ImageBitmapTest.cpp
[modify] https://crrev.com/191c640cc25f76c0c0f8a0b448de33cd059f4a8e/third_party/WebKit/Source/core/html/AutoplayExperimentTest.cpp
[modify] https://crrev.com/191c640cc25f76c0c0f8a0b448de33cd059f4a8e/third_party/WebKit/Source/core/html/HTMLMediaElement.cpp
[modify] https://crrev.com/191c640cc25f76c0c0f8a0b448de33cd059f4a8e/third_party/WebKit/Source/core/svg/SVGElementRareData.cpp
[modify] https://crrev.com/191c640cc25f76c0c0f8a0b448de33cd059f4a8e/third_party/WebKit/Source/core/svg/graphics/SVGImageChromeClient.cpp
[modify] https://crrev.com/191c640cc25f76c0c0f8a0b448de33cd059f4a8e/third_party/WebKit/Source/devtools/front_end/timeline/TimelineModel.js
[modify] https://crrev.com/191c640cc25f76c0c0f8a0b448de33cd059f4a8e/third_party/WebKit/Source/modules/canvas2d/CanvasRenderingContext2DTest.cpp
[modify] https://crrev.com/191c640cc25f76c0c0f8a0b448de33cd059f4a8e/third_party/WebKit/Source/modules/csspaint/PaintWorkletTest.cpp
[modify] https://crrev.com/191c640cc25f76c0c0f8a0b448de33cd059f4a8e/third_party/WebKit/Source/modules/fetch/BodyStreamBufferTest.cpp
[modify] https://crrev.com/191c640cc25f76c0c0f8a0b448de33cd059f4a8e/third_party/WebKit/Source/modules/fetch/DataConsumerTeeTest.cpp
[modify] https://crrev.com/191c640cc25f76c0c0f8a0b448de33cd059f4a8e/third_party/WebKit/Source/modules/fetch/FetchBlobDataConsumerHandleTest.cpp
[modify] https://crrev.com/191c640cc25f76c0c0f8a0b448de33cd059f4a8e/third_party/WebKit/Source/modules/indexeddb/IDBTransactionTest.cpp
[modify] https://crrev.com/191c640cc25f76c0c0f8a0b448de33cd059f4a8e/third_party/WebKit/Source/platform/LifecycleContextTest.cpp
[modify] https://crrev.com/191c640cc25f76c0c0f8a0b448de33cd059f4a8e/third_party/WebKit/Source/platform/Timer.h
[modify] https://crrev.com/191c640cc25f76c0c0f8a0b448de33cd059f4a8e/third_party/WebKit/Source/platform/exported/Platform.cpp
[modify] https://crrev.com/191c640cc25f76c0c0f8a0b448de33cd059f4a8e/third_party/WebKit/Source/platform/heap/BlinkGC.h
[modify] https://crrev.com/191c640cc25f76c0c0f8a0b448de33cd059f4a8e/third_party/WebKit/Source/platform/heap/BlinkGCMemoryDumpProvider.cpp
[modify] https://crrev.com/191c640cc25f76c0c0f8a0b448de33cd059f4a8e/third_party/WebKit/Source/platform/heap/GarbageCollected.h
[modify] https://crrev.com/191c640cc25f76c0c0f8a0b448de33cd059f4a8e/third_party/WebKit/Source/platform/heap/Handle.h
[modify] https://crrev.com/191c640cc25f76c0c0f8a0b448de33cd059f4a8e/third_party/WebKit/Source/platform/heap/Heap.cpp
[modify] https://crrev.com/191c640cc25f76c0c0f8a0b448de33cd059f4a8e/third_party/WebKit/Source/platform/heap/Heap.h
[modify] https://crrev.com/191c640cc25f76c0c0f8a0b448de33cd059f4a8e/third_party/WebKit/Source/platform/heap/HeapAllocator.h
[modify] https://crrev.com/191c640cc25f76c0c0f8a0b448de33cd059f4a8e/third_party/WebKit/Source/platform/heap/HeapPage.cpp
[modify] https://crrev.com/191c640cc25f76c0c0f8a0b448de33cd059f4a8e/third_party/WebKit/Source/platform/heap/HeapPage.h
[modify] https://crrev.com/191c640cc25f76c0c0f8a0b448de33cd059f4a8e/third_party/WebKit/Source/platform/heap/HeapTerminatedArray.h
[modify] https://crrev.com/191c640cc25f76c0c0f8a0b448de33cd059f4a8e/third_party/WebKit/Source/platform/heap/HeapTest.cpp
[modify] https://crrev.com/191c640cc25f76c0c0f8a0b448de33cd059f4a8e/third_party/WebKit/Source/platform/heap/MarkingVisitorImpl.h
[modify] https://crrev.com/191c640cc25f76c0c0f8a0b448de33cd059f4a8e/third_party/WebKit/Source/platform/heap/PagePool.cpp
[modify] https://crrev.com/191c640cc25f76c0c0f8a0b448de33cd059f4a8e/third_party/WebKit/Source/platform/heap/RunAllTests.cpp
[modify] https://crrev.com/191c640cc25f76c0c0f8a0b448de33cd059f4a8e/third_party/WebKit/Source/platform/heap/ThreadState.cpp
[modify] https://crrev.com/191c640cc25f76c0c0f8a0b448de33cd059f4a8e/third_party/WebKit/Source/platform/heap/ThreadState.h
[modify] https://crrev.com/191c640cc25f76c0c0f8a0b448de33cd059f4a8e/third_party/WebKit/Source/platform/scroll/ScrollAnimatorTest.cpp
[modify] https://crrev.com/191c640cc25f76c0c0f8a0b448de33cd059f4a8e/third_party/WebKit/Source/platform/scroll/ScrollableAreaTest.cpp
[modify] https://crrev.com/191c640cc25f76c0c0f8a0b448de33cd059f4a8e/third_party/WebKit/Source/platform/testing/RunAllTests.cpp
[modify] https://crrev.com/191c640cc25f76c0c0f8a0b448de33cd059f4a8e/third_party/WebKit/Source/web/WebFrame.cpp
[modify] https://crrev.com/191c640cc25f76c0c0f8a0b448de33cd059f4a8e/third_party/WebKit/Source/web/WebHeap.cpp
[modify] https://crrev.com/191c640cc25f76c0c0f8a0b448de33cd059f4a8e/third_party/WebKit/Source/web/tests/WebViewTest.cpp
[modify] https://crrev.com/191c640cc25f76c0c0f8a0b448de33cd059f4a8e/tools/perf/measurements/oilpan_gc_times.py
[modify] https://crrev.com/191c640cc25f76c0c0f8a0b448de33cd059f4a8e/tools/perf/measurements/oilpan_gc_times_unittest.py

Project Member

Comment 10 by bugdroid1@chromium.org, Apr 14 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/b298b9354505c3ded029dadd92675ca3405a6381

commit b298b9354505c3ded029dadd92675ca3405a6381
Author: keishi <keishi@chromium.org>
Date: Thu Apr 14 07:12:38 2016

Revert of Rename Heap to ThreadHeap (patchset #13 id:240001 of https://codereview.chromium.org/1845543002/ )

Reason for revert:
Failed memory_test: content_unittests

Original issue's description:
> Rename Heap to ThreadHeap
>
> BUG= 591606 
> CQ_EXTRA_TRYBOTS=tryserver.chromium.perf:winx64_10_perf_cq;tryserver.chromium.perf:mac_retina_perf_cq;tryserver.chromium.perf:linux_perf_cq
>
> Committed: https://crrev.com/191c640cc25f76c0c0f8a0b448de33cd059f4a8e
> Cr-Commit-Position: refs/heads/master@{#387233}

TBR=oilpan-reviews@chromium.org,haraken@chromium.org,peria@chromium.org,caseq@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG= 591606 

Review URL: https://codereview.chromium.org/1884083002

Cr-Commit-Position: refs/heads/master@{#387254}

[modify] https://crrev.com/b298b9354505c3ded029dadd92675ca3405a6381/third_party/WebKit/Source/bindings/core/v8/ScriptPromiseResolverTest.cpp
[modify] https://crrev.com/b298b9354505c3ded029dadd92675ca3405a6381/third_party/WebKit/Source/bindings/core/v8/V8GCController.cpp
[modify] https://crrev.com/b298b9354505c3ded029dadd92675ca3405a6381/third_party/WebKit/Source/bindings/core/v8/WrapperTypeInfo.h
[modify] https://crrev.com/b298b9354505c3ded029dadd92675ca3405a6381/third_party/WebKit/Source/core/animation/AnimationInputHelpersTest.cpp
[modify] https://crrev.com/b298b9354505c3ded029dadd92675ca3405a6381/third_party/WebKit/Source/core/animation/AnimationStackTest.cpp
[modify] https://crrev.com/b298b9354505c3ded029dadd92675ca3405a6381/third_party/WebKit/Source/core/animation/AnimationTest.cpp
[modify] https://crrev.com/b298b9354505c3ded029dadd92675ca3405a6381/third_party/WebKit/Source/core/animation/AnimationTimelineTest.cpp
[modify] https://crrev.com/b298b9354505c3ded029dadd92675ca3405a6381/third_party/WebKit/Source/core/animation/CompositorAnimationsTest.cpp
[modify] https://crrev.com/b298b9354505c3ded029dadd92675ca3405a6381/third_party/WebKit/Source/core/css/CSSStyleSheetResourceTest.cpp
[modify] https://crrev.com/b298b9354505c3ded029dadd92675ca3405a6381/third_party/WebKit/Source/core/css/CSSValue.h
[modify] https://crrev.com/b298b9354505c3ded029dadd92675ca3405a6381/third_party/WebKit/Source/core/css/StylePropertySet.cpp
[modify] https://crrev.com/b298b9354505c3ded029dadd92675ca3405a6381/third_party/WebKit/Source/core/css/resolver/MatchedPropertiesCache.h
[modify] https://crrev.com/b298b9354505c3ded029dadd92675ca3405a6381/third_party/WebKit/Source/core/dom/CrossThreadTaskTest.cpp
[modify] https://crrev.com/b298b9354505c3ded029dadd92675ca3405a6381/third_party/WebKit/Source/core/dom/DocumentStatisticsCollectorTest.cpp
[modify] https://crrev.com/b298b9354505c3ded029dadd92675ca3405a6381/third_party/WebKit/Source/core/dom/DocumentTest.cpp
[modify] https://crrev.com/b298b9354505c3ded029dadd92675ca3405a6381/third_party/WebKit/Source/core/dom/ElementData.cpp
[modify] https://crrev.com/b298b9354505c3ded029dadd92675ca3405a6381/third_party/WebKit/Source/core/dom/ElementData.h
[modify] https://crrev.com/b298b9354505c3ded029dadd92675ca3405a6381/third_party/WebKit/Source/core/dom/IntersectionObserver.cpp
[modify] https://crrev.com/b298b9354505c3ded029dadd92675ca3405a6381/third_party/WebKit/Source/core/dom/Node.h
[modify] https://crrev.com/b298b9354505c3ded029dadd92675ca3405a6381/third_party/WebKit/Source/core/dom/ScriptRunnerTest.cpp
[modify] https://crrev.com/b298b9354505c3ded029dadd92675ca3405a6381/third_party/WebKit/Source/core/editing/markers/DocumentMarkerControllerTest.cpp
[modify] https://crrev.com/b298b9354505c3ded029dadd92675ca3405a6381/third_party/WebKit/Source/core/frame/EventHandlerRegistry.cpp
[modify] https://crrev.com/b298b9354505c3ded029dadd92675ca3405a6381/third_party/WebKit/Source/core/frame/ImageBitmapTest.cpp
[modify] https://crrev.com/b298b9354505c3ded029dadd92675ca3405a6381/third_party/WebKit/Source/core/html/AutoplayExperimentTest.cpp
[modify] https://crrev.com/b298b9354505c3ded029dadd92675ca3405a6381/third_party/WebKit/Source/core/html/HTMLMediaElement.cpp
[modify] https://crrev.com/b298b9354505c3ded029dadd92675ca3405a6381/third_party/WebKit/Source/core/svg/SVGElementRareData.cpp
[modify] https://crrev.com/b298b9354505c3ded029dadd92675ca3405a6381/third_party/WebKit/Source/core/svg/graphics/SVGImageChromeClient.cpp
[modify] https://crrev.com/b298b9354505c3ded029dadd92675ca3405a6381/third_party/WebKit/Source/devtools/front_end/timeline/TimelineModel.js
[modify] https://crrev.com/b298b9354505c3ded029dadd92675ca3405a6381/third_party/WebKit/Source/modules/canvas2d/CanvasRenderingContext2DTest.cpp
[modify] https://crrev.com/b298b9354505c3ded029dadd92675ca3405a6381/third_party/WebKit/Source/modules/csspaint/PaintWorkletTest.cpp
[modify] https://crrev.com/b298b9354505c3ded029dadd92675ca3405a6381/third_party/WebKit/Source/modules/fetch/BodyStreamBufferTest.cpp
[modify] https://crrev.com/b298b9354505c3ded029dadd92675ca3405a6381/third_party/WebKit/Source/modules/fetch/DataConsumerTeeTest.cpp
[modify] https://crrev.com/b298b9354505c3ded029dadd92675ca3405a6381/third_party/WebKit/Source/modules/fetch/FetchBlobDataConsumerHandleTest.cpp
[modify] https://crrev.com/b298b9354505c3ded029dadd92675ca3405a6381/third_party/WebKit/Source/modules/indexeddb/IDBTransactionTest.cpp
[modify] https://crrev.com/b298b9354505c3ded029dadd92675ca3405a6381/third_party/WebKit/Source/platform/LifecycleContextTest.cpp
[modify] https://crrev.com/b298b9354505c3ded029dadd92675ca3405a6381/third_party/WebKit/Source/platform/Timer.h
[modify] https://crrev.com/b298b9354505c3ded029dadd92675ca3405a6381/third_party/WebKit/Source/platform/exported/Platform.cpp
[modify] https://crrev.com/b298b9354505c3ded029dadd92675ca3405a6381/third_party/WebKit/Source/platform/heap/BlinkGC.h
[modify] https://crrev.com/b298b9354505c3ded029dadd92675ca3405a6381/third_party/WebKit/Source/platform/heap/BlinkGCMemoryDumpProvider.cpp
[modify] https://crrev.com/b298b9354505c3ded029dadd92675ca3405a6381/third_party/WebKit/Source/platform/heap/GarbageCollected.h
[modify] https://crrev.com/b298b9354505c3ded029dadd92675ca3405a6381/third_party/WebKit/Source/platform/heap/Handle.h
[modify] https://crrev.com/b298b9354505c3ded029dadd92675ca3405a6381/third_party/WebKit/Source/platform/heap/Heap.cpp
[modify] https://crrev.com/b298b9354505c3ded029dadd92675ca3405a6381/third_party/WebKit/Source/platform/heap/Heap.h
[modify] https://crrev.com/b298b9354505c3ded029dadd92675ca3405a6381/third_party/WebKit/Source/platform/heap/HeapAllocator.h
[modify] https://crrev.com/b298b9354505c3ded029dadd92675ca3405a6381/third_party/WebKit/Source/platform/heap/HeapPage.cpp
[modify] https://crrev.com/b298b9354505c3ded029dadd92675ca3405a6381/third_party/WebKit/Source/platform/heap/HeapPage.h
[modify] https://crrev.com/b298b9354505c3ded029dadd92675ca3405a6381/third_party/WebKit/Source/platform/heap/HeapTerminatedArray.h
[modify] https://crrev.com/b298b9354505c3ded029dadd92675ca3405a6381/third_party/WebKit/Source/platform/heap/HeapTest.cpp
[modify] https://crrev.com/b298b9354505c3ded029dadd92675ca3405a6381/third_party/WebKit/Source/platform/heap/MarkingVisitorImpl.h
[modify] https://crrev.com/b298b9354505c3ded029dadd92675ca3405a6381/third_party/WebKit/Source/platform/heap/PagePool.cpp
[modify] https://crrev.com/b298b9354505c3ded029dadd92675ca3405a6381/third_party/WebKit/Source/platform/heap/RunAllTests.cpp
[modify] https://crrev.com/b298b9354505c3ded029dadd92675ca3405a6381/third_party/WebKit/Source/platform/heap/ThreadState.cpp
[modify] https://crrev.com/b298b9354505c3ded029dadd92675ca3405a6381/third_party/WebKit/Source/platform/heap/ThreadState.h
[modify] https://crrev.com/b298b9354505c3ded029dadd92675ca3405a6381/third_party/WebKit/Source/platform/scroll/ScrollAnimatorTest.cpp
[modify] https://crrev.com/b298b9354505c3ded029dadd92675ca3405a6381/third_party/WebKit/Source/platform/scroll/ScrollableAreaTest.cpp
[modify] https://crrev.com/b298b9354505c3ded029dadd92675ca3405a6381/third_party/WebKit/Source/platform/testing/RunAllTests.cpp
[modify] https://crrev.com/b298b9354505c3ded029dadd92675ca3405a6381/third_party/WebKit/Source/web/WebFrame.cpp
[modify] https://crrev.com/b298b9354505c3ded029dadd92675ca3405a6381/third_party/WebKit/Source/web/WebHeap.cpp
[modify] https://crrev.com/b298b9354505c3ded029dadd92675ca3405a6381/third_party/WebKit/Source/web/tests/WebViewTest.cpp
[modify] https://crrev.com/b298b9354505c3ded029dadd92675ca3405a6381/tools/perf/measurements/oilpan_gc_times.py
[modify] https://crrev.com/b298b9354505c3ded029dadd92675ca3405a6381/tools/perf/measurements/oilpan_gc_times_unittest.py

Project Member

Comment 11 by bugdroid1@chromium.org, Apr 14 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/011b8046a07ac2cedde530a05f81e8fb41f80c62

commit 011b8046a07ac2cedde530a05f81e8fb41f80c62
Author: keishi <keishi@chromium.org>
Date: Thu Apr 14 14:21:40 2016

Rename Heap to ThreadHeap

BUG= 591606 
CQ_EXTRA_TRYBOTS=tryserver.chromium.perf:winx64_10_perf_cq;tryserver.chromium.perf:mac_retina_perf_cq;tryserver.chromium.perf:linux_perf_cq

Committed: https://crrev.com/191c640cc25f76c0c0f8a0b448de33cd059f4a8e
Cr-Commit-Position: refs/heads/master@{#387233}

Review URL: https://codereview.chromium.org/1845543002

Cr-Commit-Position: refs/heads/master@{#387307}

[modify] https://crrev.com/011b8046a07ac2cedde530a05f81e8fb41f80c62/third_party/WebKit/Source/bindings/core/v8/ScriptPromiseResolverTest.cpp
[modify] https://crrev.com/011b8046a07ac2cedde530a05f81e8fb41f80c62/third_party/WebKit/Source/bindings/core/v8/V8GCController.cpp
[modify] https://crrev.com/011b8046a07ac2cedde530a05f81e8fb41f80c62/third_party/WebKit/Source/bindings/core/v8/WrapperTypeInfo.h
[modify] https://crrev.com/011b8046a07ac2cedde530a05f81e8fb41f80c62/third_party/WebKit/Source/core/animation/AnimationInputHelpersTest.cpp
[modify] https://crrev.com/011b8046a07ac2cedde530a05f81e8fb41f80c62/third_party/WebKit/Source/core/animation/AnimationStackTest.cpp
[modify] https://crrev.com/011b8046a07ac2cedde530a05f81e8fb41f80c62/third_party/WebKit/Source/core/animation/AnimationTest.cpp
[modify] https://crrev.com/011b8046a07ac2cedde530a05f81e8fb41f80c62/third_party/WebKit/Source/core/animation/AnimationTimelineTest.cpp
[modify] https://crrev.com/011b8046a07ac2cedde530a05f81e8fb41f80c62/third_party/WebKit/Source/core/animation/CompositorAnimationsTest.cpp
[modify] https://crrev.com/011b8046a07ac2cedde530a05f81e8fb41f80c62/third_party/WebKit/Source/core/css/CSSStyleSheetResourceTest.cpp
[modify] https://crrev.com/011b8046a07ac2cedde530a05f81e8fb41f80c62/third_party/WebKit/Source/core/css/CSSValue.h
[modify] https://crrev.com/011b8046a07ac2cedde530a05f81e8fb41f80c62/third_party/WebKit/Source/core/css/StylePropertySet.cpp
[modify] https://crrev.com/011b8046a07ac2cedde530a05f81e8fb41f80c62/third_party/WebKit/Source/core/css/resolver/MatchedPropertiesCache.h
[modify] https://crrev.com/011b8046a07ac2cedde530a05f81e8fb41f80c62/third_party/WebKit/Source/core/dom/CrossThreadTaskTest.cpp
[modify] https://crrev.com/011b8046a07ac2cedde530a05f81e8fb41f80c62/third_party/WebKit/Source/core/dom/DocumentStatisticsCollectorTest.cpp
[modify] https://crrev.com/011b8046a07ac2cedde530a05f81e8fb41f80c62/third_party/WebKit/Source/core/dom/DocumentTest.cpp
[modify] https://crrev.com/011b8046a07ac2cedde530a05f81e8fb41f80c62/third_party/WebKit/Source/core/dom/ElementData.cpp
[modify] https://crrev.com/011b8046a07ac2cedde530a05f81e8fb41f80c62/third_party/WebKit/Source/core/dom/ElementData.h
[modify] https://crrev.com/011b8046a07ac2cedde530a05f81e8fb41f80c62/third_party/WebKit/Source/core/dom/IntersectionObserver.cpp
[modify] https://crrev.com/011b8046a07ac2cedde530a05f81e8fb41f80c62/third_party/WebKit/Source/core/dom/Node.h
[modify] https://crrev.com/011b8046a07ac2cedde530a05f81e8fb41f80c62/third_party/WebKit/Source/core/dom/ScriptRunnerTest.cpp
[modify] https://crrev.com/011b8046a07ac2cedde530a05f81e8fb41f80c62/third_party/WebKit/Source/core/editing/markers/DocumentMarkerControllerTest.cpp
[modify] https://crrev.com/011b8046a07ac2cedde530a05f81e8fb41f80c62/third_party/WebKit/Source/core/frame/EventHandlerRegistry.cpp
[modify] https://crrev.com/011b8046a07ac2cedde530a05f81e8fb41f80c62/third_party/WebKit/Source/core/frame/ImageBitmapTest.cpp
[modify] https://crrev.com/011b8046a07ac2cedde530a05f81e8fb41f80c62/third_party/WebKit/Source/core/html/AutoplayExperimentTest.cpp
[modify] https://crrev.com/011b8046a07ac2cedde530a05f81e8fb41f80c62/third_party/WebKit/Source/core/html/HTMLMediaElement.cpp
[modify] https://crrev.com/011b8046a07ac2cedde530a05f81e8fb41f80c62/third_party/WebKit/Source/core/svg/SVGElementRareData.cpp
[modify] https://crrev.com/011b8046a07ac2cedde530a05f81e8fb41f80c62/third_party/WebKit/Source/core/svg/graphics/SVGImageChromeClient.cpp
[modify] https://crrev.com/011b8046a07ac2cedde530a05f81e8fb41f80c62/third_party/WebKit/Source/devtools/front_end/timeline/TimelineModel.js
[modify] https://crrev.com/011b8046a07ac2cedde530a05f81e8fb41f80c62/third_party/WebKit/Source/modules/canvas2d/CanvasRenderingContext2DTest.cpp
[modify] https://crrev.com/011b8046a07ac2cedde530a05f81e8fb41f80c62/third_party/WebKit/Source/modules/csspaint/PaintWorkletTest.cpp
[modify] https://crrev.com/011b8046a07ac2cedde530a05f81e8fb41f80c62/third_party/WebKit/Source/modules/fetch/BodyStreamBufferTest.cpp
[modify] https://crrev.com/011b8046a07ac2cedde530a05f81e8fb41f80c62/third_party/WebKit/Source/modules/fetch/DataConsumerTeeTest.cpp
[modify] https://crrev.com/011b8046a07ac2cedde530a05f81e8fb41f80c62/third_party/WebKit/Source/modules/fetch/FetchBlobDataConsumerHandleTest.cpp
[modify] https://crrev.com/011b8046a07ac2cedde530a05f81e8fb41f80c62/third_party/WebKit/Source/modules/indexeddb/IDBTransactionTest.cpp
[modify] https://crrev.com/011b8046a07ac2cedde530a05f81e8fb41f80c62/third_party/WebKit/Source/platform/LifecycleContextTest.cpp
[modify] https://crrev.com/011b8046a07ac2cedde530a05f81e8fb41f80c62/third_party/WebKit/Source/platform/Timer.h
[modify] https://crrev.com/011b8046a07ac2cedde530a05f81e8fb41f80c62/third_party/WebKit/Source/platform/exported/Platform.cpp
[modify] https://crrev.com/011b8046a07ac2cedde530a05f81e8fb41f80c62/third_party/WebKit/Source/platform/heap/BlinkGC.h
[modify] https://crrev.com/011b8046a07ac2cedde530a05f81e8fb41f80c62/third_party/WebKit/Source/platform/heap/BlinkGCMemoryDumpProvider.cpp
[modify] https://crrev.com/011b8046a07ac2cedde530a05f81e8fb41f80c62/third_party/WebKit/Source/platform/heap/GarbageCollected.h
[modify] https://crrev.com/011b8046a07ac2cedde530a05f81e8fb41f80c62/third_party/WebKit/Source/platform/heap/Handle.h
[modify] https://crrev.com/011b8046a07ac2cedde530a05f81e8fb41f80c62/third_party/WebKit/Source/platform/heap/Heap.cpp
[modify] https://crrev.com/011b8046a07ac2cedde530a05f81e8fb41f80c62/third_party/WebKit/Source/platform/heap/Heap.h
[modify] https://crrev.com/011b8046a07ac2cedde530a05f81e8fb41f80c62/third_party/WebKit/Source/platform/heap/HeapAllocator.h
[modify] https://crrev.com/011b8046a07ac2cedde530a05f81e8fb41f80c62/third_party/WebKit/Source/platform/heap/HeapPage.cpp
[modify] https://crrev.com/011b8046a07ac2cedde530a05f81e8fb41f80c62/third_party/WebKit/Source/platform/heap/HeapPage.h
[modify] https://crrev.com/011b8046a07ac2cedde530a05f81e8fb41f80c62/third_party/WebKit/Source/platform/heap/HeapTerminatedArray.h
[modify] https://crrev.com/011b8046a07ac2cedde530a05f81e8fb41f80c62/third_party/WebKit/Source/platform/heap/HeapTest.cpp
[modify] https://crrev.com/011b8046a07ac2cedde530a05f81e8fb41f80c62/third_party/WebKit/Source/platform/heap/MarkingVisitorImpl.h
[modify] https://crrev.com/011b8046a07ac2cedde530a05f81e8fb41f80c62/third_party/WebKit/Source/platform/heap/PagePool.cpp
[modify] https://crrev.com/011b8046a07ac2cedde530a05f81e8fb41f80c62/third_party/WebKit/Source/platform/heap/RunAllTests.cpp
[modify] https://crrev.com/011b8046a07ac2cedde530a05f81e8fb41f80c62/third_party/WebKit/Source/platform/heap/ThreadState.cpp
[modify] https://crrev.com/011b8046a07ac2cedde530a05f81e8fb41f80c62/third_party/WebKit/Source/platform/heap/ThreadState.h
[modify] https://crrev.com/011b8046a07ac2cedde530a05f81e8fb41f80c62/third_party/WebKit/Source/platform/scroll/ScrollAnimatorTest.cpp
[modify] https://crrev.com/011b8046a07ac2cedde530a05f81e8fb41f80c62/third_party/WebKit/Source/platform/scroll/ScrollableAreaTest.cpp
[modify] https://crrev.com/011b8046a07ac2cedde530a05f81e8fb41f80c62/third_party/WebKit/Source/platform/testing/RunAllTests.cpp
[modify] https://crrev.com/011b8046a07ac2cedde530a05f81e8fb41f80c62/third_party/WebKit/Source/web/WebFrame.cpp
[modify] https://crrev.com/011b8046a07ac2cedde530a05f81e8fb41f80c62/third_party/WebKit/Source/web/WebHeap.cpp
[modify] https://crrev.com/011b8046a07ac2cedde530a05f81e8fb41f80c62/third_party/WebKit/Source/web/tests/WebViewTest.cpp
[modify] https://crrev.com/011b8046a07ac2cedde530a05f81e8fb41f80c62/tools/perf/measurements/oilpan_gc_times.py
[modify] https://crrev.com/011b8046a07ac2cedde530a05f81e8fb41f80c62/tools/perf/measurements/oilpan_gc_times_unittest.py

Project Member

Comment 12 by bugdroid1@chromium.org, Apr 22 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/b5cb70c9e931fad94b21fc6d8c9356b7e3860b6b

commit b5cb70c9e931fad94b21fc6d8c9356b7e3860b6b
Author: keishi <keishi@chromium.org>
Date: Fri Apr 22 07:57:45 2016

Prepare for multiple ThreadHeaps

Convert static methods and members on ThreadHeap to non-static

BUG= 591606 

Review URL: https://codereview.chromium.org/1892713003

Cr-Commit-Position: refs/heads/master@{#389052}

[modify] https://crrev.com/b5cb70c9e931fad94b21fc6d8c9356b7e3860b6b/third_party/WebKit/Source/bindings/core/v8/WrapperTypeInfo.h
[modify] https://crrev.com/b5cb70c9e931fad94b21fc6d8c9356b7e3860b6b/third_party/WebKit/Source/platform/WebThreadSupportingGC.cpp
[modify] https://crrev.com/b5cb70c9e931fad94b21fc6d8c9356b7e3860b6b/third_party/WebKit/Source/platform/exported/Platform.cpp
[modify] https://crrev.com/b5cb70c9e931fad94b21fc6d8c9356b7e3860b6b/third_party/WebKit/Source/platform/heap/Heap.cpp
[modify] https://crrev.com/b5cb70c9e931fad94b21fc6d8c9356b7e3860b6b/third_party/WebKit/Source/platform/heap/Heap.h
[modify] https://crrev.com/b5cb70c9e931fad94b21fc6d8c9356b7e3860b6b/third_party/WebKit/Source/platform/heap/HeapPage.cpp
[modify] https://crrev.com/b5cb70c9e931fad94b21fc6d8c9356b7e3860b6b/third_party/WebKit/Source/platform/heap/HeapTest.cpp
[modify] https://crrev.com/b5cb70c9e931fad94b21fc6d8c9356b7e3860b6b/third_party/WebKit/Source/platform/heap/MarkingVisitorImpl.h
[modify] https://crrev.com/b5cb70c9e931fad94b21fc6d8c9356b7e3860b6b/third_party/WebKit/Source/platform/heap/PagePool.cpp
[modify] https://crrev.com/b5cb70c9e931fad94b21fc6d8c9356b7e3860b6b/third_party/WebKit/Source/platform/heap/SafePoint.cpp
[modify] https://crrev.com/b5cb70c9e931fad94b21fc6d8c9356b7e3860b6b/third_party/WebKit/Source/platform/heap/ThreadState.cpp
[modify] https://crrev.com/b5cb70c9e931fad94b21fc6d8c9356b7e3860b6b/third_party/WebKit/Source/platform/heap/ThreadState.h
[modify] https://crrev.com/b5cb70c9e931fad94b21fc6d8c9356b7e3860b6b/third_party/WebKit/Source/platform/testing/RunAllTests.cpp

Project Member

Comment 13 by bugdroid1@chromium.org, Apr 25 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/8cc8d1cd549bcb95653e501f6f2a4b77170d1d1a

commit 8cc8d1cd549bcb95653e501f6f2a4b77170d1d1a
Author: keishi <keishi@chromium.org>
Date: Mon Apr 25 08:46:09 2016

CrossThreadPersistentRegion will only trace objects belonging to the current GCing ThreadHeap

CrossThreadPersistentRegion will be globally shared so we need to limit the tracing to objects belonging to the current GCing ThreadHeap.

BUG= 591606 

Review URL: https://codereview.chromium.org/1904003002

Cr-Commit-Position: refs/heads/master@{#389433}

[modify] https://crrev.com/8cc8d1cd549bcb95653e501f6f2a4b77170d1d1a/third_party/WebKit/Source/platform/heap/PersistentNode.cpp
[modify] https://crrev.com/8cc8d1cd549bcb95653e501f6f2a4b77170d1d1a/third_party/WebKit/Source/platform/heap/PersistentNode.h
[modify] https://crrev.com/8cc8d1cd549bcb95653e501f6f2a4b77170d1d1a/third_party/WebKit/Source/platform/heap/Visitor.h

Project Member

Comment 15 by bugdroid1@chromium.org, Apr 25 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/537b169259bf719f6c31fd2085b69f0180bb4dc2

commit 537b169259bf719f6c31fd2085b69f0180bb4dc2
Author: keishi <keishi@chromium.org>
Date: Mon Apr 25 12:11:00 2016

Revert of Add enablePerThreadHeap flag to ThreadState (patchset #3 id:40001 of https://codereview.chromium.org/1910753002/ )

Reason for revert:
Reverting in order to revert r 389052

Original issue's description:
> Add enablePerThreadHeap flag to ThreadState
>
> BUG= 591606 
>
> Committed: https://crrev.com/e5b8b2832fccc2cbe039698235669b029778f58d
> Cr-Commit-Position: refs/heads/master@{#389436}

TBR=oilpan-reviews@chromium.org,haraken@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG= 591606 

Review URL: https://codereview.chromium.org/1919793002

Cr-Commit-Position: refs/heads/master@{#389451}

[modify] https://crrev.com/537b169259bf719f6c31fd2085b69f0180bb4dc2/third_party/WebKit/Source/platform/WebThreadSupportingGC.cpp
[modify] https://crrev.com/537b169259bf719f6c31fd2085b69f0180bb4dc2/third_party/WebKit/Source/platform/WebThreadSupportingGC.h
[modify] https://crrev.com/537b169259bf719f6c31fd2085b69f0180bb4dc2/third_party/WebKit/Source/platform/heap/HeapTest.cpp
[modify] https://crrev.com/537b169259bf719f6c31fd2085b69f0180bb4dc2/third_party/WebKit/Source/platform/heap/ThreadState.cpp
[modify] https://crrev.com/537b169259bf719f6c31fd2085b69f0180bb4dc2/third_party/WebKit/Source/platform/heap/ThreadState.h

Project Member

Comment 16 by bugdroid1@chromium.org, Apr 25 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/5fbae6f24d9399952ed8f056b35d3a78ee7d6e75

commit 5fbae6f24d9399952ed8f056b35d3a78ee7d6e75
Author: keishi <keishi@chromium.org>
Date: Mon Apr 25 14:06:16 2016

Revert of CrossThreadPersistentRegion will only trace objects belonging to the current GCing ThreadHeap (patchset #2 id:20001 of https://codereview.chromium.org/1904003002/ )

Reason for revert:
Reverting in order to revert r 389052

Original issue's description:
> CrossThreadPersistentRegion will only trace objects belonging to the current GCing ThreadHeap
>
> CrossThreadPersistentRegion will be globally shared so we need to limit the tracing to objects belonging to the current GCing ThreadHeap.
>
> BUG= 591606 
>
> Committed: https://crrev.com/8cc8d1cd549bcb95653e501f6f2a4b77170d1d1a
> Cr-Commit-Position: refs/heads/master@{#389433}

TBR=oilpan-reviews@chromium.org,haraken@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG= 591606 

Review URL: https://codereview.chromium.org/1922493002

Cr-Commit-Position: refs/heads/master@{#389469}

[modify] https://crrev.com/5fbae6f24d9399952ed8f056b35d3a78ee7d6e75/third_party/WebKit/Source/platform/heap/PersistentNode.cpp
[modify] https://crrev.com/5fbae6f24d9399952ed8f056b35d3a78ee7d6e75/third_party/WebKit/Source/platform/heap/PersistentNode.h
[modify] https://crrev.com/5fbae6f24d9399952ed8f056b35d3a78ee7d6e75/third_party/WebKit/Source/platform/heap/Visitor.h

Project Member

Comment 17 by bugdroid1@chromium.org, Apr 25 2016

Labels: merge-merged-2716
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/8cc8d1cd549bcb95653e501f6f2a4b77170d1d1a

commit 8cc8d1cd549bcb95653e501f6f2a4b77170d1d1a
Author: keishi <keishi@chromium.org>
Date: Mon Apr 25 08:46:09 2016

CrossThreadPersistentRegion will only trace objects belonging to the current GCing ThreadHeap

CrossThreadPersistentRegion will be globally shared so we need to limit the tracing to objects belonging to the current GCing ThreadHeap.

BUG= 591606 

Review URL: https://codereview.chromium.org/1904003002

Cr-Commit-Position: refs/heads/master@{#389433}

[modify] https://crrev.com/8cc8d1cd549bcb95653e501f6f2a4b77170d1d1a/third_party/WebKit/Source/platform/heap/PersistentNode.cpp
[modify] https://crrev.com/8cc8d1cd549bcb95653e501f6f2a4b77170d1d1a/third_party/WebKit/Source/platform/heap/PersistentNode.h
[modify] https://crrev.com/8cc8d1cd549bcb95653e501f6f2a4b77170d1d1a/third_party/WebKit/Source/platform/heap/Visitor.h

Project Member

Comment 19 by bugdroid1@chromium.org, Apr 25 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/537b169259bf719f6c31fd2085b69f0180bb4dc2

commit 537b169259bf719f6c31fd2085b69f0180bb4dc2
Author: keishi <keishi@chromium.org>
Date: Mon Apr 25 12:11:00 2016

Revert of Add enablePerThreadHeap flag to ThreadState (patchset #3 id:40001 of https://codereview.chromium.org/1910753002/ )

Reason for revert:
Reverting in order to revert r 389052

Original issue's description:
> Add enablePerThreadHeap flag to ThreadState
>
> BUG= 591606 
>
> Committed: https://crrev.com/e5b8b2832fccc2cbe039698235669b029778f58d
> Cr-Commit-Position: refs/heads/master@{#389436}

TBR=oilpan-reviews@chromium.org,haraken@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG= 591606 

Review URL: https://codereview.chromium.org/1919793002

Cr-Commit-Position: refs/heads/master@{#389451}

[modify] https://crrev.com/537b169259bf719f6c31fd2085b69f0180bb4dc2/third_party/WebKit/Source/platform/WebThreadSupportingGC.cpp
[modify] https://crrev.com/537b169259bf719f6c31fd2085b69f0180bb4dc2/third_party/WebKit/Source/platform/WebThreadSupportingGC.h
[modify] https://crrev.com/537b169259bf719f6c31fd2085b69f0180bb4dc2/third_party/WebKit/Source/platform/heap/HeapTest.cpp
[modify] https://crrev.com/537b169259bf719f6c31fd2085b69f0180bb4dc2/third_party/WebKit/Source/platform/heap/ThreadState.cpp
[modify] https://crrev.com/537b169259bf719f6c31fd2085b69f0180bb4dc2/third_party/WebKit/Source/platform/heap/ThreadState.h

Project Member

Comment 20 by bugdroid1@chromium.org, Apr 25 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/5fbae6f24d9399952ed8f056b35d3a78ee7d6e75

commit 5fbae6f24d9399952ed8f056b35d3a78ee7d6e75
Author: keishi <keishi@chromium.org>
Date: Mon Apr 25 14:06:16 2016

Revert of CrossThreadPersistentRegion will only trace objects belonging to the current GCing ThreadHeap (patchset #2 id:20001 of https://codereview.chromium.org/1904003002/ )

Reason for revert:
Reverting in order to revert r 389052

Original issue's description:
> CrossThreadPersistentRegion will only trace objects belonging to the current GCing ThreadHeap
>
> CrossThreadPersistentRegion will be globally shared so we need to limit the tracing to objects belonging to the current GCing ThreadHeap.
>
> BUG= 591606 
>
> Committed: https://crrev.com/8cc8d1cd549bcb95653e501f6f2a4b77170d1d1a
> Cr-Commit-Position: refs/heads/master@{#389433}

TBR=oilpan-reviews@chromium.org,haraken@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG= 591606 

Review URL: https://codereview.chromium.org/1922493002

Cr-Commit-Position: refs/heads/master@{#389469}

[modify] https://crrev.com/5fbae6f24d9399952ed8f056b35d3a78ee7d6e75/third_party/WebKit/Source/platform/heap/PersistentNode.cpp
[modify] https://crrev.com/5fbae6f24d9399952ed8f056b35d3a78ee7d6e75/third_party/WebKit/Source/platform/heap/PersistentNode.h
[modify] https://crrev.com/5fbae6f24d9399952ed8f056b35d3a78ee7d6e75/third_party/WebKit/Source/platform/heap/Visitor.h

Project Member

Comment 21 by bugdroid1@chromium.org, Apr 26 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/835c8e2a01956eb74ab7c66a2612fcc27e7fed7a

commit 835c8e2a01956eb74ab7c66a2612fcc27e7fed7a
Author: keishi <keishi@chromium.org>
Date: Tue Apr 26 03:42:08 2016

Revert of Prepare for multiple ThreadHeaps (patchset #8 id:140001 of https://codereview.chromium.org/1892713003/ )

Reason for revert:
Failed TSan bots

Original issue's description:
> Prepare for multiple ThreadHeaps
>
> Convert static methods and members on ThreadHeap to non-static
>
> BUG= 591606 
>
> Committed: https://crrev.com/b5cb70c9e931fad94b21fc6d8c9356b7e3860b6b
> Cr-Commit-Position: refs/heads/master@{#389052}

TBR=oilpan-reviews@chromium.org,haraken@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG= 591606 

Review URL: https://codereview.chromium.org/1919773002

Cr-Commit-Position: refs/heads/master@{#389685}

[modify] https://crrev.com/835c8e2a01956eb74ab7c66a2612fcc27e7fed7a/third_party/WebKit/Source/bindings/core/v8/WrapperTypeInfo.h
[modify] https://crrev.com/835c8e2a01956eb74ab7c66a2612fcc27e7fed7a/third_party/WebKit/Source/platform/WebThreadSupportingGC.cpp
[modify] https://crrev.com/835c8e2a01956eb74ab7c66a2612fcc27e7fed7a/third_party/WebKit/Source/platform/exported/Platform.cpp
[modify] https://crrev.com/835c8e2a01956eb74ab7c66a2612fcc27e7fed7a/third_party/WebKit/Source/platform/heap/Heap.cpp
[modify] https://crrev.com/835c8e2a01956eb74ab7c66a2612fcc27e7fed7a/third_party/WebKit/Source/platform/heap/Heap.h
[modify] https://crrev.com/835c8e2a01956eb74ab7c66a2612fcc27e7fed7a/third_party/WebKit/Source/platform/heap/HeapPage.cpp
[modify] https://crrev.com/835c8e2a01956eb74ab7c66a2612fcc27e7fed7a/third_party/WebKit/Source/platform/heap/HeapTest.cpp
[modify] https://crrev.com/835c8e2a01956eb74ab7c66a2612fcc27e7fed7a/third_party/WebKit/Source/platform/heap/MarkingVisitorImpl.h
[modify] https://crrev.com/835c8e2a01956eb74ab7c66a2612fcc27e7fed7a/third_party/WebKit/Source/platform/heap/PagePool.cpp
[modify] https://crrev.com/835c8e2a01956eb74ab7c66a2612fcc27e7fed7a/third_party/WebKit/Source/platform/heap/SafePoint.cpp
[modify] https://crrev.com/835c8e2a01956eb74ab7c66a2612fcc27e7fed7a/third_party/WebKit/Source/platform/heap/ThreadState.cpp
[modify] https://crrev.com/835c8e2a01956eb74ab7c66a2612fcc27e7fed7a/third_party/WebKit/Source/platform/heap/ThreadState.h
[modify] https://crrev.com/835c8e2a01956eb74ab7c66a2612fcc27e7fed7a/third_party/WebKit/Source/platform/testing/RunAllTests.cpp

Project Member

Comment 22 by bugdroid1@chromium.org, Apr 27 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/75da88a74e60c92b4456b06f254aa6919b4dee01

commit 75da88a74e60c92b4456b06f254aa6919b4dee01
Author: keishi <keishi@chromium.org>
Date: Wed Apr 27 12:58:58 2016

Prepare for multiple ThreadHeaps

Convert static methods and members on ThreadHeap to non-static

BUG= 591606 

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

Review URL: https://codereview.chromium.org/1892713003

Cr-Commit-Position: refs/heads/master@{#390056}

[modify] https://crrev.com/75da88a74e60c92b4456b06f254aa6919b4dee01/third_party/WebKit/Source/bindings/core/v8/WrapperTypeInfo.h
[modify] https://crrev.com/75da88a74e60c92b4456b06f254aa6919b4dee01/third_party/WebKit/Source/platform/WebThreadSupportingGC.cpp
[modify] https://crrev.com/75da88a74e60c92b4456b06f254aa6919b4dee01/third_party/WebKit/Source/platform/exported/Platform.cpp
[modify] https://crrev.com/75da88a74e60c92b4456b06f254aa6919b4dee01/third_party/WebKit/Source/platform/heap/Heap.cpp
[modify] https://crrev.com/75da88a74e60c92b4456b06f254aa6919b4dee01/third_party/WebKit/Source/platform/heap/Heap.h
[modify] https://crrev.com/75da88a74e60c92b4456b06f254aa6919b4dee01/third_party/WebKit/Source/platform/heap/HeapPage.cpp
[modify] https://crrev.com/75da88a74e60c92b4456b06f254aa6919b4dee01/third_party/WebKit/Source/platform/heap/HeapTest.cpp
[modify] https://crrev.com/75da88a74e60c92b4456b06f254aa6919b4dee01/third_party/WebKit/Source/platform/heap/MarkingVisitorImpl.h
[modify] https://crrev.com/75da88a74e60c92b4456b06f254aa6919b4dee01/third_party/WebKit/Source/platform/heap/PagePool.cpp
[modify] https://crrev.com/75da88a74e60c92b4456b06f254aa6919b4dee01/third_party/WebKit/Source/platform/heap/SafePoint.cpp
[modify] https://crrev.com/75da88a74e60c92b4456b06f254aa6919b4dee01/third_party/WebKit/Source/platform/heap/ThreadState.cpp
[modify] https://crrev.com/75da88a74e60c92b4456b06f254aa6919b4dee01/third_party/WebKit/Source/platform/heap/ThreadState.h
[modify] https://crrev.com/75da88a74e60c92b4456b06f254aa6919b4dee01/third_party/WebKit/Source/platform/testing/RunAllTests.cpp
[modify] https://crrev.com/75da88a74e60c92b4456b06f254aa6919b4dee01/third_party/WebKit/Source/wtf/AddressSanitizer.h

Project Member

Comment 24 by bugdroid1@chromium.org, May 6 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/371965616636dbdaed9c91fe0369dcba50abe56b

commit 371965616636dbdaed9c91fe0369dcba50abe56b
Author: keishi <keishi@chromium.org>
Date: Fri May 06 07:37:31 2016

CrossThreadPersistentRegion will only trace objects belonging to the current GCing ThreadHeap

CrossThreadPersistentRegion will be globally shared so we need to limit the tracing to objects belonging to the current GCing ThreadHeap.

BUG= 591606 

Committed: https://crrev.com/8cc8d1cd549bcb95653e501f6f2a4b77170d1d1a
Cr-Commit-Position: refs/heads/master@{#389433}

Review-Url: https://codereview.chromium.org/1904003002
Cr-Commit-Position: refs/heads/master@{#392021}

[modify] https://crrev.com/371965616636dbdaed9c91fe0369dcba50abe56b/third_party/WebKit/Source/platform/heap/PersistentNode.cpp
[modify] https://crrev.com/371965616636dbdaed9c91fe0369dcba50abe56b/third_party/WebKit/Source/platform/heap/PersistentNode.h
[modify] https://crrev.com/371965616636dbdaed9c91fe0369dcba50abe56b/third_party/WebKit/Source/platform/heap/Visitor.h

Project Member

Comment 26 by bugdroid1@chromium.org, May 27 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/fcd2684d99a74875a8d6444015362692ee6b08fd

commit fcd2684d99a74875a8d6444015362692ee6b08fd
Author: keishi <keishi@chromium.org>
Date: Fri May 27 04:32:29 2016

Add checks for per thread heap

A GC should only mark the objects that belong in its heap.
visitStack should only mark the objects that belong in its heap.
isHeapObjectAlive will only work if the object belongs to the current GCing thread's heap.

BUG= 591606 

Review-Url: https://codereview.chromium.org/2012763002
Cr-Commit-Position: refs/heads/master@{#396379}

[modify] https://crrev.com/fcd2684d99a74875a8d6444015362692ee6b08fd/third_party/WebKit/Source/platform/heap/Heap.cpp
[modify] https://crrev.com/fcd2684d99a74875a8d6444015362692ee6b08fd/third_party/WebKit/Source/platform/heap/Heap.h
[modify] https://crrev.com/fcd2684d99a74875a8d6444015362692ee6b08fd/third_party/WebKit/Source/platform/heap/MarkingVisitorImpl.h
[modify] https://crrev.com/fcd2684d99a74875a8d6444015362692ee6b08fd/third_party/WebKit/Source/platform/heap/ThreadState.cpp

Project Member

Comment 28 by bugdroid1@chromium.org, May 31 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/687a70a8ab0b0cfdd17f8d57d639d9b0ad7cd1c9

commit 687a70a8ab0b0cfdd17f8d57d639d9b0ad7cd1c9
Author: keishi <keishi@chromium.org>
Date: Tue May 31 02:57:33 2016

Revert of Allow CrossThreadPersistent in collections (patchset #7 id:120001 of https://codereview.chromium.org/2007283002/ )

Reason for revert:
blink_perf.bindings regression on android.webview
crbug.com/615832

Original issue's description:
> Allow CrossThreadPersistent in collections
>
> BUG= 591606 
>
> Committed: https://crrev.com/5adc4454691bffcbfdc7cd60866c145a4e67de0d
> Cr-Commit-Position: refs/heads/master@{#396410}

TBR=oilpan-reviews@chromium.org,haraken@chromium.org,sigbjornf@opera.com
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG= 591606 

Review-Url: https://codereview.chromium.org/2028433003
Cr-Commit-Position: refs/heads/master@{#396772}

[modify] https://crrev.com/687a70a8ab0b0cfdd17f8d57d639d9b0ad7cd1c9/third_party/WebKit/Source/platform/heap/HeapAllocator.h
[modify] https://crrev.com/687a70a8ab0b0cfdd17f8d57d639d9b0ad7cd1c9/third_party/WebKit/Source/platform/heap/HeapTest.cpp
[modify] https://crrev.com/687a70a8ab0b0cfdd17f8d57d639d9b0ad7cd1c9/third_party/WebKit/Source/platform/heap/Persistent.h
[modify] https://crrev.com/687a70a8ab0b0cfdd17f8d57d639d9b0ad7cd1c9/third_party/WebKit/Source/platform/heap/PersistentNode.cpp

Project Member

Comment 30 by bugdroid1@chromium.org, Jun 2 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/993e3a74102e0843bebb2f52de1a904f151843d4

commit 993e3a74102e0843bebb2f52de1a904f151843d4
Author: keishi <keishi@chromium.org>
Date: Thu Jun 02 13:34:50 2016

Enable per thread heap for database thread

BUG= 591606 

Review-Url: https://codereview.chromium.org/1909813002
Cr-Commit-Position: refs/heads/master@{#397383}

[modify] https://crrev.com/993e3a74102e0843bebb2f52de1a904f151843d4/third_party/WebKit/Source/modules/webdatabase/Database.cpp
[modify] https://crrev.com/993e3a74102e0843bebb2f52de1a904f151843d4/third_party/WebKit/Source/modules/webdatabase/DatabaseAuthorizer.cpp
[modify] https://crrev.com/993e3a74102e0843bebb2f52de1a904f151843d4/third_party/WebKit/Source/modules/webdatabase/DatabaseContext.cpp
[modify] https://crrev.com/993e3a74102e0843bebb2f52de1a904f151843d4/third_party/WebKit/Source/modules/webdatabase/DatabaseManager.cpp
[modify] https://crrev.com/993e3a74102e0843bebb2f52de1a904f151843d4/third_party/WebKit/Source/modules/webdatabase/DatabaseThread.cpp
[modify] https://crrev.com/993e3a74102e0843bebb2f52de1a904f151843d4/third_party/WebKit/Source/modules/webdatabase/DatabaseThread.h
[modify] https://crrev.com/993e3a74102e0843bebb2f52de1a904f151843d4/third_party/WebKit/Source/modules/webdatabase/SQLResultSet.cpp
[modify] https://crrev.com/993e3a74102e0843bebb2f52de1a904f151843d4/third_party/WebKit/Source/modules/webdatabase/SQLStatement.cpp
[modify] https://crrev.com/993e3a74102e0843bebb2f52de1a904f151843d4/third_party/WebKit/Source/modules/webdatabase/SQLStatementBackend.cpp
[modify] https://crrev.com/993e3a74102e0843bebb2f52de1a904f151843d4/third_party/WebKit/Source/modules/webdatabase/SQLTransaction.cpp
[modify] https://crrev.com/993e3a74102e0843bebb2f52de1a904f151843d4/third_party/WebKit/Source/modules/webdatabase/SQLTransactionBackend.cpp
[modify] https://crrev.com/993e3a74102e0843bebb2f52de1a904f151843d4/third_party/WebKit/Source/modules/webdatabase/SQLTransactionCoordinator.cpp
[modify] https://crrev.com/993e3a74102e0843bebb2f52de1a904f151843d4/third_party/WebKit/Source/modules/webdatabase/SQLTransactionCoordinator.h
[modify] https://crrev.com/993e3a74102e0843bebb2f52de1a904f151843d4/third_party/WebKit/Source/platform/heap/Heap.h

Project Member

Comment 31 by bugdroid1@chromium.org, Jun 4 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/9f495748a0415dfc1ce1899dbcb207fa9982f327

commit 9f495748a0415dfc1ce1899dbcb207fa9982f327
Author: haraken <haraken@chromium.org>
Date: Sat Jun 04 01:37:52 2016

Revert of Enable per thread heap for database thread (patchset #12 id:220001 of https://codereview.chromium.org/1909813002/ )

Reason for revert:
This CL is causing a top #1 crasher.

https://bugs.chromium.org/p/chromium/issues/detail?id=617141

Original issue's description:
> Enable per thread heap for database thread
>
> BUG= 591606 
>
> Committed: https://crrev.com/993e3a74102e0843bebb2f52de1a904f151843d4
> Cr-Commit-Position: refs/heads/master@{#397383}

TBR=oilpan-reviews@chromium.org,keishi@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG= 591606 

Review-Url: https://codereview.chromium.org/2037233002
Cr-Commit-Position: refs/heads/master@{#397884}

[modify] https://crrev.com/9f495748a0415dfc1ce1899dbcb207fa9982f327/third_party/WebKit/Source/modules/webdatabase/Database.cpp
[modify] https://crrev.com/9f495748a0415dfc1ce1899dbcb207fa9982f327/third_party/WebKit/Source/modules/webdatabase/DatabaseAuthorizer.cpp
[modify] https://crrev.com/9f495748a0415dfc1ce1899dbcb207fa9982f327/third_party/WebKit/Source/modules/webdatabase/DatabaseContext.cpp
[modify] https://crrev.com/9f495748a0415dfc1ce1899dbcb207fa9982f327/third_party/WebKit/Source/modules/webdatabase/DatabaseManager.cpp
[modify] https://crrev.com/9f495748a0415dfc1ce1899dbcb207fa9982f327/third_party/WebKit/Source/modules/webdatabase/DatabaseThread.cpp
[modify] https://crrev.com/9f495748a0415dfc1ce1899dbcb207fa9982f327/third_party/WebKit/Source/modules/webdatabase/DatabaseThread.h
[modify] https://crrev.com/9f495748a0415dfc1ce1899dbcb207fa9982f327/third_party/WebKit/Source/modules/webdatabase/SQLResultSet.cpp
[modify] https://crrev.com/9f495748a0415dfc1ce1899dbcb207fa9982f327/third_party/WebKit/Source/modules/webdatabase/SQLStatement.cpp
[modify] https://crrev.com/9f495748a0415dfc1ce1899dbcb207fa9982f327/third_party/WebKit/Source/modules/webdatabase/SQLStatementBackend.cpp
[modify] https://crrev.com/9f495748a0415dfc1ce1899dbcb207fa9982f327/third_party/WebKit/Source/modules/webdatabase/SQLTransaction.cpp
[modify] https://crrev.com/9f495748a0415dfc1ce1899dbcb207fa9982f327/third_party/WebKit/Source/modules/webdatabase/SQLTransactionBackend.cpp
[modify] https://crrev.com/9f495748a0415dfc1ce1899dbcb207fa9982f327/third_party/WebKit/Source/modules/webdatabase/SQLTransactionCoordinator.cpp
[modify] https://crrev.com/9f495748a0415dfc1ce1899dbcb207fa9982f327/third_party/WebKit/Source/modules/webdatabase/SQLTransactionCoordinator.h
[modify] https://crrev.com/9f495748a0415dfc1ce1899dbcb207fa9982f327/third_party/WebKit/Source/platform/heap/Heap.h

Project Member

Comment 32 by bugdroid1@chromium.org, Jun 8 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/20a19e64ad3ca5728e557762b7040731b93400c4

commit 20a19e64ad3ca5728e557762b7040731b93400c4
Author: keishi <keishi@chromium.org>
Date: Wed Jun 08 15:15:11 2016

Enable per thread heap for database thread

BUG= 591606 

Committed: https://crrev.com/993e3a74102e0843bebb2f52de1a904f151843d4
Review-Url: https://codereview.chromium.org/1909813002
Cr-Original-Commit-Position: refs/heads/master@{#397383}
Cr-Commit-Position: refs/heads/master@{#398558}

[modify] https://crrev.com/20a19e64ad3ca5728e557762b7040731b93400c4/third_party/WebKit/Source/modules/webdatabase/Database.cpp
[modify] https://crrev.com/20a19e64ad3ca5728e557762b7040731b93400c4/third_party/WebKit/Source/modules/webdatabase/Database.h
[modify] https://crrev.com/20a19e64ad3ca5728e557762b7040731b93400c4/third_party/WebKit/Source/modules/webdatabase/DatabaseAuthorizer.cpp
[modify] https://crrev.com/20a19e64ad3ca5728e557762b7040731b93400c4/third_party/WebKit/Source/modules/webdatabase/DatabaseContext.cpp
[modify] https://crrev.com/20a19e64ad3ca5728e557762b7040731b93400c4/third_party/WebKit/Source/modules/webdatabase/DatabaseManager.cpp
[modify] https://crrev.com/20a19e64ad3ca5728e557762b7040731b93400c4/third_party/WebKit/Source/modules/webdatabase/DatabaseThread.cpp
[modify] https://crrev.com/20a19e64ad3ca5728e557762b7040731b93400c4/third_party/WebKit/Source/modules/webdatabase/DatabaseThread.h
[modify] https://crrev.com/20a19e64ad3ca5728e557762b7040731b93400c4/third_party/WebKit/Source/modules/webdatabase/DatabaseTracker.h
[modify] https://crrev.com/20a19e64ad3ca5728e557762b7040731b93400c4/third_party/WebKit/Source/modules/webdatabase/SQLResultSet.cpp
[modify] https://crrev.com/20a19e64ad3ca5728e557762b7040731b93400c4/third_party/WebKit/Source/modules/webdatabase/SQLStatement.cpp
[modify] https://crrev.com/20a19e64ad3ca5728e557762b7040731b93400c4/third_party/WebKit/Source/modules/webdatabase/SQLStatementBackend.cpp
[modify] https://crrev.com/20a19e64ad3ca5728e557762b7040731b93400c4/third_party/WebKit/Source/modules/webdatabase/SQLTransaction.cpp
[modify] https://crrev.com/20a19e64ad3ca5728e557762b7040731b93400c4/third_party/WebKit/Source/modules/webdatabase/SQLTransactionBackend.cpp
[modify] https://crrev.com/20a19e64ad3ca5728e557762b7040731b93400c4/third_party/WebKit/Source/modules/webdatabase/SQLTransactionBackend.h
[modify] https://crrev.com/20a19e64ad3ca5728e557762b7040731b93400c4/third_party/WebKit/Source/modules/webdatabase/SQLTransactionCoordinator.cpp
[modify] https://crrev.com/20a19e64ad3ca5728e557762b7040731b93400c4/third_party/WebKit/Source/modules/webdatabase/SQLTransactionCoordinator.h
[modify] https://crrev.com/20a19e64ad3ca5728e557762b7040731b93400c4/third_party/WebKit/Source/modules/webdatabase/sqlite/SQLiteDatabase.cpp
[modify] https://crrev.com/20a19e64ad3ca5728e557762b7040731b93400c4/third_party/WebKit/Source/modules/webdatabase/sqlite/SQLiteDatabase.h
[modify] https://crrev.com/20a19e64ad3ca5728e557762b7040731b93400c4/third_party/WebKit/Source/platform/heap/Heap.h

Project Member

Comment 33 by bugdroid1@chromium.org, Aug 17 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/90126696891433f36c73c61905f238ced7777500

commit 90126696891433f36c73c61905f238ced7777500
Author: keishi <keishi@chromium.org>
Date: Wed Aug 17 11:51:04 2016

Check that heap collection backing reallocation happens on the same heap

BUG= 591606 

Review-Url: https://codereview.chromium.org/2210483002
Cr-Commit-Position: refs/heads/master@{#412500}

[modify] https://crrev.com/90126696891433f36c73c61905f238ced7777500/third_party/WebKit/Source/platform/heap/HeapAllocator.cpp

Project Member

Comment 36 by bugdroid1@chromium.org, Aug 19 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/33f9e3a0fb535159cbd009663b9f194fc27f4205

commit 33f9e3a0fb535159cbd009663b9f194fc27f4205
Author: keishi <keishi@chromium.org>
Date: Fri Aug 19 13:30:20 2016

Check thread is attached when creating a Persistent

BUG= 591606 

Review-Url: https://codereview.chromium.org/2257333003
Cr-Commit-Position: refs/heads/master@{#413125}

[modify] https://crrev.com/33f9e3a0fb535159cbd009663b9f194fc27f4205/third_party/WebKit/Source/platform/heap/Persistent.h

Project Member

Comment 37 by bugdroid1@chromium.org, Sep 13 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/c59f742ca99b5695cab3c7d118d23d2c0a800d8c

commit c59f742ca99b5695cab3c7d118d23d2c0a800d8c
Author: keishi <keishi@chromium.org>
Date: Tue Sep 13 05:17:25 2016

Prepare CompositorWorker for per thread heap

Replace cross thread references with CrossThreadPersistents.

BUG= 591606 

Review-Url: https://codereview.chromium.org/2204183002
Cr-Commit-Position: refs/heads/master@{#418175}

[delete] https://crrev.com/b83d755a52a7fe9792af9ee360e551c157ad576e/third_party/WebKit/LayoutTests/virtual/threaded/fast/compositorworker/compositor-proxy-disconnect-worker-terminate.html
[delete] https://crrev.com/b83d755a52a7fe9792af9ee360e551c157ad576e/third_party/WebKit/LayoutTests/virtual/threaded/fast/compositorworker/resources/proxy-idle.js
[modify] https://crrev.com/c59f742ca99b5695cab3c7d118d23d2c0a800d8c/third_party/WebKit/Source/core/dom/CompositorProxy.h
[modify] https://crrev.com/c59f742ca99b5695cab3c7d118d23d2c0a800d8c/third_party/WebKit/Source/core/dom/CompositorProxyClient.h
[modify] https://crrev.com/c59f742ca99b5695cab3c7d118d23d2c0a800d8c/third_party/WebKit/Source/core/workers/InProcessWorkerObjectProxy.h
[modify] https://crrev.com/c59f742ca99b5695cab3c7d118d23d2c0a800d8c/third_party/WebKit/Source/core/workers/WorkerGlobalScope.cpp
[modify] https://crrev.com/c59f742ca99b5695cab3c7d118d23d2c0a800d8c/third_party/WebKit/Source/core/workers/WorkerGlobalScope.h
[modify] https://crrev.com/c59f742ca99b5695cab3c7d118d23d2c0a800d8c/third_party/WebKit/Source/core/workers/WorkerThread.h
[modify] https://crrev.com/c59f742ca99b5695cab3c7d118d23d2c0a800d8c/third_party/WebKit/Source/modules/compositorworker/CompositorWorkerGlobalScope.cpp
[modify] https://crrev.com/c59f742ca99b5695cab3c7d118d23d2c0a800d8c/third_party/WebKit/Source/modules/compositorworker/CompositorWorkerGlobalScope.h
[modify] https://crrev.com/c59f742ca99b5695cab3c7d118d23d2c0a800d8c/third_party/WebKit/Source/modules/compositorworker/CompositorWorkerThreadTest.cpp
[modify] https://crrev.com/c59f742ca99b5695cab3c7d118d23d2c0a800d8c/third_party/WebKit/Source/web/CompositorMutatorImpl.cpp
[modify] https://crrev.com/c59f742ca99b5695cab3c7d118d23d2c0a800d8c/third_party/WebKit/Source/web/CompositorMutatorImpl.h
[modify] https://crrev.com/c59f742ca99b5695cab3c7d118d23d2c0a800d8c/third_party/WebKit/Source/web/CompositorProxyClientImpl.cpp
[modify] https://crrev.com/c59f742ca99b5695cab3c7d118d23d2c0a800d8c/third_party/WebKit/Source/web/CompositorProxyClientImpl.h

Project Member

Comment 38 by bugdroid1@chromium.org, Sep 14 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/56b559a7548bb3928df3ac0d57a57b84dc170826

commit 56b559a7548bb3928df3ac0d57a57b84dc170826
Author: keishi <keishi@chromium.org>
Date: Wed Sep 14 09:15:49 2016

Support CrossThreadWeakPersistent PostTask on per thread heap enabled thread

Implements Unwrap(CrossThreadWeakPersistent) that returns a CrossThreadPersistent so the object doesn't get collected during callback invocation.

BUG= 591606 

Review-Url: https://codereview.chromium.org/2207333002
Cr-Commit-Position: refs/heads/master@{#418516}

[modify] https://crrev.com/56b559a7548bb3928df3ac0d57a57b84dc170826/third_party/WebKit/Source/platform/heap/Persistent.h

Project Member

Comment 40 by bugdroid1@chromium.org, Sep 21 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/d49b99a5517d33b68ba4d0d8f281a1fe95738c27

commit d49b99a5517d33b68ba4d0d8f281a1fe95738c27
Author: keishi <keishi@chromium.org>
Date: Wed Sep 21 09:04:00 2016

Use CrossThreadPersistent for LocalFileSystem

Use CrossThreadPersistent for LocalFileSystem in case its used from a Worker.

Required to enable per thread heap on DedicatedWorker.

BUG= 591606 

Review-Url: https://codereview.chromium.org/2354903004
Cr-Commit-Position: refs/heads/master@{#420009}

[modify] https://crrev.com/d49b99a5517d33b68ba4d0d8f281a1fe95738c27/third_party/WebKit/Source/modules/filesystem/LocalFileSystem.cpp

Project Member

Comment 41 by bugdroid1@chromium.org, Sep 21 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/e2ca66fcac787dd082c15057225ac82bf7a3c0f7

commit e2ca66fcac787dd082c15057225ac82bf7a3c0f7
Author: keishi <keishi@chromium.org>
Date: Wed Sep 21 11:16:58 2016

Use enum for per thread heap enabled flag

Using enum instead of a bool for per thread heap enabled flag

BUG= 591606 

Review-Url: https://codereview.chromium.org/2355193002
Cr-Commit-Position: refs/heads/master@{#420026}

[modify] https://crrev.com/e2ca66fcac787dd082c15057225ac82bf7a3c0f7/third_party/WebKit/Source/core/html/parser/HTMLParserThread.cpp
[modify] https://crrev.com/e2ca66fcac787dd082c15057225ac82bf7a3c0f7/third_party/WebKit/Source/core/workers/WorkerBackingThread.cpp
[modify] https://crrev.com/e2ca66fcac787dd082c15057225ac82bf7a3c0f7/third_party/WebKit/Source/modules/fetch/DataConsumerHandleTestUtil.cpp
[modify] https://crrev.com/e2ca66fcac787dd082c15057225ac82bf7a3c0f7/third_party/WebKit/Source/modules/webdatabase/DatabaseThread.cpp
[modify] https://crrev.com/e2ca66fcac787dd082c15057225ac82bf7a3c0f7/third_party/WebKit/Source/platform/WebThreadSupportingGC.cpp
[modify] https://crrev.com/e2ca66fcac787dd082c15057225ac82bf7a3c0f7/third_party/WebKit/Source/platform/WebThreadSupportingGC.h
[modify] https://crrev.com/e2ca66fcac787dd082c15057225ac82bf7a3c0f7/third_party/WebKit/Source/platform/heap/BlinkGC.h
[modify] https://crrev.com/e2ca66fcac787dd082c15057225ac82bf7a3c0f7/third_party/WebKit/Source/platform/heap/Heap.cpp
[modify] https://crrev.com/e2ca66fcac787dd082c15057225ac82bf7a3c0f7/third_party/WebKit/Source/platform/heap/HeapTest.cpp
[modify] https://crrev.com/e2ca66fcac787dd082c15057225ac82bf7a3c0f7/third_party/WebKit/Source/platform/heap/ThreadState.cpp
[modify] https://crrev.com/e2ca66fcac787dd082c15057225ac82bf7a3c0f7/third_party/WebKit/Source/platform/heap/ThreadState.h

Project Member

Comment 42 by bugdroid1@chromium.org, Sep 27 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/5e3e05a33a5db4b1c7b836f172e0f9899c9965e3

commit 5e3e05a33a5db4b1c7b836f172e0f9899c9965e3
Author: keishi <keishi@chromium.org>
Date: Tue Sep 27 05:49:39 2016

Prepare DedicatedWorker for per thread heap

Adds flag to turn on per thread heap on DedicatedWorkers. Add tests so both per thread haep enable and disabled are tested.

BUG= 591606 

Review-Url: https://codereview.chromium.org/2335063003
Cr-Commit-Position: refs/heads/master@{#421118}

[modify] https://crrev.com/5e3e05a33a5db4b1c7b836f172e0f9899c9965e3/third_party/WebKit/Source/core/loader/ThreadableLoaderTest.cpp
[modify] https://crrev.com/5e3e05a33a5db4b1c7b836f172e0f9899c9965e3/third_party/WebKit/Source/core/workers/DedicatedWorkerTest.cpp
[modify] https://crrev.com/5e3e05a33a5db4b1c7b836f172e0f9899c9965e3/third_party/WebKit/Source/core/workers/DedicatedWorkerThread.cpp
[modify] https://crrev.com/5e3e05a33a5db4b1c7b836f172e0f9899c9965e3/third_party/WebKit/Source/core/workers/SharedWorkerThread.cpp
[modify] https://crrev.com/5e3e05a33a5db4b1c7b836f172e0f9899c9965e3/third_party/WebKit/Source/core/workers/WorkerBackingThread.cpp
[modify] https://crrev.com/5e3e05a33a5db4b1c7b836f172e0f9899c9965e3/third_party/WebKit/Source/core/workers/WorkerBackingThread.h
[modify] https://crrev.com/5e3e05a33a5db4b1c7b836f172e0f9899c9965e3/third_party/WebKit/Source/core/workers/WorkerThreadTest.cpp
[modify] https://crrev.com/5e3e05a33a5db4b1c7b836f172e0f9899c9965e3/third_party/WebKit/Source/core/workers/WorkerThreadTestHelper.h
[modify] https://crrev.com/5e3e05a33a5db4b1c7b836f172e0f9899c9965e3/third_party/WebKit/Source/modules/serviceworkers/ServiceWorkerThread.cpp

Project Member

Comment 44 by bugdroid1@chromium.org, Sep 29 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/097490296ebb78772f2b058e7f2b144894a4197e

commit 097490296ebb78772f2b058e7f2b144894a4197e
Author: keishi <keishi@chromium.org>
Date: Thu Sep 29 01:54:52 2016

Prepare for per thread heap on service worker thread.

ServiceWorkerGlobalScope is created in the worker thread and ServiceWorkerGlobalScopeProxy is created in the main thread so the handle should be a CrossThreadPersistent.

BUG= 591606 

Review-Url: https://codereview.chromium.org/2369013004
Cr-Commit-Position: refs/heads/master@{#421715}

[modify] https://crrev.com/097490296ebb78772f2b058e7f2b144894a4197e/third_party/WebKit/Source/web/ServiceWorkerGlobalScopeProxy.cpp
[modify] https://crrev.com/097490296ebb78772f2b058e7f2b144894a4197e/third_party/WebKit/Source/web/ServiceWorkerGlobalScopeProxy.h

Project Member

Comment 47 by bugdroid1@chromium.org, Oct 5 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/961a2b750b9c80ae8d6a56499a0d3a023212687a

commit 961a2b750b9c80ae8d6a56499a0d3a023212687a
Author: keishi <keishi@chromium.org>
Date: Wed Oct 05 08:26:45 2016

Revert of Enable per thread heap on DedicatedWorker (patchset #1 id:1 of https://codereview.chromium.org/2377583002/ )

Reason for revert:
Caused crashes
crbug.com/652058
 crbug.com/651702 
crbug.com/652001
crbug.com/652264
crbug.com/652183
 crbug.com/652556 

Original issue's description:
> Enable per thread heap on DedicatedWorker
>
> blink-dev PSA email: https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/wROZOp-YKSc
>
> BUG= 591606 
>
> Committed: https://crrev.com/d1aef67937aa7bcb5fb79a18fbf2759de21a3025
> Cr-Commit-Position: refs/heads/master@{#421456}

TBR=oilpan-reviews@chromium.org,haraken@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG= 591606 

Review-Url: https://codereview.chromium.org/2392233002
Cr-Commit-Position: refs/heads/master@{#423113}

[modify] https://crrev.com/961a2b750b9c80ae8d6a56499a0d3a023212687a/third_party/WebKit/Source/core/workers/DedicatedWorkerThread.cpp

Project Member

Comment 48 by bugdroid1@chromium.org, Oct 5 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/e01a95ada6400bcba59f521d336cfe2909c7b929

commit e01a95ada6400bcba59f521d336cfe2909c7b929
Author: keishi <keishi@chromium.org>
Date: Wed Oct 05 09:57:56 2016

Revert of Enable per thread heap on SharedWorker (patchset #1 id:1 of https://codereview.chromium.org/2374693002/ )

Reason for revert:
Caused crashes
crbug.com/652058
 crbug.com/651702 
crbug.com/652001
crbug.com/652264
crbug.com/652183
 crbug.com/652556 

Original issue's description:
> Enable per thread heap on SharedWorker
>
> blink-dev PSA email: https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/wROZOp-YKSc
>
> BUG= 591606 
>
> Committed: https://crrev.com/62d4e2e8f5ca66516854cbdbd1402d344410ec75
> Cr-Commit-Position: refs/heads/master@{#421731}

TBR=oilpan-reviews@chromium.org,haraken@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG= 591606 

Review-Url: https://codereview.chromium.org/2386263004
Cr-Commit-Position: refs/heads/master@{#423122}

[modify] https://crrev.com/e01a95ada6400bcba59f521d336cfe2909c7b929/third_party/WebKit/Source/core/workers/SharedWorkerThread.cpp

Project Member

Comment 49 by bugdroid1@chromium.org, Oct 5 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/e12bb732e68991ae704c93b631c3ff06def24fb1

commit e12bb732e68991ae704c93b631c3ff06def24fb1
Author: keishi <keishi@chromium.org>
Date: Wed Oct 05 10:07:57 2016

Revert of Enable per thread heap on ServiceWorker (patchset #1 id:1 of https://codereview.chromium.org/2386523002/ )

Reason for revert:
Caused crashes
crbug.com/652058
 crbug.com/651702 
crbug.com/652001
crbug.com/652264
crbug.com/652183
 crbug.com/652556 

Original issue's description:
> Enable per thread heap on ServiceWorker
>
> blink-dev PSA email: https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/wROZOp-YKSc
>
> BUG= 591606 
>
> Committed: https://crrev.com/5adb2db4a75dbaabace06728494946dd15630ebc
> Cr-Commit-Position: refs/heads/master@{#422056}

TBR=haraken@chromium.org,oilpan-reviews@chromium.org,nhiroki@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG= 591606 

Review-Url: https://codereview.chromium.org/2394763002
Cr-Commit-Position: refs/heads/master@{#423123}

[modify] https://crrev.com/e12bb732e68991ae704c93b631c3ff06def24fb1/third_party/WebKit/Source/modules/serviceworkers/ServiceWorkerThread.cpp

Project Member

Comment 50 by bugdroid1@chromium.org, Oct 27 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/e12bb732e68991ae704c93b631c3ff06def24fb1

commit e12bb732e68991ae704c93b631c3ff06def24fb1
Author: keishi <keishi@chromium.org>
Date: Wed Oct 05 10:07:57 2016

Revert of Enable per thread heap on ServiceWorker (patchset #1 id:1 of https://codereview.chromium.org/2386523002/ )

Reason for revert:
Caused crashes
crbug.com/652058
 crbug.com/651702 
crbug.com/652001
crbug.com/652264
crbug.com/652183
 crbug.com/652556 

Original issue's description:
> Enable per thread heap on ServiceWorker
>
> blink-dev PSA email: https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/wROZOp-YKSc
>
> BUG= 591606 
>
> Committed: https://crrev.com/5adb2db4a75dbaabace06728494946dd15630ebc
> Cr-Commit-Position: refs/heads/master@{#422056}

TBR=haraken@chromium.org,oilpan-reviews@chromium.org,nhiroki@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG= 591606 

Review-Url: https://codereview.chromium.org/2394763002
Cr-Commit-Position: refs/heads/master@{#423123}

[modify] https://crrev.com/e12bb732e68991ae704c93b631c3ff06def24fb1/third_party/WebKit/Source/modules/serviceworkers/ServiceWorkerThread.cpp

Project Member

Comment 51 by bugdroid1@chromium.org, Oct 27 2016

Labels: merge-merged-2840
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/e01a95ada6400bcba59f521d336cfe2909c7b929

commit e01a95ada6400bcba59f521d336cfe2909c7b929
Author: keishi <keishi@chromium.org>
Date: Wed Oct 05 09:57:56 2016

Revert of Enable per thread heap on SharedWorker (patchset #1 id:1 of https://codereview.chromium.org/2374693002/ )

Reason for revert:
Caused crashes
crbug.com/652058
 crbug.com/651702 
crbug.com/652001
crbug.com/652264
crbug.com/652183
 crbug.com/652556 

Original issue's description:
> Enable per thread heap on SharedWorker
>
> blink-dev PSA email: https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/wROZOp-YKSc
>
> BUG= 591606 
>
> Committed: https://crrev.com/62d4e2e8f5ca66516854cbdbd1402d344410ec75
> Cr-Commit-Position: refs/heads/master@{#421731}

TBR=oilpan-reviews@chromium.org,haraken@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG= 591606 

Review-Url: https://codereview.chromium.org/2386263004
Cr-Commit-Position: refs/heads/master@{#423122}

[modify] https://crrev.com/e01a95ada6400bcba59f521d336cfe2909c7b929/third_party/WebKit/Source/core/workers/SharedWorkerThread.cpp

Project Member

Comment 52 by bugdroid1@chromium.org, Dec 1 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/3e7fe3520659860b931521884673f5ecf68c21e0

commit 3e7fe3520659860b931521884673f5ecf68c21e0
Author: keishi <keishi@chromium.org>
Date: Thu Dec 01 08:03:37 2016

Enable per thread heap on parser thread

BUG=  591606 

Review-Url: https://codereview.chromium.org/2540143002
Cr-Commit-Position: refs/heads/master@{#435578}

[modify] https://crrev.com/3e7fe3520659860b931521884673f5ecf68c21e0/third_party/WebKit/Source/core/html/parser/HTMLParserThread.cpp

Blocking: 671856
Project Member

Comment 54 by bugdroid1@chromium.org, Jan 12 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/5c3ca4cbc1798a8d5551b6636252cf701b2051ad

commit 5c3ca4cbc1798a8d5551b6636252cf701b2051ad
Author: keishi <keishi@chromium.org>
Date: Thu Jan 12 09:29:29 2017

Remove ThreadHeapMode and make per thread heap mandatory

Removes ThreadHeapMode enum and makes per thread heap mandatory.

Remove HeapTests.{GCParkingTimeout,RecursiveMutex,ObjectDeadBit} because they test threads attached to the main thread's heap.

BUG= 591606 

Review-Url: https://codereview.chromium.org/2625293002
Cr-Commit-Position: refs/heads/master@{#443190}

[modify] https://crrev.com/5c3ca4cbc1798a8d5551b6636252cf701b2051ad/third_party/WebKit/Source/core/html/parser/HTMLParserThread.cpp
[modify] https://crrev.com/5c3ca4cbc1798a8d5551b6636252cf701b2051ad/third_party/WebKit/Source/core/workers/WorkerBackingThread.cpp
[modify] https://crrev.com/5c3ca4cbc1798a8d5551b6636252cf701b2051ad/third_party/WebKit/Source/modules/fetch/DataConsumerHandleTestUtil.cpp
[modify] https://crrev.com/5c3ca4cbc1798a8d5551b6636252cf701b2051ad/third_party/WebKit/Source/modules/webdatabase/DatabaseThread.cpp
[modify] https://crrev.com/5c3ca4cbc1798a8d5551b6636252cf701b2051ad/third_party/WebKit/Source/platform/WebThreadSupportingGC.cpp
[modify] https://crrev.com/5c3ca4cbc1798a8d5551b6636252cf701b2051ad/third_party/WebKit/Source/platform/WebThreadSupportingGC.h
[modify] https://crrev.com/5c3ca4cbc1798a8d5551b6636252cf701b2051ad/third_party/WebKit/Source/platform/heap/BlinkGC.h
[modify] https://crrev.com/5c3ca4cbc1798a8d5551b6636252cf701b2051ad/third_party/WebKit/Source/platform/heap/Heap.cpp
[modify] https://crrev.com/5c3ca4cbc1798a8d5551b6636252cf701b2051ad/third_party/WebKit/Source/platform/heap/HeapTest.cpp
[modify] https://crrev.com/5c3ca4cbc1798a8d5551b6636252cf701b2051ad/third_party/WebKit/Source/platform/heap/ThreadState.cpp
[modify] https://crrev.com/5c3ca4cbc1798a8d5551b6636252cf701b2051ad/third_party/WebKit/Source/platform/heap/ThreadState.h

Status: Fixed (was: Started)
A significant architectural improvement, well done. :)

Sign in to add a comment