diff --git a/core/java/android/provider/MediaStore.java b/core/java/android/provider/MediaStore.java index 964567a0a95b2..1d06f9035cf09 100644 --- a/core/java/android/provider/MediaStore.java +++ b/core/java/android/provider/MediaStore.java @@ -972,7 +972,7 @@ public final class MediaStore { public static String keyFor(String name) { if (name != null) { boolean sortfirst = false; - if (name.equals(android.media.MediaFile.UNKNOWN_STRING)) { + if (name.equals(UNKNOWN_STRING)) { return "\001"; } // Check if the first character is \001. We use this to diff --git a/media/java/android/media/MediaFile.java b/media/java/android/media/MediaFile.java index 8180a7cfbc30e..a09aa5ce3a2a4 100644 --- a/media/java/android/media/MediaFile.java +++ b/media/java/android/media/MediaFile.java @@ -155,8 +155,6 @@ public class MediaFile { sFileExtensions = builder.toString(); } - public static final String UNKNOWN_STRING = ""; - public static boolean isAudioFileType(int fileType) { return ((fileType >= FIRST_AUDIO_FILE_TYPE && fileType <= LAST_AUDIO_FILE_TYPE) || diff --git a/media/java/android/media/MediaScanner.java b/media/java/android/media/MediaScanner.java index 4296afb3968b3..3a2f47ad729b3 100644 --- a/media/java/android/media/MediaScanner.java +++ b/media/java/android/media/MediaScanner.java @@ -637,8 +637,8 @@ public class MediaScanner map.put(MediaStore.MediaColumns.MIME_TYPE, mMimeType); if (MediaFile.isVideoFileType(mFileType)) { - map.put(Video.Media.ARTIST, (mArtist != null && mArtist.length() > 0 ? mArtist : MediaFile.UNKNOWN_STRING)); - map.put(Video.Media.ALBUM, (mAlbum != null && mAlbum.length() > 0 ? mAlbum : MediaFile.UNKNOWN_STRING)); + map.put(Video.Media.ARTIST, (mArtist != null && mArtist.length() > 0 ? mArtist : MediaStore.UNKNOWN_STRING)); + map.put(Video.Media.ALBUM, (mAlbum != null && mAlbum.length() > 0 ? mAlbum : MediaStore.UNKNOWN_STRING)); map.put(Video.Media.DURATION, mDuration); map.put(Video.Media.DATE_TAKEN, mLastModified * 1000); // FIXME - add RESOLUTION @@ -648,8 +648,8 @@ public class MediaScanner // contains date time information. map.put(Images.Media.DATE_TAKEN, mLastModified * 1000); } else if (MediaFile.isAudioFileType(mFileType)) { - map.put(Audio.Media.ARTIST, (mArtist != null && mArtist.length() > 0 ? mArtist : MediaFile.UNKNOWN_STRING)); - map.put(Audio.Media.ALBUM, (mAlbum != null && mAlbum.length() > 0 ? mAlbum : MediaFile.UNKNOWN_STRING)); + map.put(Audio.Media.ARTIST, (mArtist != null && mArtist.length() > 0 ? mArtist : MediaStore.UNKNOWN_STRING)); + map.put(Audio.Media.ALBUM, (mAlbum != null && mAlbum.length() > 0 ? mAlbum : MediaStore.UNKNOWN_STRING)); map.put(Audio.Media.COMPOSER, mComposer); if (mYear != 0) { map.put(Audio.Media.YEAR, mYear); @@ -705,7 +705,7 @@ public class MediaScanner values.put(MediaStore.MediaColumns.TITLE, title); } String album = values.getAsString(Audio.Media.ALBUM); - if (MediaFile.UNKNOWN_STRING.equals(album)) { + if (MediaStore.UNKNOWN_STRING.equals(album)) { album = values.getAsString(MediaStore.MediaColumns.DATA); // extract last path segment before file name int lastSlash = album.lastIndexOf('/');