From ded797a710c1a07b06f8d84d697c54671a942b35 Mon Sep 17 00:00:00 2001 From: mlamouri Date: Fri, 4 Sep 2015 03:54:23 -0700 Subject: [PATCH] Android Media Notification: remove "Touch to return" text. The notification only shows the origin. Touching the notification to return to the playing application is a very common pattern on Android. It is not required for us to reinforce it with text, making the notification less appealing. BUG=528202 Review URL: https://codereview.chromium.org/1326163002 Cr-Commit-Position: refs/heads/master@{#347377} --- .../media/ui/NotificationMediaPlaybackControls.java | 10 +--------- 1 file changed, 1 insertion(+), 9 deletions(-) diff --git a/chrome/android/java/src/org/chromium/chrome/browser/media/ui/NotificationMediaPlaybackControls.java b/chrome/android/java/src/org/chromium/chrome/browser/media/ui/NotificationMediaPlaybackControls.java index 94175d120cada..7df892a41985a 100644 --- a/chrome/android/java/src/org/chromium/chrome/browser/media/ui/NotificationMediaPlaybackControls.java +++ b/chrome/android/java/src/org/chromium/chrome/browser/media/ui/NotificationMediaPlaybackControls.java @@ -346,14 +346,6 @@ private String sanitizeMediaTitle(String title) { return title.startsWith("\u25B6") ? title.substring(1).trim() : title; } - private String getStatus() { - if (mMediaNotificationInfo.origin != null) { - return mContext.getString(R.string.media_notification_link_text, - mMediaNotificationInfo.origin); - } - return mContext.getString(R.string.media_notification_text_no_link); - } - private PendingIntent createContentIntent() { int tabId = mMediaNotificationInfo.tabId; return PendingIntent.getActivity( @@ -411,7 +403,7 @@ private void updateNotification() { RemoteViews contentView = createContentView(); contentView.setTextViewText(R.id.title, mMediaNotificationInfo.title); - contentView.setTextViewText(R.id.status, getStatus()); + contentView.setTextViewText(R.id.status, mMediaNotificationInfo.origin); if (mNotificationIcon != null) { contentView.setImageViewBitmap(R.id.icon, mNotificationIcon); } else {