ASSERT -> RELEASE_ASSERT for investigation of issue 466793

Source/platform/graphics/Canvas2DLayerBridge.cpp

 /*
  * Copyright (C) 2012 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:
  *
  * 1.  Redistributions of source code must retain the above copyright
  *     notice, this list of conditions and the following disclaimer.
  * 2.  Redistributions in binary form must reproduce the above copyright
@@ skipping 101 matching lines @@
     m_layer->setNearestNeighbor(m_filterQuality == kNone_SkFilterQuality);
     m_canvas->setNotificationClient(this);
 #ifndef NDEBUG
     canvas2DLayerBridgeInstanceCounter.increment();
 #endif
 }
 
 Canvas2DLayerBridge::~Canvas2DLayerBridge()
 {
     ASSERT(m_destructionInProgress);
-    ASSERT(!Canvas2DLayerManager::get().isInList(this));

[Stephen Chennney, 2015/06/25 16:26:24]

Why not just change this to RELEASE_ASSERT(...)?

+    if (Canvas2DLayerManager::get().isInList(this)) {
+        // TODO(junov): This can go back to being an ASSERT when done investigating crbug.com/466793
+        CRASH();
+    }
     if (m_canvas)
         m_canvas->setNotificationClient(nullptr);
     m_layer.clear();
     ASSERT(m_mailboxes.size() == 0);
 #ifndef NDEBUG
     canvas2DLayerBridgeInstanceCounter.decrement();
 #endif
 }
 
 void Canvas2DLayerBridge::beginDestruction()
@@ skipping 399 matching lines @@
     return adoptRef(m_canvas->newImageSnapshot());
 }
 
 Canvas2DLayerBridge::MailboxInfo::MailboxInfo(const MailboxInfo& other) {
     memcpy(&m_mailbox, &other.m_mailbox, sizeof(m_mailbox));
     m_image = other.m_image;
     m_parentLayerBridge = other.m_parentLayerBridge;
 }
 
 } // namespace blink