MD History: Move the card sizing CSS to apply to the iron-list

MD History: Move the card sizing CSS to apply to the iron-list

This fixes an issue where it was possible to click on the margins of
the list to select items. These margins are now part of the list itself,
and individual items only occupy the exact horizontal space they need.
This change should have no visual impact on the layout of the list.

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

Committed: https://crrev.com/af77c9922af8486fc555b268a0eaeeb8d6055c57
Cr-Commit-Position: refs/heads/master@{#422043}

[tsergeant, 2016-09-30 04:45:44]

Description was changed from

==========
MD History: Move the card sizing CSS to apply to the iron-list

This fixes an issue where it was possible to click on the margins of
the list to select items. These margins are now part of the list itself,
and individual items only occupy the exact horizontal space they need.
This change should have no visual impact on the layout of the list.

BUG=651556
==========

to

==========
MD History: Move the card sizing CSS to apply to the iron-list

This fixes an issue where it was possible to click on the margins of
the list to select items. These margins are now part of the list itself,
and individual items only occupy the exact horizontal space they need.
This change should have no visual impact on the layout of the list.

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

[tsergeant, 2016-09-30 04:47:57]

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

[tsergeant, 2016-09-30 04:47:58]

https://codereview.chromium.org/2378903004/diff/1/chrome/browser/resources/md...
File chrome/browser/resources/md_history/history_item.js (right):

https://codereview.chromium.org/2378903004/diff/1/chrome/browser/resources/md...
chrome/browser/resources/md_history/history_item.js:127:
this.$['main-container'], null, new FocusRowDelegate(this));
I don't think there's any problems with moving where this is applied to

[calamity, 2016-09-30 04:58:45]

lgtm

https://codereview.chromium.org/2378903004/diff/1/chrome/browser/resources/md...
File chrome/browser/resources/md_history/history_item.js (right):

https://codereview.chromium.org/2378903004/diff/1/chrome/browser/resources/md...
chrome/browser/resources/md_history/history_item.js:127:
this.$['main-container'], null, new FocusRowDelegate(this));
On 2016/09/30 04:47:58, tsergeant wrote:
> I don't think there's any problems with moving where this is applied to

Acknowledged.

https://codereview.chromium.org/2378903004/diff/1/chrome/browser/resources/md...
File chrome/browser/resources/md_history/history_list.html (right):

https://codereview.chromium.org/2378903004/diff/1/chrome/browser/resources/md...
chrome/browser/resources/md_history/history_list.html:25: }
This can be removed.

[tsergeant, 2016-09-30 05:11:28]

https://codereview.chromium.org/2378903004/diff/1/chrome/browser/resources/md...
File chrome/browser/resources/md_history/history_list.html (right):

https://codereview.chromium.org/2378903004/diff/1/chrome/browser/resources/md...
chrome/browser/resources/md_history/history_list.html:25: }
On 2016/09/30 04:58:45, calamity wrote:
> This can be removed.

whoops, looks like this could have been removed a while ago.

[tsergeant, 2016-09-30 05:11:33]

The CQ bit was checked by tsergeant@chromium.org

[tsergeant, 2016-09-30 05:11:33]

The patchset sent to the CQ was uploaded after l-g-t-m from
calamity@chromium.org
Link to the patchset: https://codereview.chromium.org/2378903004/#ps20001
(title: "Remove unused CSS")

[commit-bot: I haz the power, 2016-09-30 05:11:45]

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-09-30 06:29:11]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-09-30 06:31:25]

Description was changed from

==========
MD History: Move the card sizing CSS to apply to the iron-list

This fixes an issue where it was possible to click on the margins of
the list to select items. These margins are now part of the list itself,
and individual items only occupy the exact horizontal space they need.
This change should have no visual impact on the layout of the list.

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

to

==========
MD History: Move the card sizing CSS to apply to the iron-list

This fixes an issue where it was possible to click on the margins of
the list to select items. These margins are now part of the list itself,
and individual items only occupy the exact horizontal space they need.
This change should have no visual impact on the layout of the list.

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

Committed: https://crrev.com/af77c9922af8486fc555b268a0eaeeb8d6055c57
Cr-Commit-Position: refs/heads/master@{#422043}
==========

[commit-bot: I haz the power, 2016-09-30 06:31:26]

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