Wrap GCed raw pointer parameters of WTF::bind with Persistent

third_party/WebKit/Source/core/html/HTMLSlotElement.cpp

 /*
  * Copyright (C) 2015 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 276 matching lines @@
         if (isHTMLSlotElement(child))
             appendDistributedNodesFrom(toHTMLSlotElement(child));
         else
             appendDistributedNode(child);
     }
 }
 
 void HTMLSlotElement::enqueueSlotChangeEvent()
 {
     if (!m_slotchangeEventEnqueued) {
-        Microtask::enqueueMicrotask(WTF::bind(&HTMLSlotElement::dispatchSlotChangeEvent, Persistent<HTMLSlotElement>(this)));
+        Microtask::enqueueMicrotask(WTF::bind(&HTMLSlotElement::dispatchSlotChangeEvent, wrapCrossThreadPersistent(this)));

[hiroshige, 2016/06/15 12:12:39]

Shouldn't this be |wrapPersistent(this)| to keep the current behavior, because
it is originally Persistent<HTMLSlotElement>?

[tzik, 2016/06/21 09:09:04]

Done.

         m_slotchangeEventEnqueued = true;
     }
 
     ShadowRoot* root = containingShadowRoot();
     DCHECK(root);
     DCHECK(root->isV1());
     root->owner()->setNeedsDistributionRecalc();
 
     if (ShadowRoot* parentShadowRoot = v1ShadowRootOfParent()) {
         if (HTMLSlotElement* next = parentShadowRoot->ensureSlotAssignment().findSlot(*this))
@@ skipping 28 matching lines @@
 
 DEFINE_TRACE(HTMLSlotElement)
 {
     visitor->trace(m_assignedNodes);
     visitor->trace(m_distributedNodes);
     visitor->trace(m_distributedIndices);
     HTMLElement::trace(visitor);
 }
 
 } // namespace blink