Monitor VideoResourceUpdater reusing destructed resource in Debug mode.

cc/resources/video_resource_updater.cc

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "cc/resources/video_resource_updater.h"
 
 #include <stddef.h>
 #include <stdint.h>
 
 #include <algorithm>
@@ skipping 114 matching lines @@
       mailbox(mailbox),
       ref_count(0),
       frame_ptr(nullptr),
       plane_index(0u) {
 }
 
 bool VideoResourceUpdater::PlaneResourceMatchesUniqueID(
     const PlaneResource& plane_resource,
     const media::VideoFrame* video_frame,
     size_t plane_index) {
-  return plane_resource.frame_ptr == video_frame &&
-         plane_resource.plane_index == plane_index &&
-         plane_resource.timestamp == video_frame->timestamp();
+  if (plane_resource.frame_ptr == video_frame &&
+      plane_resource.plane_index == plane_index &&
+      plane_resource.timestamp == video_frame->timestamp()) {
+    bool video_frame_in_resource;

[miu, 2016/02/22 21:17:37]

There's a helper method in VideoFrameMetadata to avoid having to do all this. 
You can just:

  return plane_resource.frame_ptr == video_frame &&
         video_frame->metadata()->IsTrue(
             media::VideoFrameMetadata::RESOURCE_UPDATER_FLAG) && 
         plane_resource.plane_index == plane_index &&
         plane_resource.timestamp == video_frame->timestamp();

[xjz, 2016/02/23 03:36:03]

Done.

+    if (video_frame->metadata()->GetBoolean(
+            media::VideoFrameMetadata::RESOURCE_UPDATER_FLAG,
+            &video_frame_in_resource))
+      return video_frame_in_resource;
+    else
+      return false;
+  }
+  return false;
 }
 
 void VideoResourceUpdater::SetPlaneResourceUniqueId(
-    const media::VideoFrame* video_frame,
+    media::VideoFrame* video_frame,
     size_t plane_index,
     PlaneResource* plane_resource) {
   plane_resource->frame_ptr = video_frame;
   plane_resource->plane_index = plane_index;
   plane_resource->timestamp = video_frame->timestamp();
+  video_frame->metadata()->SetBoolean(
+      media::VideoFrameMetadata::RESOURCE_UPDATER_FLAG, true);
 }
 
 VideoFrameExternalResources::VideoFrameExternalResources()
     : type(NONE),
       read_lock_fences_enabled(false),
       offset(0.0f),
       multiplier(1.0f) {}
 
 VideoFrameExternalResources::~VideoFrameExternalResources() {}
 
@@ skipping 524 matching lines @@
     resource_it->ref_count = 0;
     updater->DeleteResource(resource_it);
     return;
   }
 
   --resource_it->ref_count;
   DCHECK_GE(resource_it->ref_count, 0);
 }
 
 }  // namespace cc