Added UI to enable debugging features on CrOS

chrome/browser/resources/chromeos/login/oobe_screen_network.js

 // Copyright (c) 2012 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 Oobe network screen implementation.
  */
 
 login.createScreen('NetworkScreen', 'connect', function() {
   return {
@@ skipping 18 matching lines @@
       Oobe.setupSelect($('keyboard-select'),
                        loadTimeData.getValue('inputMethodsList'),
                        'networkOnInputMethodChanged');
 
       Oobe.setupSelect($('timezone-select'),
                        loadTimeData.getValue('timezoneList'),
                        'networkOnTimezoneChanged');
 
       this.dropdown_ = $('networks-list');
       cr.ui.DropDown.decorate(this.dropdown_);
+
+      $('connect-debugging-features-link').addEventListener('click',
+        this.handleDeveloperFeaturesLinkClick_.bind(this));
+      $('connect-debugging-features-link').addEventListener('keyup',
+        function(event) {
+          if (event.keyCode == 32)

[xiyuan, 2014/10/30 03:52:08]

Seems like we need a button that responds to space key rather than a link. We
used to have css to make a button looks like a link but somehow I am no longer
able to find it. :(

[zel, 2014/10/31 01:22:26]

I've just copies what we already do for accessibility link.

[xiyuan, 2014/10/31 03:36:01]

Yep, let's just keep it as is.

+            this.handleDeveloperFeaturesLinkClick_(event);
+        }
+      );
     },
 
     onBeforeShow: function(data) {
       cr.ui.DropDown.show('networks-list', true, -1);
+      this.classList.remove('connect-debugging-view');
+      if ('isDeveloperMode' in data && data['isDeveloperMode'])
+        this.classList.add('connect-debugging-view');

[xiyuan, 2014/10/30 03:52:08]

nit: this.classList.toggle('connect-debugging-view', 'isDeveloperMode' in data
&& data['isDeveloperMode']);

[zel, 2014/10/31 01:22:26]

Done.

     },
 
     onBeforeHide: function() {
       cr.ui.DropDown.hide('networks-list');
       this.enableContinueButton(false);
     },
 
     /**
      * Header text of the screen.
      * @type {string}
@@ skipping 24 matching lines @@
     },
 
     /**
      * Returns a control which should receive an initial focus.
      */
     get defaultControl() {
       return $('language-select');
     },
 
     /**
+     * Enable developer features link handler.
+     */
+    handleDeveloperFeaturesLinkClick_: function() {
+      chrome.send('toggleEnableDebuggingScreen');
+    },
+
+    /**
      * Enables/disables continue button.
      * @param {boolean} enable Should the button be enabled?
      */
     enableContinueButton: function(enable) {
       $('continue-button').disabled = !enable;
     },
 
     /**
      * Sets the current input method.
      * @param {string} inputMethodId The ID of the input method to select.
@@ skipping 25 matching lines @@
       error.appendChild(messageDiv);
       error.setAttribute('role', 'alert');
 
       $('bubble').showContentForElement($('networks-list'),
                                         cr.ui.Bubble.Attachment.BOTTOM,
                                         error);
     }
   };
 });