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

Unified Diff: third_party/polymer/v1_0/components-chromium/paper-dialog-behavior/paper-dialog-behavior-extracted.js

Issue 1862213002: Roll third_party/polymer. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Remove obsolete appearance_browsertest.js, result of a previous bad merge. Created 4 years, 8 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: third_party/polymer/v1_0/components-chromium/paper-dialog-behavior/paper-dialog-behavior-extracted.js
diff --git a/third_party/polymer/v1_0/components-chromium/paper-dialog-behavior/paper-dialog-behavior-extracted.js b/third_party/polymer/v1_0/components-chromium/paper-dialog-behavior/paper-dialog-behavior-extracted.js
index fadeaec18d4053e0f7471af8024d813c38726d47..3ad9e271db5a028290322ea080f2a5f05ba3f16c 100644
--- a/third_party/polymer/v1_0/components-chromium/paper-dialog-behavior/paper-dialog-behavior-extracted.js
+++ b/third_party/polymer/v1_0/components-chromium/paper-dialog-behavior/paper-dialog-behavior-extracted.js
@@ -41,8 +41,6 @@ to override this attribute with `role="alertdialog"`.
If `modal` is set, the element will set `aria-modal` and prevent the focus from exiting the element.
It will also ensure that focus remains in the dialog.
-The `aria-labelledby` attribute will be set to the header element, if one exists.
-
@hero hero.svg
@demo demo/index.html
@polymerBehavior Polymer.PaperDialogBehavior
@@ -82,16 +80,6 @@ The `aria-labelledby` attribute will be set to the header element, if one exists
this.__prevWithBackdrop = this.withBackdrop;
},
- attached: function() {
- // this._observer is used by iron-overlay-behavior
- this._ariaObserver = Polymer.dom(this).observeNodes(this._updateAriaLabelledBy);
- this._updateAriaLabelledBy();
- },
-
- detached: function() {
- Polymer.dom(this).unobserveNodes(this._ariaObserver);
- },
-
_modalChanged: function(modal, readied) {
if (modal) {
this.setAttribute('aria-modal', 'true');
@@ -123,27 +111,6 @@ The `aria-labelledby` attribute will be set to the header element, if one exists
}
},
- _updateAriaLabelledBy: function() {
- var header = Polymer.dom(this).querySelector('h2');
- if (!header) {
- this.removeAttribute('aria-labelledby');
- return;
- }
- var headerId = header.getAttribute('id');
- if (headerId && this.getAttribute('aria-labelledby') === headerId) {
- return;
- }
- // set aria-describedBy to the header element
- var labelledById;
- if (headerId) {
- labelledById = headerId;
- } else {
- labelledById = 'paper-dialog-header-' + new Date().getUTCMilliseconds();
- header.setAttribute('id', labelledById);
- }
- this.setAttribute('aria-labelledby', labelledById);
- },
-
_updateClosingReasonConfirmed: function(confirmed) {
this.closingReason = this.closingReason || {};
this.closingReason.confirmed = confirmed;

Powered by Google App Engine
This is Rietveld 408576698