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

Unified Diff: chrome/common/extensions/docs/examples/extensions/calendar/javascript/options.js

Issue 890173003: Replace the sample Calendar extension with a deprecation notice (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: . Created 5 years, 11 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/common/extensions/docs/examples/extensions/calendar/javascript/options.js
diff --git a/chrome/common/extensions/docs/examples/extensions/calendar/javascript/options.js b/chrome/common/extensions/docs/examples/extensions/calendar/javascript/options.js
index 677d43303aa07a22061cb70fd1d6d9200118ced8..a524b8982b6f70b3c1c8f410a3b7ace45c46a1bd 100644
--- a/chrome/common/extensions/docs/examples/extensions/calendar/javascript/options.js
+++ b/chrome/common/extensions/docs/examples/extensions/calendar/javascript/options.js
@@ -4,62 +4,14 @@
* LICENSE file.
*/
-//Contains true if multiple calendar option is checked, false otherwise.
-var isMultiCalendar;
+var $ = document.getElementById.bind(document);
-//adding listener when body is loaded to call init function.
-window.addEventListener('load', init, false);
+var url = 'https://chrome.google.com/webstore/detail/' +
+ 'google-calendar-by-google/gmbgaklkmjakoegficnlkhebmhkjfich';
-/**
- * Sets the value of multiple calendar checkbox based on value from
- * local storage.
- */
- function init() {
- isMultiCalendar = JSON.parse(localStorage.multiCalendar || false);
- $('multiCalendar').checked = isMultiCalendar;
- $('multiCalendarText').innerHTML =
- chrome.i18n.getMessage('multiCalendarText');
- $('multiCalendar').addEventListener('click', save);
- $('optionsTitle').innerHTML = chrome.i18n.getMessage('optionsTitle');
- $('imageTooltip').title = chrome.i18n.getMessage('imageTooltip');
- $('imageTooltip').alt = chrome.i18n.getMessage('imageTooltip');
- $('multiCalendarText').title = chrome.i18n.getMessage('multiCalendarToolTip');
- $('multiCalendar').title = chrome.i18n.getMessage('multiCalendarToolTip');
- $('extensionName').innerHTML = chrome.i18n.getMessage('extensionName');
- if (chrome.i18n.getMessage('direction') == 'rtl') {
- $('body').style.direction = 'rtl';
- }
-};
-
-/**
- * Saves the value of the checkbox into local storage.
- */
-function save() {
- var multiCalendarId = $('multiCalendar');
- localStorage.multiCalendar = multiCalendarId.checked;
- if (multiCalendarId) {
- multiCalendar.disabled = true;
- }
- $('status').innerHTML = chrome.i18n.getMessage('status_saving');
- $('status').style.display = 'block';
-
- // Sends message to the background page notifying it that the settings
- // have updated.
- chrome.runtime.getBackgroundPage(function(bg) {
- bg.onSettingsChange();
- statusSaved();
- });
-};
-
-function statusSaved() {
- if ($('multiCalendar')) {
- if ($('multiCalendar').disabled) {
- $('status').innerHTML = chrome.i18n.getMessage('status_saved');
- $('status').style.display = 'block';
- setTimeout(
- function() { $('status').style.display = 'none'; }
- , 1500);
- }
- $('multiCalendar').disabled = false;
- }
+$('name').textContent = chrome.i18n.getMessage('name');
+$('link').href = url;
+$('remove').onclick = function() {
+ chrome.management.uninstallSelf({showConfirmDialog: true});
+ window.close();
};

Powered by Google App Engine
This is Rietveld 408576698