Make app list start page doodle clickable and have alt text.

chrome/browser/ui/webui/app_list/start_page_browsertest.js

 // Copyright 2013 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.
 
 /**
  * TestFixture for kiosk app settings WebUI testing.
  * @extends {testing.Test}
  * @constructor
  */
 function AppListStartPageWebUITest() {}
@@ skipping 14 matching lines @@
   createScriptProcessor: function(bufSize, in_channels, out_channels) {
     return {connect: function(destination) {},
             disconnect: function() {}};
   }
 };
 
 AppListStartPageWebUITest.prototype = {
   __proto__: testing.Test.prototype,
 
   /**
-   * Sample doodle data.
-   */
-  doodleData_: {
-    'ddljson': {
-      'transparent_large_image': {
-        'url': 'doodle.png'
-      }
-    }
-  },
-
-  /**
    * Browser to app launcher start page.
    */
   browsePreload: 'chrome://app-list/',
 
   /**
    * Placeholder for mock speech recognizer.
    */
   speechRecognizer: null,
 
   /**
@@ skipping 72 matching lines @@
     navigator.webkitGetUserMedia = this.mockGetUserMedia_.bind(this);
     this.audioTrackMocks = [mock(MediaStreamTrack), mock(MediaStreamTrack)];
   }
 };
 
 TEST_F('AppListStartPageWebUITest', 'Basic', function() {
   assertEquals(this.browsePreload, document.location.href);
 });
 
 TEST_F('AppListStartPageWebUITest', 'LoadDoodle', function() {
-  var doodle = $('doodle');
-  assertEquals('', doodle.src);
-  appList.startPage.onAppListDoodleUpdated(this.doodleData_,
+  var doodleData = {
+    'ddljson': {
+      'transparent_large_image': {
+        'url': 'doodle.png'
+      },
+      'alt_text': 'Doodle alt text',
+      'target_url': '/target.html'
+    }
+  };
+
+  assertEquals(null, $('doodle'));
+
+  // Load the doodle with a target url and alt text.
+  appList.startPage.onAppListDoodleUpdated(doodleData,
                                            'http://example.com/');

[Matt Giuca, 2015/02/03 07:08:50]

Can you make this http://example.com/x/.

Then you test relative vs absolute paths. (src should be
http://example.com/x/doodle.png, whereas href should be
http://example.com/target.html.)

[calamity, 2015/02/04 03:16:22]

Done.

-  assertEquals('http://example.com/doodle.png', doodle.src);
+  assertNotEquals(null, $('doodle'));
+  assertEquals('http://example.com/doodle.png', $('doodle_image').src);
+  assertEquals(doodleData.ddljson.alt_text, $('doodle_image').title);
+  assertEquals('http://example.com/target.html', $('doodle_link').href);
+
+  // Reload the doodle without a target url and alt text.
+  doodleData.ddljson.alt_text = undefined;
+  doodleData.ddljson.target_url = undefined;
+  appList.startPage.onAppListDoodleUpdated(doodleData,
+                                           'http://example.com/');
+  assertNotEquals(null, $('doodle'));
+  assertEquals('http://example.com/doodle.png', $('doodle_image').src);
+  assertEquals('', $('doodle_image').title);
+  assertEquals(null, $('doodle_link'));
+
+
+  appList.startPage.onAppListDoodleUpdated({},
+                                           'http://example.com/');
+  assertEquals(null, $('doodle'));
 });
 
 TEST_F('AppListStartPageWebUITest', 'SpeechRecognitionState', function() {
   this.mockHandler.expects(once()).setSpeechRecognitionState('READY');
   appList.startPage.onAppListShown();
   this.mockHandler.expects(once()).setSpeechRecognitionState('RECOGNIZING');
   appList.startPage.toggleSpeechRecognition();
   Mock4JS.verifyAllMocks();
   Mock4JS.clearMocksToVerify();
 
@@ skipping 36 matching lines @@
   Mock4JS.verifyAllMocks();
   Mock4JS.clearMocksToVerify();
 
   this.mockHandler.expects(once()).speechResult('test,true');
   this.mockHandler.expects(once()).setSpeechRecognitionState('READY');
   for (var i = 0; i < this.audioTrackMocks.length; ++i) {
     this.audioTrackMocks[i].expects(once()).stop();
   }
   this.sendSpeechResult('test', true);
 });