MD History: Improve dom-if usage in history-item

MD History: Improve dom-if usage in history-item

This adds a dom-if to the bookmark star, which is rarely shown and takes
time to stamp. Also removes a dom-if from a very light div which did not
need the conditional stamping.

In artificial tests, this saves about 10% of the time to stamp a list of
history-items. This should correspond to a small, 1-2% improvement on the
main history page.

BUG=629710
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Committed: https://crrev.com/534998a0abe5f0a40cb87ead60162090101ce43f
Cr-Commit-Position: refs/heads/master@{#410313}

[tsergeant, 2016-08-04 05:29:03]

Description was changed from

==========
MD History: Use dom-if more sensibly in history-item

This adds a dom-if to the bookmark star, which is rarely shown and takes
time to stamp. Also removes a dom-if from a very light div which did not
need the conditional stamping.

In aritifial tests, this saves about 10% of the time to stamp a list of
history-items. This should correspond to a small, 1-2% improvement on the
main history page.

BUG=629710
==========

to

==========
MD History: Use dom-if more sensibly in history-item

This adds a dom-if to the bookmark star, which is rarely shown and takes
time to stamp. Also removes a dom-if from a very light div which did not
need the conditional stamping.

In aritifial tests, this saves about 10% of the time to stamp a list of
history-items. This should correspond to a small, 1-2% improvement on the
main history page.

BUG=629710
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[tsergeant, 2016-08-04 05:48:53]

Description was changed from

==========
MD History: Use dom-if more sensibly in history-item

This adds a dom-if to the bookmark star, which is rarely shown and takes
time to stamp. Also removes a dom-if from a very light div which did not
need the conditional stamping.

In aritifial tests, this saves about 10% of the time to stamp a list of
history-items. This should correspond to a small, 1-2% improvement on the
main history page.

BUG=629710
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

to

==========
MD History: Improve dom-if usage in history-item

This adds a dom-if to the bookmark star, which is rarely shown and takes
time to stamp. Also removes a dom-if from a very light div which did not
need the conditional stamping.

In aritifial tests, this saves about 10% of the time to stamp a list of
history-items. This should correspond to a small, 1-2% improvement on the
main history page.

BUG=629710
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[tsergeant, 2016-08-04 05:49:03]

Description was changed from

==========
MD History: Improve dom-if usage in history-item

This adds a dom-if to the bookmark star, which is rarely shown and takes
time to stamp. Also removes a dom-if from a very light div which did not
need the conditional stamping.

In aritifial tests, this saves about 10% of the time to stamp a list of
history-items. This should correspond to a small, 1-2% improvement on the
main history page.

BUG=629710
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

to

==========
MD History: Improve dom-if usage in history-item

This adds a dom-if to the bookmark star, which is rarely shown and takes
time to stamp. Also removes a dom-if from a very light div which did not
need the conditional stamping.

In aritifial tests, this saves about 10% of the time to stamp a list of
history-items. This should correspond to a small, 1-2% improvement on the
main history page.

BUG=629710
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[tsergeant, 2016-08-04 05:49:45]

The CQ bit was checked by tsergeant@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-08-04 05:50:06]

Dry run: 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, 2016-08-04 06:31:56]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-08-04 06:31:57]

Dry run: Try jobs failed on following builders:
  linux_chromium_asan_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[tsergeant, 2016-08-05 01:50:35]

Patchset #2 (id:20001) has been deleted

[tsergeant, 2016-08-08 04:19:28]

tsergeant@chromium.org changed reviewers:
+ calamity@chromium.org

[tsergeant, 2016-08-08 04:19:40]

The CQ bit was checked by tsergeant@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-08-08 04:19:50]

Dry run: 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, 2016-08-08 05:43:13]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-08-08 05:43:14]

Dry run: This issue passed the CQ dry run.

[calamity, 2016-08-08 06:34:05]

lgtm

https://codereview.chromium.org/2215613002/diff/40001/chrome/browser/resource...
File chrome/browser/resources/md_history/grouped_list.html (left):

https://codereview.chromium.org/2215613002/diff/40001/chrome/browser/resource...
chrome/browser/resources/md_history/grouped_list.html:1: <link rel="import"
href="chrome://resources/html/polymer.html">
CL comment: s/artifial/artificial/

[tsergeant, 2016-08-08 07:36:27]

Description was changed from

==========
MD History: Improve dom-if usage in history-item

This adds a dom-if to the bookmark star, which is rarely shown and takes
time to stamp. Also removes a dom-if from a very light div which did not
need the conditional stamping.

In aritifial tests, this saves about 10% of the time to stamp a list of
history-items. This should correspond to a small, 1-2% improvement on the
main history page.

BUG=629710
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

to

==========
MD History: Improve dom-if usage in history-item

This adds a dom-if to the bookmark star, which is rarely shown and takes
time to stamp. Also removes a dom-if from a very light div which did not
need the conditional stamping.

In artificial tests, this saves about 10% of the time to stamp a list of
history-items. This should correspond to a small, 1-2% improvement on the
main history page.

BUG=629710
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[tsergeant, 2016-08-08 07:36:44]

https://codereview.chromium.org/2215613002/diff/40001/chrome/browser/resource...
File chrome/browser/resources/md_history/grouped_list.html (left):

https://codereview.chromium.org/2215613002/diff/40001/chrome/browser/resource...
chrome/browser/resources/md_history/grouped_list.html:1: <link rel="import"
href="chrome://resources/html/polymer.html">
On 2016/08/08 06:34:05, calamity wrote:
> CL comment: s/artifial/artificial/

Done.

[tsergeant, 2016-08-08 07:36:52]

The CQ bit was checked by tsergeant@chromium.org

[commit-bot: I haz the power, 2016-08-08 07:37:02]

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, 2016-08-08 07:40:28]

Description was changed from

==========
MD History: Improve dom-if usage in history-item

This adds a dom-if to the bookmark star, which is rarely shown and takes
time to stamp. Also removes a dom-if from a very light div which did not
need the conditional stamping.

In artificial tests, this saves about 10% of the time to stamp a list of
history-items. This should correspond to a small, 1-2% improvement on the
main history page.

BUG=629710
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

to

==========
MD History: Improve dom-if usage in history-item

This adds a dom-if to the bookmark star, which is rarely shown and takes
time to stamp. Also removes a dom-if from a very light div which did not
need the conditional stamping.

In artificial tests, this saves about 10% of the time to stamp a list of
history-items. This should correspond to a small, 1-2% improvement on the
main history page.

BUG=629710
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[commit-bot: I haz the power, 2016-08-08 07:40:29]

Committed patchset #2 (id:40001)

[commit-bot: I haz the power, 2016-08-08 07:42:09]

Description was changed from

==========
MD History: Improve dom-if usage in history-item

This adds a dom-if to the bookmark star, which is rarely shown and takes
time to stamp. Also removes a dom-if from a very light div which did not
need the conditional stamping.

In artificial tests, this saves about 10% of the time to stamp a list of
history-items. This should correspond to a small, 1-2% improvement on the
main history page.

BUG=629710
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

to

==========
MD History: Improve dom-if usage in history-item

This adds a dom-if to the bookmark star, which is rarely shown and takes
time to stamp. Also removes a dom-if from a very light div which did not
need the conditional stamping.

In artificial tests, this saves about 10% of the time to stamp a list of
history-items. This should correspond to a small, 1-2% improvement on the
main history page.

BUG=629710
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Committed: https://crrev.com/534998a0abe5f0a40cb87ead60162090101ce43f
Cr-Commit-Position: refs/heads/master@{#410313}
==========

[commit-bot: I haz the power, 2016-08-08 07:42:12]

Patchset 2 (id:??) landed as
https://crrev.com/534998a0abe5f0a40cb87ead60162090101ce43f
Cr-Commit-Position: refs/heads/master@{#410313}