Remove lines of CSS which have invalid property values.

Remove lines of CSS which have invalid property values.

BUG=243208
R=dbeam@chromium.org

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=202799

[commit-bot: I haz the power, 2013-05-23 06:46:48]

No comments yet.

[umar.hansa, 2013-05-23 06:49:48]

dbeam: Please review when you get a chance, thanks.

[umar.hansa, 2013-05-25 14:02:24]

The lines of invalid CSS have been removed.

[PhistucK, 2013-05-25 15:52:53]

Just make sure you update the commit log according to the new change.

Thank you!

[umar.hansa, 2013-05-25 20:39:42]

I think I have! $ git log shows this:

commit 5075acc7ccba530da17b0f5d11e61695cdef4b19
Author: Umar Hansa <umar.hansa@gmail.com>
Date:   Sat May 25 14:57:13 2013 +0100

    As per this comment:
https://code.google.com/p/chromium/issues/detail?id=243208#c4 we will remove the
lines which would have been ignored anyway.

Which correlates to Patch 2, hopefully that's correct, let me know if anything
else is needed, thanks.

[PhistucK, 2013-05-26 16:02:52]

On 2013/05/25 20:39:42, umar.hansa wrote:
> I think I have! $ git log shows this:
> 
> commit 5075acc7ccba530da17b0f5d11e61695cdef4b19
> Author: Umar Hansa <mailto:umar.hansa@gmail.com>
> Date:   Sat May 25 14:57:13 2013 +0100
> 
>     As per this comment:
> https://code.google.com/p/chromium/issues/detail?id=243208#c4 we will remove
the
> lines which would have been ignored anyway.
> 
> Which correlates to Patch 2, hopefully that's correct, let me know if anything
> else is needed, thanks.

What I see is the Rietveld description at
https://codereview.chromium.org/15823004/ - it is the old one. I am not sure how
to change it, maybe others know.

[umar.hansa, 2013-05-26 16:05:59]

On 2013/05/26 16:02:52, PhistucK wrote:
> On 2013/05/25 20:39:42, umar.hansa wrote:
> > I think I have! $ git log shows this:
> > 
> > commit 5075acc7ccba530da17b0f5d11e61695cdef4b19
> > Author: Umar Hansa <mailto:umar.hansa@gmail.com>
> > Date:   Sat May 25 14:57:13 2013 +0100
> > 
> >     As per this comment:
> > https://code.google.com/p/chromium/issues/detail?id=243208#c4 we will remove
> the
> > lines which would have been ignored anyway.
> > 
> > Which correlates to Patch 2, hopefully that's correct, let me know if
anything
> > else is needed, thanks.
> 
> What I see is the Rietveld description at
> https://codereview.chromium.org/15823004/ - it is the old one. I am not sure
how
> to change it, maybe others know.

I have updated the description, thanks.

[Dan Beam, 2013-05-29 03:13:10]

lgtm

I know it's strange to take this route, it's annoying to introduce regressions
from cleanup bugs...

[commit-bot: I haz the power, 2013-05-29 03:13:14]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/umar.hansa@gmail.com/15823004/4001

[commit-bot: I haz the power, 2013-05-29 07:44:16]

Change committed as 202799