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

Side by Side Diff: Source/core/html/parser/HTMLConstructionSite.cpp

Issue 1112363003: Oilpan: Remove OffHeapCollectionTrait (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 5 years, 7 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 | Annotate | Revision Log
OLDNEW
1 /* 1 /*
2 * Copyright (C) 2010 Google, Inc. All Rights Reserved. 2 * Copyright (C) 2010 Google, Inc. All Rights Reserved.
3 * Copyright (C) 2011 Apple Inc. All rights reserved. 3 * Copyright (C) 2011 Apple Inc. All rights reserved.
4 * 4 *
5 * Redistribution and use in source and binary forms, with or without 5 * Redistribution and use in source and binary forms, with or without
6 * modification, are permitted provided that the following conditions 6 * modification, are permitted provided that the following conditions
7 * are met: 7 * are met:
8 * 1. Redistributions of source code must retain the above copyright 8 * 1. 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 * 2. Redistributions in binary form must reproduce the above copyright 10 * 2. Redistributions in binary form must reproduce the above copyright
(...skipping 337 matching lines...) Expand 10 before | Expand all | Expand 10 after
348 // Depending on why we're being destroyed it might be OK 348 // Depending on why we're being destroyed it might be OK
349 // to forget queued tasks, but currently we don't expect to. 349 // to forget queued tasks, but currently we don't expect to.
350 ASSERT(m_taskQueue.isEmpty()); 350 ASSERT(m_taskQueue.isEmpty());
351 // Currently we assume that text will never be the last token in the 351 // Currently we assume that text will never be the last token in the
352 // document and that we'll always queue some additional task to cause it to flush. 352 // document and that we'll always queue some additional task to cause it to flush.
353 ASSERT(m_pendingText.isEmpty()); 353 ASSERT(m_pendingText.isEmpty());
354 } 354 }
355 355
356 DEFINE_TRACE(HTMLConstructionSite) 356 DEFINE_TRACE(HTMLConstructionSite)
357 { 357 {
358 #if ENABLE(OILPAN)
358 visitor->trace(m_document); 359 visitor->trace(m_document);
359 visitor->trace(m_attachmentRoot); 360 visitor->trace(m_attachmentRoot);
360 visitor->trace(m_head); 361 visitor->trace(m_head);
361 visitor->trace(m_form); 362 visitor->trace(m_form);
362 visitor->trace(m_openElements); 363 visitor->trace(m_openElements);
363 visitor->trace(m_activeFormattingElements); 364 visitor->trace(m_activeFormattingElements);
364 visitor->trace(m_taskQueue); 365 visitor->trace(m_taskQueue);
365 visitor->trace(m_pendingText); 366 visitor->trace(m_pendingText);
367 #endif
366 } 368 }
367 369
368 void HTMLConstructionSite::detach() 370 void HTMLConstructionSite::detach()
369 { 371 {
370 // FIXME: We'd like to ASSERT here that we're canceling and not just discard ing 372 // FIXME: We'd like to ASSERT here that we're canceling and not just discard ing
371 // text that really should have made it into the DOM earlier, but there 373 // text that really should have made it into the DOM earlier, but there
372 // doesn't seem to be a nice way to do that. 374 // doesn't seem to be a nice way to do that.
373 m_pendingText.discard(); 375 m_pendingText.discard();
374 m_document = nullptr; 376 m_document = nullptr;
375 m_attachmentRoot = nullptr; 377 m_attachmentRoot = nullptr;
(...skipping 503 matching lines...) Expand 10 before | Expand all | Expand 10 after
879 } 881 }
880 882
881 DEFINE_TRACE(HTMLConstructionSite::PendingText) 883 DEFINE_TRACE(HTMLConstructionSite::PendingText)
882 { 884 {
883 visitor->trace(parent); 885 visitor->trace(parent);
884 visitor->trace(nextChild); 886 visitor->trace(nextChild);
885 } 887 }
886 888
887 889
888 } 890 }
OLDNEW
« no previous file with comments | « Source/core/html/canvas/WebGLRenderingContextBase.cpp ('k') | Source/core/html/parser/HTMLFormattingElementList.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698