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

Unified Diff: chrome/browser/resources/settings/about_page/about_page_browser_proxy.js

Issue 1987813004: MD Settings: About page, implementing update status. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Update test, after changing "=" to "$=" Created 4 years, 7 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/settings/about_page/about_page_browser_proxy.js
diff --git a/chrome/browser/resources/settings/about_page/about_page_browser_proxy.js b/chrome/browser/resources/settings/about_page/about_page_browser_proxy.js
index 491eddeb8bec8a5062ed583a0c89b2004a5f0845..3adbabf1b1d3bfe5338c53dda3e12b0ec01ec5e6 100644
--- a/chrome/browser/resources/settings/about_page/about_page_browser_proxy.js
+++ b/chrome/browser/resources/settings/about_page/about_page_browser_proxy.js
@@ -36,7 +36,45 @@ var BrowserChannel = {
};
</if>
+/**
+ * Enumeration of all possible update statuses. The string literals must match
+ * the ones defined at |AboutHandler::UpdateStatusToString|.
+ * @enum {string}
+ */
+var UpdateStatus = {
+ CHECKING: 'checking',
+ UPDATING: 'updating',
+ NEARLY_UPDATED: 'nearly_updated',
+ UPDATED: 'updated',
+ FAILED: 'failed',
+ DISABLED: 'disabled',
+ DISABLED_BY_ADMIN: 'disabled_by_admin',
+};
+
+/**
+ * @typedef {{
+ * status: !UpdateStatus,
+ * progress: number,
+ * message: string,
+ * }}
+ */
+var UpdateStatusChangedEvent;
+
cr.define('settings', function() {
+ /**
+ * @param {!BrowserChannel} channel
+ * @return {string}
+ */
+ function browserChannelToI18nId(channel) {
+ switch (channel) {
+ case BrowserChannel.BETA: return 'aboutChannelBeta';
+ case BrowserChannel.DEV: return 'aboutChannelDev';
+ case BrowserChannel.STABLE: return 'aboutChannelStable';
+ }
+
+ assertNotReached();
+ }
+
/** @interface */
function AboutPageBrowserProxy() {}
@@ -164,5 +202,6 @@ cr.define('settings', function() {
return {
AboutPageBrowserProxy: AboutPageBrowserProxy,
AboutPageBrowserProxyImpl: AboutPageBrowserProxyImpl,
+ browserChannelToI18nId: browserChannelToI18nId,
};
});

Powered by Google App Engine
This is Rietveld 408576698