DevTools: Switching to custom user agent no longer clears preexisting value

third_party/WebKit/Source/devtools/front_end/network/NetworkConfigView.js

 // Copyright (c) 2015 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.
 
 /**
  * @constructor
  * @extends {WebInspector.VBox}
  */
 WebInspector.NetworkConfigView = function()
 {
@@ skipping 111 matching lines @@
     function userAgentSelected()
     {
         var value = userAgentSelectElement.options[userAgentSelectElement.selectedIndex].value;
         if (value !== customOverride.value) {
             userAgentSetting.set(value);
             otherUserAgentElement.value = value;
             otherUserAgentElement.title = value;
             otherUserAgentElement.readOnly = true;
         } else {
             otherUserAgentElement.readOnly = false;
-            otherUserAgentElement.value = "";
             otherUserAgentElement.focus();
         }
     }
 
     function settingChanged()
     {
         var value = userAgentSetting.get();
         var options = userAgentSelectElement.options;
         var selectionRestored = false;
         for (var i = 0; i < options.length; ++i) {
@@ skipping 16 matching lines @@
 
     function textKeyDown(event)
     {
         if (isEnterKey(event))
             textChanged();
     }
 
     function textDoubleClicked()
     {
         if (userAgentSelectElement.selectedIndex === 0)
-            return;

[lushnikov, 2016/04/26 16:39:17]

semicolon is actually needed after the "return"

[luoe, 2016/04/26 23:11:42]

Done.

+            return
         userAgentSelectElement.selectedIndex = 0;
         userAgentSelected();
     }
 
     function textChanged()
     {
         if (userAgentSetting.get() !== otherUserAgentElement.value) {
             userAgentSetting.set(otherUserAgentElement.value);
             settingChanged();
         }
@@ skipping 108 matching lines @@
      * @param {!WebInspector.Context} context
      * @param {string} actionId
      * @return {boolean}
      */
     handleAction: function(context, actionId)
     {
         WebInspector.inspectorView.showViewInDrawer("network.config");
         return true;
     }
 }