Clean-up: Change chrome/browser's CSS time units from s to ms.

Clean-up: Change chrome/browser's CSS time units from s to ms if >= 1s duration (according to new web dev style guidance).

R=estade@chromium.org,jhawkins@chromium.org
BUG=None
TEST=Nothing WebUI acts differently, everything's easier to read.

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=111450

[Dan Beam, 2011-11-23 06:29:09]

I used:

  egrep -Rn ' ([0-9]+|[0-9]+\.[0-9]+|\.[0-9])s[^t]' --include=*.html
--include=*.css --include=*.js chrome/browser/resources/ | grep -v Guido | grep
-v 'The 2s'

so make sure to tell me if that's a bad regexp to check for second units in CSS.

[Evan Stade, 2011-11-23 06:31:42]

+jhawkins

lgtm, can you also add a rule to the style guide

http://codereview.chromium.org/8680003/diff/1/chrome/browser/resources/sync_s...
File chrome/browser/resources/sync_setup_overlay.css (right):

http://codereview.chromium.org/8680003/diff/1/chrome/browser/resources/sync_s...
chrome/browser/resources/sync_setup_overlay.css:181: -webkit-transition:
margin-top 330ms ease-out, opacity 660ms ease-out;
333/667 I guess :)

[Dan Beam, 2011-11-23 06:33:54]

http://codereview.chromium.org/8680003/diff/1/chrome/browser/resources/sync_s...
File chrome/browser/resources/sync_setup_overlay.css (right):

http://codereview.chromium.org/8680003/diff/1/chrome/browser/resources/sync_s...
chrome/browser/resources/sync_setup_overlay.css:181: -webkit-transition:
margin-top 330ms ease-out, opacity 660ms ease-out;
On 2011/11/23 06:31:42, Evan Stade wrote:
> 333/667 I guess :)

Hehe, yeah, I saw this - I didn't know if there was the tiniest possibility if
they meant to happen just a little bit before .333 or .667 or not, so I just did
exactly what it was before.

[Dan Beam, 2011-11-23 06:42:00]

> lgtm, can you also add a rule to the style guide

Done.

[James Hawkins, 2011-11-23 16:36:11]

LGTM

[Dan Beam, 2011-11-23 16:37:22]

* wonders how many merge conflicts this will cause

[commit-bot: I haz the power, 2011-11-23 22:46:07]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/dbeam@chromium.org/8680003/4007

[commit-bot: I haz the power, 2011-11-23 23:55:58]

Change committed as 111450