Revise accessibility for Date/TimeView.

chrome/browser/resources/chromeos/chromevox/cvox2/background/background.js

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 /**
  * @fileoverview The entry point for all ChromeVox2 related code for the
  * background page.
  */
 
 goog.provide('Background');
@@ skipping 617 matching lines @@
         cvox.ChromeVox.tts.speak(
             announce, cvox.QueueMode.FLUSH,
             cvox.AbstractTts.PERSONALITY_ANNOTATION);
         return false;
       case 'cyclePunctuationEcho':
         cvox.ChromeVox.tts.speak(Msgs.getMsg(
             global.backgroundTts.cyclePunctuationEcho()),
                        cvox.QueueMode.FLUSH);
         return false;
       case 'speakTimeAndDate':
-        var output = new Output();
-        var dateTime = new Date();
-        output.withString(
-            dateTime.toLocaleTimeString() +
-                ', ' + dateTime.toLocaleDateString()).go();
+        chrome.automation.getDesktop(function(d) {
+          // First, try speaking the on-screen time.
+          var allTime = d.findAll({role: RoleType.time});

[dmazzoni, 2016/05/06 15:53:14]

Not for this change, but I think it'd be useful if we
could do a findAll but limit to one tree.

[David Tseng, 2016/05/06 20:40:14]

Agreed; in this case, it would involve a leaf predicate that restricts
descending into rootWebArea nodes and a root predicate for desktop (and
descending into an initial subtree). (a tree "cutout"). I have a cl to do this
in the works.

+          allTime.filter(function(t) {
+            return t.root.role == RoleType.desktop;
+          });
+
+          var timeString = '';
+          allTime.forEach(function(t) {
+            if (t.name)
+              timeString = t.name;
+          });
+          if (timeString) {
+            cvox.ChromeVox.tts.speak(timeString, cvox.QueueMode.FLUSH);
+          } else {
+            // Fallback to the old way of speaking time.
+            var output = new Output();
+            var dateTime = new Date();
+            output.withString(
+                dateTime.toLocaleTimeString() +
+                    ', ' + dateTime.toLocaleDateString()).go();
+          }
+        });
         return false;
       case 'readCurrentTitle':
         var target = this.currentRange_.start.node;
         var output = new Output();
 
         if (target.root.role == RoleType.rootWebArea) {
           // Web.
           target = target.root;
           output.withString(target.name || target.docUrl);
         } else {
@@ skipping 358 matching lines @@
     return glob.replace(/[.+^$(){}|[\]\\]/g, '\\$&')
         .replace(/\*/g, '.*')
         .replace(/\?/g, '.');
   }).join('|') + ')$');
 };
 
 /** @type {Background} */
 global.backgroundObj = new Background();
 
 });  // goog.scope