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

Side by Side Diff: third_party/WebKit/Source/modules/accessibility/AXNodeObject.cpp

Issue 2526133003: [CANCELED] Ensure clean style for hasEditableLevel when it doesn't break other stuff (Closed)
Patch Set: Fri Nov 25 14:48:44 JST 2016 Created 4 years 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 unified diff | Download patch
OLDNEW
1 /* 1 /*
2 * Copyright (C) 2012, Google Inc. All rights reserved. 2 * Copyright (C) 2012, Google Inc. All rights reserved.
3 * 3 *
4 * Redistribution and use in source and binary forms, with or without 4 * Redistribution and use in source and binary forms, with or without
5 * modification, are permitted provided that the following conditions 5 * modification, are permitted provided that the following conditions
6 * are met: 6 * are met:
7 * 7 *
8 * 1. Redistributions of source code must retain the above copyright 8 * 1. Redistributions of source code must retain the above copyright
9 * notice, this list of conditions and the following disclaimer. 9 * notice, this list of conditions and the following disclaimer.
10 * 2. Redistributions in binary form must reproduce the above copyright 10 * 2. Redistributions in binary form must reproduce the above copyright
(...skipping 1049 matching lines...) Expand 10 before | Expand all | Expand 10 after
1060 if (!isPresentational() && ariaRoleAttribute() != UnknownRole && 1060 if (!isPresentational() && ariaRoleAttribute() != UnknownRole &&
1061 ancestorExposesActiveDescendant()) 1061 ancestorExposesActiveDescendant())
1062 return true; 1062 return true;
1063 1063
1064 // NOTE: It would be more accurate to ask the document whether 1064 // NOTE: It would be more accurate to ask the document whether
1065 // setFocusedNode() would do anything. For example, setFocusedNode() will do 1065 // setFocusedNode() would do anything. For example, setFocusedNode() will do
1066 // nothing if the current focused node will not relinquish the focus. 1066 // nothing if the current focused node will not relinquish the focus.
1067 if (isDisabledFormControl(node)) 1067 if (isDisabledFormControl(node))
1068 return false; 1068 return false;
1069 1069
1070 AutoReset<bool> inAXScope(checkingEditableStyleFromAXIndicator(), true);
1070 return node->isElementNode() && toElement(node)->supportsFocus(); 1071 return node->isElementNode() && toElement(node)->supportsFocus();
1071 } 1072 }
1072 1073
1073 bool AXNodeObject::canSetValueAttribute() const { 1074 bool AXNodeObject::canSetValueAttribute() const {
1074 if (equalIgnoringCase(getAttribute(aria_readonlyAttr), "true")) 1075 if (equalIgnoringCase(getAttribute(aria_readonlyAttr), "true"))
1075 return false; 1076 return false;
1076 1077
1077 if (isProgressIndicator() || isSlider()) 1078 if (isProgressIndicator() || isSlider())
1078 return true; 1079 return true;
1079 1080
(...skipping 1815 matching lines...) Expand 10 before | Expand all | Expand 10 after
2895 } 2896 }
2896 return placeholder; 2897 return placeholder;
2897 } 2898 }
2898 2899
2899 DEFINE_TRACE(AXNodeObject) { 2900 DEFINE_TRACE(AXNodeObject) {
2900 visitor->trace(m_node); 2901 visitor->trace(m_node);
2901 AXObject::trace(visitor); 2902 AXObject::trace(visitor);
2902 } 2903 }
2903 2904
2904 } // namespace blink 2905 } // namespace blink
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698