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

Unified Diff: chrome/browser/resources/settings/languages_page/languages_page.js

Issue 2461113002: WebUI: Make settings-action-menu re-usable as cr-action-menu. (Closed)
Patch Set: getComputedStyle instead of util.js Created 4 years, 1 month 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/languages_page/languages_page.js
diff --git a/chrome/browser/resources/settings/languages_page/languages_page.js b/chrome/browser/resources/settings/languages_page/languages_page.js
index 9cfd9a2fc614d34ac14f2ff0407bfc7c7c2856ff..1ba3d569d5f78a52f49326ce2ce1abc4cbf2c1c1 100644
--- a/chrome/browser/resources/settings/languages_page/languages_page.js
+++ b/chrome/browser/resources/settings/languages_page/languages_page.js
@@ -185,7 +185,7 @@ Polymer({
// Reset the chosen UI language to the actual UI language.
this.languageHelper.resetUILanguage();
}
- /** @type {!SettingsActionMenuElement} */(this.$.menu.get()).close();
+ /** @type {!CrActionMenuElement} */(this.$.menu.get()).close();
},
/**
@@ -215,7 +215,7 @@ Polymer({
this.languageHelper.disableTranslateLanguage(
this.detailLanguage_.language.code);
}
- /** @type {!SettingsActionMenuElement} */(this.$.menu.get()).close();
+ /** @type {!CrActionMenuElement} */(this.$.menu.get()).close();
},
/**
@@ -235,7 +235,7 @@ Polymer({
* @private
*/
onMoveToTopTap_: function() {
- /** @type {!SettingsActionMenuElement} */(this.$.menu.get()).close();
+ /** @type {!CrActionMenuElement} */(this.$.menu.get()).close();
this.languageHelper.moveLanguageToFront(this.detailLanguage_.language.code);
},
@@ -244,7 +244,7 @@ Polymer({
* @private
*/
onMoveUpTap_: function() {
- /** @type {!SettingsActionMenuElement} */(this.$.menu.get()).close();
+ /** @type {!CrActionMenuElement} */(this.$.menu.get()).close();
this.languageHelper.moveLanguage(this.detailLanguage_.language.code, -1);
},
@@ -253,7 +253,7 @@ Polymer({
* @private
*/
onMoveDownTap_: function() {
- /** @type {!SettingsActionMenuElement} */(this.$.menu.get()).close();
+ /** @type {!CrActionMenuElement} */(this.$.menu.get()).close();
this.languageHelper.moveLanguage(this.detailLanguage_.language.code, 1);
},
@@ -262,7 +262,7 @@ Polymer({
* @private
*/
onRemoveLanguageTap_: function() {
- /** @type {!SettingsActionMenuElement} */(this.$.menu.get()).close();
+ /** @type {!CrActionMenuElement} */(this.$.menu.get()).close();
this.languageHelper.disableLanguage(this.detailLanguage_.language.code);
},
@@ -457,10 +457,10 @@ Polymer({
/** @type {!{model: !{item: !LanguageState}}} */(e).model.item;
// Ensure the template has been stamped.
- var menu = /** @type {?SettingsActionMenuElement} */(
+ var menu = /** @type {?CrActionMenuElement} */(
this.$.menu.getIfExists());
if (!menu) {
- menu = /** @type {!SettingsActionMenuElement} */(this.$.menu.get());
+ menu = /** @type {!CrActionMenuElement} */(this.$.menu.get());
this.initializeMenu_(menu);
}
@@ -469,7 +469,7 @@ Polymer({
/**
* Applies Chrome OS session tweaks to the menu.
- * @param {!SettingsActionMenuElement} menu
+ * @param {!CrActionMenuElement} menu
* @private
*/
initializeMenu_: function(menu) {

Powered by Google App Engine
This is Rietveld 408576698