Speculative fix when translating blob data to a file path

content/browser/media/android/media_resource_getter_impl.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 "content/browser/media/android/media_resource_getter_impl.h"
 
 #include "base/android/jni_android.h"
 #include "base/android/jni_string.h"
 #include "base/bind.h"
 #include "base/path_service.h"
@@ skipping 33 matching lines @@
 
 static void RequestPlatformPathFromBlobURL(
     const GURL& url,
     BrowserContext* browser_context,
     const base::Callback<void(const std::string&)>& callback) {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::IO));
   ChromeBlobStorageContext* context =
       ChromeBlobStorageContext::GetFor(browser_context);
   scoped_ptr<storage::BlobDataHandle> handle =
       context->context()->GetBlobDataFromPublicURL(url);
-  const std::vector<storage::BlobData::Item> items = handle->data()->items();
+  storage::BlobData* data = handle->data();
+  if (!data) {
+    ReturnResultOnUIThread(callback, "");

[xhwang, 2014/09/25 23:11:11]

If this shouldn't happen, add a NOTREACHED so that someone will hit this and we
can understand why it happened?

[qinmin, 2014/09/26 18:48:49]

Done.

+    return;
+  }
+  const std::vector<storage::BlobData::Item> items = data->items();
 
   // TODO(qinmin): handle the case when the blob data is not a single file.
   DLOG_IF(WARNING, items.size() != 1u)
       << "More than one blob data are present: " << items.size();
   ReturnResultOnUIThread(callback, items[0].path().value());
 }
 
 static void RequestPlaformPathFromFileSystemURL(
     const GURL& url,
     int render_process_id,
@@ skipping 295 matching lines @@
       FROM_HERE,
       base::Bind(&GetMediaMetadataFromFd, fd, offset, size, callback));
 }
 
 // static
 bool MediaResourceGetterImpl::RegisterMediaResourceGetter(JNIEnv* env) {
   return RegisterNativesImpl(env);
 }
 
 }  // namespace content