MD History: Update timestamp hover text when item changes

MD History: Update timestamp hover text when item changes

This fixes an issue where the time title would be stuck on the first
value it was set to, never updating as the list was scrolled and
changed.

The end result is that the code matches patchset 2 of
https://codereview.chromium.org/2454303002/.

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

Review-Url: https://codereview.chromium.org/2962133002
Cr-Commit-Position: refs/heads/master@{#483630}
Committed: https://chromium.googlesource.com/chromium/src/+/47c2896ff9bad0082bf8abddd0e55bfdd843a959

[tsergeant, 2017-06-29 05:39:14]

Description was changed from

==========
MD History: Update time hover text when item changes

This fixes an issue where the time title would be stuck on the first
value it was set to, never updating as the list was scrolled and
changed.

BUG=675056
==========

to

==========
MD History: Update time hover text when item changes

This fixes an issue where the time title would be stuck on the first
value it was set to, never updating as the list was scrolled and
changed.

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

[tsergeant, 2017-06-29 05:42:38]

Description was changed from

==========
MD History: Update time hover text when item changes

This fixes an issue where the time title would be stuck on the first
value it was set to, never updating as the list was scrolled and
changed.

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

to

==========
MD History: Update time hover text when item changes

This fixes an issue where the time title would be stuck on the first
value it was set to, never updating as the list was scrolled and
changed.

The end result is that the code matches patchset 2 of
https://codereview.chromium.org/2454303002/.

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

[tsergeant, 2017-06-29 05:45:38]

Description was changed from

==========
MD History: Update time hover text when item changes

This fixes an issue where the time title would be stuck on the first
value it was set to, never updating as the list was scrolled and
changed.

The end result is that the code matches patchset 2 of
https://codereview.chromium.org/2454303002/.

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

to

==========
MD History: Update timestamp hover text when item changes

This fixes an issue where the time title would be stuck on the first
value it was set to, never updating as the list was scrolled and
changed.

The end result is that the code matches patchset 2 of
https://codereview.chromium.org/2454303002/.

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

[tsergeant, 2017-06-29 05:46:28]

Patchset #1 (id:1) has been deleted

[tsergeant, 2017-06-29 05:46:35]

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

[commit-bot: I haz the power, 2017-06-29 05:46:45]

Dry run: CQ is trying da patch.

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

[tsergeant, 2017-06-29 05:52:27]

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

[tsergeant, 2017-06-29 05:52:28]

PTAL.

I can't think of any way to fix this bug without ditching the laziness. Let me
know if you have any ideas.

[calamity, 2017-06-29 06:46:51]

Well you _could_ add the listener back on item change.

[commit-bot: I haz the power, 2017-06-29 06:47:13]

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

[commit-bot: I haz the power, 2017-06-29 06:47:14]

Dry run: Try jobs failed on following builders:
  win_chromium_x64_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_x64_...)

[tsergeant, 2017-06-29 07:43:10]

Hmm, okay, yeah.

Here's a version that should have no load-time performance impact.

[tsergeant, 2017-06-29 07:43:10]

Hmm, okay, yeah.

Here's a version that should have no load-time performance impact.

[calamity, 2017-06-30 03:21:00]

lgtm.

lgtm.

[tsergeant, 2017-06-30 03:33:56]

The CQ bit was checked by tsergeant@chromium.org

[commit-bot: I haz the power, 2017-06-30 03:34:09]

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-06-30 04:45:36]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1498793636284930,
"parent_rev": "9de3871654a9e7d5b9de75eed7f7c158910e1dd0", "commit_rev":
"47c2896ff9bad0082bf8abddd0e55bfdd843a959"}

[commit-bot: I haz the power, 2017-06-30 04:46:11]

Description was changed from

==========
MD History: Update timestamp hover text when item changes

This fixes an issue where the time title would be stuck on the first
value it was set to, never updating as the list was scrolled and
changed.

The end result is that the code matches patchset 2 of
https://codereview.chromium.org/2454303002/.

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

to

==========
MD History: Update timestamp hover text when item changes

This fixes an issue where the time title would be stuck on the first
value it was set to, never updating as the list was scrolled and
changed.

The end result is that the code matches patchset 2 of
https://codereview.chromium.org/2454303002/.

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

Review-Url: https://codereview.chromium.org/2962133002
Cr-Commit-Position: refs/heads/master@{#483630}
Committed:
https://chromium.googlesource.com/chromium/src/+/47c2896ff9bad0082bf8abddd0e5...
==========

[commit-bot: I haz the power, 2017-06-30 04:46:12]

Committed patchset #2 (id:40001) as
https://chromium.googlesource.com/chromium/src/+/47c2896ff9bad0082bf8abddd0e5...