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

Side by Side Diff: third_party/WebKit/Source/platform/heap/ThreadState.cpp

Issue 2640163004: Replace ENABLE(ASSERT) with DCHECK_IS_ON(). (Closed)
Patch Set: Created 3 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 /* 1 /*
2 * Copyright (C) 2013 Google Inc. All rights reserved. 2 * Copyright (C) 2013 Google Inc. All rights reserved.
3 * 3 *
4 * Redistribution and use in source and binary forms, with or without 4 * Redistribution and use in source and binary forms, with or without
5 * modification, are permitted provided that the following conditions are 5 * modification, are permitted provided that the following conditions are
6 * met: 6 * met:
7 * 7 *
8 * * Redistributions of source code must retain the above copyright 8 * * Redistributions of source code must retain the above copyright
9 * notice, this list of conditions and the following disclaimer. 9 * notice, this list of conditions and the following disclaimer.
10 * * Redistributions in binary form must reproduce the above 10 * * Redistributions in binary form must reproduce the above
(...skipping 1224 matching lines...) Expand 10 before | Expand all | Expand 10 after
1235 ASSERT_NOT_REACHED(); 1235 ASSERT_NOT_REACHED();
1236 } 1236 }
1237 } 1237 }
1238 1238
1239 void ThreadState::prepareForThreadStateTermination() { 1239 void ThreadState::prepareForThreadStateTermination() {
1240 ASSERT(checkThread()); 1240 ASSERT(checkThread());
1241 for (int i = 0; i < BlinkGC::NumberOfArenas; ++i) 1241 for (int i = 0; i < BlinkGC::NumberOfArenas; ++i)
1242 m_arenas[i]->prepareHeapForTermination(); 1242 m_arenas[i]->prepareHeapForTermination();
1243 } 1243 }
1244 1244
1245 #if ENABLE(ASSERT) 1245 #if DCHECK_IS_ON()
1246 BasePage* ThreadState::findPageFromAddress(Address address) { 1246 BasePage* ThreadState::findPageFromAddress(Address address) {
1247 for (int i = 0; i < BlinkGC::NumberOfArenas; ++i) { 1247 for (int i = 0; i < BlinkGC::NumberOfArenas; ++i) {
1248 if (BasePage* page = m_arenas[i]->findPageFromAddress(address)) 1248 if (BasePage* page = m_arenas[i]->findPageFromAddress(address))
1249 return page; 1249 return page;
1250 } 1250 }
1251 return nullptr; 1251 return nullptr;
1252 } 1252 }
1253 #endif 1253 #endif
1254 1254
1255 size_t ThreadState::objectPayloadSizeForTesting() { 1255 size_t ThreadState::objectPayloadSizeForTesting() {
(...skipping 528 matching lines...) Expand 10 before | Expand all | Expand 10 after
1784 collectGarbage(BlinkGC::NoHeapPointersOnStack, BlinkGC::GCWithSweep, 1784 collectGarbage(BlinkGC::NoHeapPointersOnStack, BlinkGC::GCWithSweep,
1785 BlinkGC::ForcedGC); 1785 BlinkGC::ForcedGC);
1786 size_t liveObjects = heap().heapStats().markedObjectSize(); 1786 size_t liveObjects = heap().heapStats().markedObjectSize();
1787 if (liveObjects == previousLiveObjects) 1787 if (liveObjects == previousLiveObjects)
1788 break; 1788 break;
1789 previousLiveObjects = liveObjects; 1789 previousLiveObjects = liveObjects;
1790 } 1790 }
1791 } 1791 }
1792 1792
1793 } // namespace blink 1793 } // namespace blink
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698