[MD History] Use box-shadow to render card shadows everywhere.

[MD History] Use box-shadow to render card shadows everywhere.

This CL changes the filter that was used to render the shadow on the
history list into a box shadow to fix scrolling performance on various
systems.

It was also necessary to add a clipping div around the background so that the shadow wouldn't render over other items.

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

Committed: https://crrev.com/986a0334a792a32a5dac3d72c0edfbaceeeff0c7
Cr-Commit-Position: refs/heads/master@{#416807}

[calamity, 2016-08-26 05:44:02]

Description was changed from

==========
[MD History] Use box-shadow to render card shadows everywhere.

This CL changes the filter that was used to render the shadow on the
history list into a box shadow to fix scrolling performance on various
systems.

BUG=641223
==========

to

==========
[MD History] Use box-shadow to render card shadows everywhere.

This CL changes the filter that was used to render the shadow on the
history list into a box shadow to fix scrolling performance on various
systems.

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

[calamity, 2016-08-26 05:58:38]

Patchset #1 (id:1) has been deleted

[calamity, 2016-08-26 05:58:46]

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

[tsergeant, 2016-08-28 23:20:31]

On 2016/08/26 05:58:47, calamity wrote:

I'm seeing quite noticeable lines between items when I patch this in locally.

[calamity, 2016-09-06 04:43:58]

I think this may fix the lines. PTAL.

[tsergeant, 2016-09-06 05:04:26]

This fixes everything for me but some veeeery faint lines at 110%. I'm happy
enough to go with this anyway.

Can you please add a comment to document this, though? There's a lot of magic
numbers whose purpose isn't immediately clear.

https://codereview.chromium.org/2278383002/diff/40001/chrome/browser/resource...
File chrome/browser/resources/md_history/history_item.html (right):

https://codereview.chromium.org/2278383002/diff/40001/chrome/browser/resource...
chrome/browser/resources/md_history/history_item.html:121:
@apply(--shadow-elevation-2dp);
"Is it not a strange fate that we should suffer so much fear and doubt for so
small a thing?"

[calamity, 2016-09-06 06:00:44]

Description was changed from

==========
[MD History] Use box-shadow to render card shadows everywhere.

This CL changes the filter that was used to render the shadow on the
history list into a box shadow to fix scrolling performance on various
systems.

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

to

==========
[MD History] Use box-shadow to render card shadows everywhere.

This CL changes the filter that was used to render the shadow on the
history list into a box shadow to fix scrolling performance on various
systems.

It was also necessary to add a clipping div around the background so that the
shadow wouldn't render over other items.

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

[calamity, 2016-09-06 06:01:02]

https://codereview.chromium.org/2278383002/diff/40001/chrome/browser/resource...
File chrome/browser/resources/md_history/history_item.html (right):

https://codereview.chromium.org/2278383002/diff/40001/chrome/browser/resource...
chrome/browser/resources/md_history/history_item.html:121:
@apply(--shadow-elevation-2dp);
On 2016/09/06 05:04:26, tsergeant wrote:
> "Is it not a strange fate that we should suffer so much fear and doubt for so
> small a thing?"

A box-shadow on a rounded-rect.. A shadow.. ring, if you will.

[tsergeant, 2016-09-06 07:59:16]

lgtm

[tsergeant, 2016-09-06 23:29:44]

The CQ bit was checked by tsergeant@chromium.org

[commit-bot: I haz the power, 2016-09-06 23:29:59]

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-07 00:56:17]

Description was changed from

==========
[MD History] Use box-shadow to render card shadows everywhere.

This CL changes the filter that was used to render the shadow on the
history list into a box shadow to fix scrolling performance on various
systems.

It was also necessary to add a clipping div around the background so that the
shadow wouldn't render over other items.

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

to

==========
[MD History] Use box-shadow to render card shadows everywhere.

This CL changes the filter that was used to render the shadow on the
history list into a box shadow to fix scrolling performance on various
systems.

It was also necessary to add a clipping div around the background so that the
shadow wouldn't render over other items.

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

[commit-bot: I haz the power, 2016-09-07 00:56:19]

Committed patchset #3 (id:60001)

[commit-bot: I haz the power, 2016-09-07 00:58:09]

Description was changed from

==========
[MD History] Use box-shadow to render card shadows everywhere.

This CL changes the filter that was used to render the shadow on the
history list into a box shadow to fix scrolling performance on various
systems.

It was also necessary to add a clipping div around the background so that the
shadow wouldn't render over other items.

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

to

==========
[MD History] Use box-shadow to render card shadows everywhere.

This CL changes the filter that was used to render the shadow on the
history list into a box shadow to fix scrolling performance on various
systems.

It was also necessary to add a clipping div around the background so that the
shadow wouldn't render over other items.

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

Committed: https://crrev.com/986a0334a792a32a5dac3d72c0edfbaceeeff0c7
Cr-Commit-Position: refs/heads/master@{#416807}
==========

[commit-bot: I haz the power, 2016-09-07 00:58:10]

Patchset 3 (id:??) landed as
https://crrev.com/986a0334a792a32a5dac3d72c0edfbaceeeff0c7
Cr-Commit-Position: refs/heads/master@{#416807}