[NTP::PhysicalWeb] Do not show published time.

chrome/android/java/src/org/chromium/chrome/browser/ntp/snippets/SnippetArticleViewHolder.java

 // Copyright 2015 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.
 
 package org.chromium.chrome.browser.ntp.snippets;
 
 import android.annotation.SuppressLint;
 import android.content.res.Resources;
 import android.graphics.Bitmap;
 import android.graphics.drawable.BitmapDrawable;
@@ skipping 167 matching lines @@
                             : R.dimen.snippets_publisher_margin_top_with_article_snippet);
 
         params.setMargins(params.leftMargin,
                           topMargin,
                           params.rightMargin,
                           params.bottomMargin);
 
         mPublisherBar.setLayoutParams(params);
     }
 
+    private static String getAttributionString(SnippetArticle article) {
+        if (article.mPublishTimestampMilliseconds == 0) return article.mPublisher;
+
+        // DateUtils.getRelativeTimeSpanString(...) calls through to TimeZone.getDefault(). If this
+        // has never been called before it loads the current time zone from disk. In most likelihood
+        // this will have been called previously and the current time zone will have been cached,
+        // but in some cases (eg instrumentation tests) it will cause a strict mode violation.
+        StrictMode.ThreadPolicy oldPolicy = StrictMode.allowThreadDiskReads();
+        CharSequence relativeTimeSpan;
+        try {
+            long time = SystemClock.elapsedRealtime();
+            relativeTimeSpan =
+                    DateUtils.getRelativeTimeSpanString(article.mPublishTimestampMilliseconds,
+                            System.currentTimeMillis(), DateUtils.MINUTE_IN_MILLIS);
+            RecordHistogram.recordTimesHistogram("Android.StrictMode.SnippetUIBuildTime",
+                    SystemClock.elapsedRealtime() - time, TimeUnit.MILLISECONDS);
+        } finally {
+            StrictMode.setThreadPolicy(oldPolicy);
+        }
+        // We format the publisher here so that having a publisher name in an RTL language
+        // doesn't mess up the formatting on an LTR device and vice versa.
+        return String.format(PUBLISHER_FORMAT_STRING,
+                BidiFormatter.getInstance().unicodeWrap(article.mPublisher), relativeTimeSpan);
+    }
+
     public void onBindViewHolder(SnippetArticle article, SuggestionsCategoryInfo categoryInfo) {
         super.onBindViewHolder();
 
         // No longer listen for offline status changes to the old article.
         if (mArticle != null) mArticle.setOfflineStatusChangeRunnable(null);
 
         mArticle = article;
         mCategoryInfo = categoryInfo;
         updateLayout();
 
         mHeadlineTextView.setText(mArticle.mTitle);
-
-        // DateUtils.getRelativeTimeSpanString(...) calls through to TimeZone.getDefault(). If this
-        // has never been called before it loads the current time zone from disk. In most likelihood
-        // this will have been called previously and the current time zone will have been cached,
-        // but in some cases (eg instrumentation tests) it will cause a strict mode violation.
-        StrictMode.ThreadPolicy oldPolicy = StrictMode.allowThreadDiskReads();
-        try {
-            long time = SystemClock.elapsedRealtime();
-            CharSequence relativeTimeSpan = DateUtils.getRelativeTimeSpanString(
-                    mArticle.mPublishTimestampMilliseconds, System.currentTimeMillis(),
-                    DateUtils.MINUTE_IN_MILLIS);
-            RecordHistogram.recordTimesHistogram("Android.StrictMode.SnippetUIBuildTime",
-                    SystemClock.elapsedRealtime() - time, TimeUnit.MILLISECONDS);
-
-            // We format the publisher here so that having a publisher name in an RTL language
-            // doesn't mess up the formatting on an LTR device and vice versa.
-            String publisherAttribution = String.format(PUBLISHER_FORMAT_STRING,
-                    BidiFormatter.getInstance().unicodeWrap(mArticle.mPublisher), relativeTimeSpan);
-            mPublisherTextView.setText(publisherAttribution);
-        } finally {
-            StrictMode.setThreadPolicy(oldPolicy);
-        }
+        mPublisherTextView.setText(getAttributionString(mArticle));
 
         // The favicon of the publisher should match the TextView height.
         int widthSpec = MeasureSpec.makeMeasureSpec(0, MeasureSpec.UNSPECIFIED);
         int heightSpec = MeasureSpec.makeMeasureSpec(0, MeasureSpec.UNSPECIFIED);
         mPublisherTextView.measure(widthSpec, heightSpec);
         mPublisherFaviconSizePx = mPublisherTextView.getMeasuredHeight();
 
         mArticleSnippetTextView.setText(mArticle.mPreviewText);
 
         // If there's still a pending thumbnail fetch, cancel it.
@@ skipping 156 matching lines @@
         ApiCompatibilityUtils.setCompoundDrawablesRelative(
                 mPublisherTextView, drawable, null, null, null);
         mPublisherTextView.setVisibility(View.VISIBLE);
     }
 
     @Override
     public boolean isDismissable() {
         return !isPeeking();
     }
 }