Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(4472)

Unified Diff: chrome/android/java/src/org/chromium/chrome/browser/download/ui/OfflineGroupHeaderView.java

Issue 2830843004: Update to newer Android Lint and suppress new Lint errors (Closed)
Patch Set: rebase Created 3 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/android/java/src/org/chromium/chrome/browser/download/ui/OfflineGroupHeaderView.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/download/ui/OfflineGroupHeaderView.java b/chrome/android/java/src/org/chromium/chrome/browser/download/ui/OfflineGroupHeaderView.java
index c2ba6904d78b8f17c71c58ba6bb0305fb9e0f81d..be97920ce5b1e954d38fb06204aca65774b1f9a9 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/download/ui/OfflineGroupHeaderView.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/download/ui/OfflineGroupHeaderView.java
@@ -4,6 +4,7 @@
package org.chromium.chrome.browser.download.ui;
+import android.annotation.SuppressLint;
import android.content.Context;
import android.content.res.ColorStateList;
import android.text.format.Formatter;
@@ -82,8 +83,10 @@ public class OfflineGroupHeaderView
* Updates the properties of this view.
* @param header The associated {@link SubsectionHeader}.
*/
+ @SuppressLint("StringFormatMatches")
public void displayHeader(SubsectionHeader header) {
this.mHeader = header;
+ // TODO(crbug.com/635567): Fix lint properly.
mPageCountHeader.setText(getResources().getString(
R.string.download_manager_offline_header_title, header.getItemCount()));
mFileSizeView.setText(Formatter.formatFileSize(getContext(), header.getTotalFileSize()));

Powered by Google App Engine
This is Rietveld 408576698