Do not auto-open histograms in singular rows.

Do not auto-open histograms in singular rows.

Currently, if there is only a single row, histogram-set-table automatically
opens its Histograms.
This is a UX optimization: we expect users to decide which cells to open and
then click them, so if we know which cells they will want to open (because there
are so few of them to choose from), then we don't have to wait for them to
pick and click, we can just go ahead and automatically open them.

In practice, most metrics produce more than one Histogram, so the single-row
situation only arises in blink_perf.dom tests.
Recent feedback has called into question the value of automatically overloading
the user with the full contents of every Histogram, so this CL removes this UX
optimization.

Review-Url: https://codereview.chromium.org/2745413003
Committed: https://chromium.googlesource.com/external/github.com/catapult-project/catapult/+/0a929d3695a1baaa4a97ca6548e83f55ead6ce22

[benjhayden, 2017-03-14 21:59:57]

benjhayden@chromium.org changed reviewers:
+ eakuefner@chromium.org

[benjhayden, 2017-03-14 21:59:57]

PTAL

[benjhayden, 2017-03-14 22:01:54]

Patchset #1 (id:1) has been deleted

[eakuefner, 2017-03-14 22:02:42]

lgtm

[benjhayden, 2017-03-15 03:12:16]

The CQ bit was checked by benjhayden@chromium.org

[commit-bot: I haz the power, 2017-03-15 03:12:26]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-03-15 03:34:29]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1489547536804690,
"parent_rev": "829e0ebae3ce758647a15d8075368780490ecb60", "commit_rev":
"0a929d3695a1baaa4a97ca6548e83f55ead6ce22"}

[commit-bot: I haz the power, 2017-03-15 03:34:32]

Description was changed from

==========
Do not auto-open histograms in singular rows.

Currently, if there is only a single row, histogram-set-table automatically
opens its Histograms.
This is a UX optimization: we expect users to decide which cells to open and
then click them, so if we know which cells they will want to open (because there
are so few of them to choose from), then we don't have to wait for them to
pick and click, we can just go ahead and automatically open them.

In practice, most metrics produce more than one Histogram, so the single-row
situation only arises in blink_perf.dom tests.
Recent feedback has called into question the value of automatically overloading
the user with the full contents of every Histogram, so this CL removes this UX
optimization.
==========

to

==========
Do not auto-open histograms in singular rows.

Currently, if there is only a single row, histogram-set-table automatically
opens its Histograms.
This is a UX optimization: we expect users to decide which cells to open and
then click them, so if we know which cells they will want to open (because there
are so few of them to choose from), then we don't have to wait for them to
pick and click, we can just go ahead and automatically open them.

In practice, most metrics produce more than one Histogram, so the single-row
situation only arises in blink_perf.dom tests.
Recent feedback has called into question the value of automatically overloading
the user with the full contents of every Histogram, so this CL removes this UX
optimization.

Review-Url: https://codereview.chromium.org/2745413003
Committed:
https://chromium.googlesource.com/external/github.com/catapult-project/catapu...
==========

[commit-bot: I haz the power, 2017-03-15 03:34:33]

Committed patchset #1 (id:20001) as
https://chromium.googlesource.com/external/github.com/catapult-project/catapu...