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

Unified Diff: chrome/browser/resources/print_preview/search/destination_list_item.js

Issue 150943006: Show destination offline status in the print preview. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 10 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/browser/resources/print_preview/search/destination_list_item.js
diff --git a/chrome/browser/resources/print_preview/search/destination_list_item.js b/chrome/browser/resources/print_preview/search/destination_list_item.js
index 40f228e048d9ba0018d713bfe958305e366a61f0..d6f1a639f70bfd14b357ffd4c59aaf787eb4ad6d 100644
--- a/chrome/browser/resources/print_preview/search/destination_list_item.js
+++ b/chrome/browser/resources/print_preview/search/destination_list_item.js
@@ -95,23 +95,10 @@ cr.define('print_preview', function() {
* @private
*/
initializeOfflineStatusElement_: function() {
- if (arrayContains([print_preview.Destination.ConnectionStatus.OFFLINE,
- print_preview.Destination.ConnectionStatus.DORMANT],
- this.destination_.connectionStatus)) {
+ if (this.destination_.isOffline) {
this.getElement().classList.add(DestinationListItem.Classes_.STALE);
- var offlineDurationMs = Date.now() - this.destination_.lastAccessTime;
- var offlineMessageId;
- if (offlineDurationMs > 31622400000.0) { // One year.
- offlineMessageId = 'offlineForYear';
- } else if (offlineDurationMs > 2678400000.0) { // One month.
- offlineMessageId = 'offlineForMonth';
- } else if (offlineDurationMs > 604800000.0) { // One week.
- offlineMessageId = 'offlineForWeek';
- } else {
- offlineMessageId = 'offline';
- }
var offlineStatusEl = this.getChildElement('.offline-status');
- offlineStatusEl.textContent = localStrings.getString(offlineMessageId);
+ offlineStatusEl.textContent = this.destination_.offlineStatusText;
setIsVisible(offlineStatusEl, true);
}
},

Powered by Google App Engine
This is Rietveld 408576698