Add attribute selectors to the SelectorCountScaling performance test.

Add attribute selectors to the SelectorCountScaling performance test.

BUG=352300
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=169783

[ojan, 2014-03-21 23:13:48]

lgtm

[commit-bot: I haz the power, 2014-03-21 23:13:53]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/chrishtr@chromium.org/209153002/20001

[ojan, 2014-03-21 23:14:11]

lgtm


On Fri, Mar 21, 2014 at 3:58 PM, <chrishtr@chromium.org> wrote:

> Reviewers: ojan,
>
> Description:
> Add attribute selectors to the SelectorCountScaling performance test.
>
> BUG=352300
>
> Please review this at https://codereview.chromium.org/209153002/
>
> SVN Base: svn://svn.chromium.org/blink/trunk
>
> Affected files (+11, -2 lines):
>   M PerformanceTests/CSS/SelectorCountScaling.html
>
>
> Index: PerformanceTests/CSS/SelectorCountScaling.html
> diff --git a/PerformanceTests/CSS/SelectorCountScaling.html
> b/PerformanceTests/CSS/SelectorCountScaling.html
> index 68a67fe827caa9cf3ff1b93f9e17024f7d44a6f2..
> 881b6da5f6cf9eb82b247667ba3e9141dea6ff6f 100644
> --- a/PerformanceTests/CSS/SelectorCountScaling.html
> +++ b/PerformanceTests/CSS/SelectorCountScaling.html
> @@ -17,18 +17,27 @@ function insertStyleSheet(css)
>      return styleElement;
>  }
>
> -function cssStr(count) {
> +function cssStrWithClassSelectors(count) {
>      var selector = '.a' + count + ' .b' + count + ' { color: red } ';
>      // Intentionally repeat the same classes many times, to test
> duplicate optimizations.
>      return selector + selector + selector + selector + selector +
> selector + selector;
>  }
>
> +function cssStrWithAttributeSelectors(count) {
> +    var selector = '[attrA' + count + '="1"]' + ' [attrB' + count +
> '="1"]' + ' { color: red } ';
> +    // Intentionally repeat the same classes many times, to test
> duplicate optimizations.
> +    return selector + selector + selector + selector + selector +
> selector + selector;
> +}
> +
>  function runFunction()
>  {
>      var numRules = 1000;
>      var arr = new Array(numRules);
>      for (var i = 0 ; i < numRules; i++) {
> -        arr[i] = cssStr(i);
> +        arr[i] = cssStrWithClassSelectors(i);
> +    }
> +    for (var i = 0 ; i < numRules; i++) {
> +        arr[numRules + i] = cssStrWithAttributeSelectors(i);
>      }
>      var styleElement = insertStyleSheet(arr.join(' '));
>
>
>
>

To unsubscribe from this group and stop receiving emails from it, send an email
to blink-reviews+unsubscribe@chromium.org.

[commit-bot: I haz the power, 2014-03-22 04:26:51]

Change committed as 169783