Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(500)

Issue 2614883006: Change PartitionAlloc to Chromium naming style. (Closed)

Created:
3 years, 11 months ago by palmer
Modified:
3 years, 11 months ago
Reviewers:
haraken
CC:
Mads Ager (chromium), chromium-reviews, Dai Mikurube (NOT FULLTIME), kouhei+heap_chromium.org, oilpan-reviews, vmpstr+watch_chromium.org, wfh+watch_chromium.org
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Change PartitionAlloc to Chromium naming style. BUG=632441 CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2 Review-Url: https://codereview.chromium.org/2614883006 Cr-Commit-Position: refs/heads/master@{#442387} Committed: https://chromium.googlesource.com/chromium/src/+/6a6836143aac1e1ad32aae278e0e9f9e2488d322

Patch Set 1 #

Patch Set 2 : Fix more names. #

Patch Set 3 : More names updated. More to go. #

Patch Set 4 : Update the call sites, too. #

Patch Set 5 : Rebase and fix some names. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1959 lines, -1929 lines) Patch
M base/allocator/partition_allocator/address_space_randomization.h View 1 2 1 chunk +1 line, -1 line 0 comments Download
M base/allocator/partition_allocator/address_space_randomization.cc View 1 2 1 chunk +1 line, -1 line 0 comments Download
M base/allocator/partition_allocator/page_allocator.h View 1 2 3 4 chunks +11 lines, -11 lines 0 comments Download
M base/allocator/partition_allocator/page_allocator.cc View 1 2 3 2 chunks +95 lines, -95 lines 0 comments Download
M base/allocator/partition_allocator/partition_alloc.h View 1 2 3 16 chunks +287 lines, -282 lines 0 comments Download
M base/allocator/partition_allocator/partition_alloc.cc View 1 2 3 21 chunks +616 lines, -612 lines 0 comments Download
M base/allocator/partition_allocator/partition_alloc_unittest.cc View 1 2 3 4 45 chunks +861 lines, -842 lines 0 comments Download
M third_party/WebKit/Source/core/layout/LayoutObject.cpp View 1 2 3 4 1 chunk +2 lines, -2 lines 0 comments Download
M third_party/WebKit/Source/core/layout/line/InlineBox.cpp View 1 2 3 1 chunk +2 lines, -2 lines 0 comments Download
M third_party/WebKit/Source/core/paint/PaintLayer.cpp View 1 2 3 4 1 chunk +2 lines, -2 lines 0 comments Download
M third_party/WebKit/Source/platform/PartitionAllocMemoryDumpProvider.cpp View 1 2 3 4 chunks +33 lines, -31 lines 0 comments Download
M third_party/WebKit/Source/platform/heap/CallbackStack.cpp View 1 2 3 1 chunk +2 lines, -2 lines 0 comments Download
M third_party/WebKit/Source/platform/heap/HeapPage.cpp View 1 2 3 4 1 chunk +3 lines, -3 lines 0 comments Download
M third_party/WebKit/Source/platform/heap/PageMemory.cpp View 1 2 3 2 chunks +6 lines, -6 lines 0 comments Download
M third_party/WebKit/Source/platform/text/BidiCharacterRun.cpp View 1 2 3 1 chunk +2 lines, -2 lines 0 comments Download
M third_party/WebKit/Source/web/WebMemoryStatistics.cpp View 1 2 3 1 chunk +3 lines, -3 lines 0 comments Download
M third_party/WebKit/Source/wtf/allocator/PartitionAllocator.h View 1 2 3 1 chunk +1 line, -1 line 0 comments Download
M third_party/WebKit/Source/wtf/allocator/Partitions.h View 1 2 3 4 chunks +21 lines, -21 lines 0 comments Download
M third_party/WebKit/Source/wtf/allocator/Partitions.cpp View 1 2 3 4 chunks +9 lines, -9 lines 0 comments Download
M third_party/WebKit/Source/wtf/typed_arrays/ArrayBufferContents.cpp View 1 2 3 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 14 (10 generated)
palmer
This is (supposed to be) a mechanical rename CL.
3 years, 11 months ago (2017-01-07 02:37:10 UTC) #3
haraken
LGTM
3 years, 11 months ago (2017-01-07 05:49:52 UTC) #8
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2614883006/80001
3 years, 11 months ago (2017-01-09 20:04:11 UTC) #11
commit-bot: I haz the power
3 years, 11 months ago (2017-01-09 22:51:10 UTC) #14
Message was sent while issue was closed.
Committed patchset #5 (id:80001) as
https://chromium.googlesource.com/chromium/src/+/6a6836143aac1e1ad32aae278e0e...

Powered by Google App Engine
This is Rietveld 408576698