[Media Intent Handler] Prevent crashing when unable to query

This CL adds a try/catch block to prevent SecurityExceptions from
crashing media opened via the media intent handler. In certain cases,
we don't have permission to query for the file display name. In those
cases, instead of crashing we will instead just fall back to having no
display name.

Bug: 874149
Change-Id: I18ed34655e093cf04c9cb745eeeb6182c8fe28b6
Reviewed-on: https://chromium-review.googlesource.com/1175212Reviewed-by: Ted Choc <tedchoc@chromium.org>
Reviewed-by: agrieve <agrieve@chromium.org>
Commit-Queue: Tommy Steimel <steimel@chromium.org>
Cr-Commit-Position: refs/heads/master@{#583312}

base/android/java/src/org/chromium/base/ContentUriUtils.java

@@ -221,10 +221,17 @@ public abstract class ContentUriUtils {
     @CalledByNative
     public static String maybeGetDisplayName(String uriString) {
         Uri uri = Uri.parse(uriString);
-        String displayName = getDisplayName(
-                uri, ContextUtils.getApplicationContext(), MediaStore.MediaColumns.DISPLAY_NAME);
 
-        return TextUtils.isEmpty(displayName) ? null : displayName;
+        try {
+            String displayName = getDisplayName(uri, ContextUtils.getApplicationContext(),
+                    MediaStore.MediaColumns.DISPLAY_NAME);
+            return TextUtils.isEmpty(displayName) ? null : displayName;
+        } catch (SecurityException e) {
+            Log.w(TAG, "Cannot open content uri: " + uriString, e);
+        }
+
+        // If we are unable to query the content URI, just return null.
+        return null;
     }
 
     /**