Fix math support on wikipedia.

Fixmath support on wikipedia.

Wikipedia now assigns a fallback classname to its 'tex' img nodes. ChromeVox, reading the className directly, needs to split the space delimited string.

BUG=448544
TEST=math now reads on wikipedia

Committed: https://crrev.com/4d2da89db85b25313a829e722965a99a9d4e8abe
Cr-Commit-Position: refs/heads/master@{#311529}

[David Tseng, 2015-01-13 22:59:30]

dtseng@chromium.org changed reviewers:
+ dmazzoni@chromium.org

[dmazzoni, 2015-01-14 08:20:34]

lgtm

lgtm with one issue

https://codereview.chromium.org/828373007/diff/1/chrome/browser/resources/chr...
File chrome/browser/resources/chromeos/chromevox/common/dom_util.js (right):

https://codereview.chromium.org/828373007/diff/1/chrome/browser/resources/chr...
chrome/browser/resources/chromeos/chromevox/common/dom_util.js:2450: return
node.className.toLowerCase().split(' ').some(function(className) {
Can you use classList.contains? Or at least, iterate over node.classList rather
than splitting on the string.

[David Tseng, 2015-01-14 18:17:35]

https://codereview.chromium.org/828373007/diff/1/chrome/browser/resources/chr...
File chrome/browser/resources/chromeos/chromevox/common/dom_util.js (right):

https://codereview.chromium.org/828373007/diff/1/chrome/browser/resources/chr...
chrome/browser/resources/chromeos/chromevox/common/dom_util.js:2450: return
node.className.toLowerCase().split(' ').some(function(className) {
On 2015/01/14 08:20:34, dmazzoni wrote:
> Can you use classList.contains?
No; since the current code lower cases the classname.

 Or at least, iterate over node.classList rather
> than splitting on the string.
Done.

[David Tseng, 2015-01-14 18:17:46]

The CQ bit was checked by dtseng@chromium.org

[commit-bot: I haz the power, 2015-01-14 18:18:44]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/828373007/20001

[commit-bot: I haz the power, 2015-01-14 19:14:16]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-01-14 19:15:15]

Patchset 2 (id:??) landed as
https://crrev.com/4d2da89db85b25313a829e722965a99a9d4e8abe
Cr-Commit-Position: refs/heads/master@{#311529}