[DevTools] Rendering option description cleanup

third_party/WebKit/Source/devtools/front_end/main/RenderingOptions.js

 /*
  * Copyright (C) 2013 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 24 matching lines @@
   constructor() {
     super(true);
     this.registerRequiredCSS('main/renderingOptions.css');
 
     /** @type {!Map.<string, !Element>} */
     this._settings = new Map();
 
     var options = [
       {
         label: Common.UIString('Paint Flashing'),
-        subtitle: Common.UIString('Highlights areas of the page that need to be repainted'),
+        subtitle: Common.UIString('Highlights areas of the page (green) that need to be repainted'),
         setterName: 'setShowPaintRects'
       },
       {
         label: Common.UIString('Layer Borders'),
         subtitle: Common.UIString('Shows layer borders (orange/olive) and tiles (cyan)'),
         setterName: 'setShowDebugBorders'
       },
       {
         label: Common.UIString('FPS Meter'),
         subtitle: Common.UIString('Plots frames per second, frame rate distribution, and GPU memory'),
         setterName: 'setShowFPSCounter'
       },
       {
         label: Common.UIString('Scrolling Performance Issues'),
-        subtitle: Common.UIString('Shows areas of the page that slow down scrolling'),
-        setterName: 'setShowScrollBottleneckRects',
-        tooltip:
-            'Touch and mousewheel event listeners can delay scrolling.\nSome areas need to repaint their content when scrolled.'
+        subtitle: Common.UIString('Highlights elements (teal) that can slow down scrolling, including touch & wheel event handlers and other main-thread scrolling situations.'),
+        setterName: 'setShowScrollBottleneckRects'
       }
     ];
     for (var i = 0; i < options.length; i++)
       this._appendCheckbox(options[i].label, options[i].setterName, options[i].subtitle, options[i].tooltip);

[dgozman, 2016/12/12 22:50:19]

Remove tooltip from here.

[phulce, 2016/12/12 23:07:19]

Done.

 
     this.contentElement.createChild('div').classList.add('panel-section-separator');
 
     var cssMediaSubtitle = Common.UIString('Forces media type for testing print and screen styles');
     var checkboxLabel = createCheckboxLabel(Common.UIString('Emulate CSS Media'), false, cssMediaSubtitle);
     this._mediaCheckbox = checkboxLabel.checkboxElement;
     this._mediaCheckbox.addEventListener('click', this._mediaToggled.bind(this), false);
     this.contentElement.appendChild(checkboxLabel);
 
     var mediaRow = this.contentElement.createChild('div', 'media-row');
@@ skipping 14 matching lines @@
       Main.RenderingOptionsView._instanceObject = new Main.RenderingOptionsView();
     return Main.RenderingOptionsView._instanceObject;
   }
 
   /**
    * @param {string} label
    * @param {string} setterName
    * @param {string=} subtitle
    * @param {string=} tooltip
    */
   _appendCheckbox(label, setterName, subtitle, tooltip) {

[dgozman, 2016/12/12 22:50:19]

And from here.

[phulce, 2016/12/12 23:07:19]

Done.

     var checkboxLabel = createCheckboxLabel(label, false, subtitle);
     this._settings.set(setterName, checkboxLabel.checkboxElement);
     checkboxLabel.checkboxElement.addEventListener('click', this._settingToggled.bind(this, setterName));
     if (tooltip)
       checkboxLabel.title = tooltip;
     this.contentElement.appendChild(checkboxLabel);
   }
 
   /**
    * @param {string} setterName
@@ skipping 34 matching lines @@
       cssModel.mediaQueryResultChanged();
   }
 
   /**
    * @override
    * @param {!SDK.Target} target
    */
   targetRemoved(target) {
   }
 };