Uprefix CSS transitions in ui/ and chrome/ styles

Uprefix CSS transitions in ui/ and chrome/ styles

Transitions no longer require webkit prefix.

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

Review-Url: https://codereview.chromium.org/2691883002
Cr-Commit-Position: refs/heads/master@{#451402}
Committed: https://chromium.googlesource.com/chromium/src/+/63f826afd2936707c9c697ce774a65e783041db7

[Eric Willigers, 2017-02-13 02:57:02]

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

[commit-bot: I haz the power, 2017-02-13 02:57:18]

Dry run: 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-02-13 03:09:16]

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

[commit-bot: I haz the power, 2017-02-13 03:09:17]

Dry run: Try jobs failed on following builders:
  cast_shell_linux on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/cast_shell_linu...)
  chromeos_amd64-generic_chromium_compile_only_ng on
master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_amd64-...)
  chromeos_daisy_chromium_compile_only_ng on master.tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_daisy_...)
  linux_chromium_asan_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_chromeos_ozone_rel_ng on master.tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_chromeos_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_tsan_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[Eric Willigers, 2017-02-13 03:17:43]

The changelists
https://codereview.chromium.org/2535573002/
https://codereview.chromium.org/2535493002/
are failing individually.

This changelist is to determine if such CLs would pass when uploaded together.

[Eric Willigers, 2017-02-13 04:41:05]

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

[commit-bot: I haz the power, 2017-02-13 04:41:22]

Dry run: 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-02-13 05:37:22]

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

[commit-bot: I haz the power, 2017-02-13 05:37:23]

Dry run: This issue passed the CQ dry run.

[Eric Willigers, 2017-02-14 23:49:22]

Description was changed from

==========
[WIP] Reduce webkit CSS prefixes in ui/ and chrome/ styles

Animations, transitions and transforms no longer require webkit prefix.

BUG=668650
==========

to

==========
[WIP] Reduce webkit CSS prefixes in ui/ and chrome/ styles

Animations, transitions and transforms no longer require webkit prefix.

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

[Eric Willigers, 2017-02-15 02:55:42]

Patchset #2 (id:20001) has been deleted

[Eric Willigers, 2017-02-15 02:56:07]

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

[commit-bot: I haz the power, 2017-02-15 02:57:28]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Eric Willigers, 2017-02-15 03:47:58]

On 2017/02/13 03:17:43, Eric Willigers wrote:
> The changelists
> https://codereview.chromium.org/2535573002/
> https://codereview.chromium.org/2535493002/
> are failing individually.
> 
> This changelist is to determine if such CLs would pass when uploaded together.


The only changes that haven't already been reviewed in the above changelists are
trivial:

https://codereview.chromium.org/2691883002/diff2/1:40001/chrome/browser/resou...
reorder flex

https://codereview.chromium.org/2691883002/diff2/1:40001/ui/login/screen_cont...
reorder bottom

[Eric Willigers, 2017-02-15 03:49:30]

Description was changed from

==========
[WIP] Reduce webkit CSS prefixes in ui/ and chrome/ styles

Animations, transitions and transforms no longer require webkit prefix.

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

to

==========
Uprefix CSS transitions in ui/ and chrome/ styles

Transitions no longer require webkit prefix.

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

[Eric Willigers, 2017-02-15 03:49:30]

ericwilligers@chromium.org changed reviewers:
+ dbeam@chromium.org

[commit-bot: I haz the power, 2017-02-15 05:20:13]

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

[commit-bot: I haz the power, 2017-02-15 05:20:14]

Dry run: This issue passed the CQ dry run.

[Dan Beam, 2017-02-17 06:36:15]

rs lgtm

[Eric Willigers, 2017-02-17 06:39:10]

The CQ bit was checked by ericwilligers@chromium.org

[commit-bot: I haz the power, 2017-02-17 06:39:35]

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-02-17 06:50:37]

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

[commit-bot: I haz the power, 2017-02-17 06:50:42]

Try jobs failed on following builders:
  chromium_presubmit on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[Eric Willigers, 2017-02-17 20:35:32]

ericwilligers@chromium.org changed reviewers:
+ sky@chromium.org

[Eric Willigers, 2017-02-17 20:35:33]

+sky for ui/OWNERS and
chrome/browser/devtools/frontend/devtools_discovery_page.html OWNERS

CSS transforms and CSS animations have previously been unprefixed in separate
patches for ui/ and chrome/browser/resources/.

However, at least some of the CSS transitions in ui/ and chrome/browser/ needed
to be unprefixed in the same patch, as they both use ensureTransitionEndEvent.

[sky, 2017-02-17 22:26:47]

LGTM

[Eric Willigers, 2017-02-17 22:31:10]

The CQ bit was checked by ericwilligers@chromium.org

[commit-bot: I haz the power, 2017-02-17 22:33: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, 2017-02-17 23:47:06]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1487370670694200,
"parent_rev": "41fb78271f7a247a47fd6a75a8f1953ff0282bce", "commit_rev":
"63f826afd2936707c9c697ce774a65e783041db7"}

[commit-bot: I haz the power, 2017-02-17 23:48:29]

Description was changed from

==========
Uprefix CSS transitions in ui/ and chrome/ styles

Transitions no longer require webkit prefix.

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

to

==========
Uprefix CSS transitions in ui/ and chrome/ styles

Transitions no longer require webkit prefix.

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

Review-Url: https://codereview.chromium.org/2691883002
Cr-Commit-Position: refs/heads/master@{#451402}
Committed:
https://chromium.googlesource.com/chromium/src/+/63f826afd2936707c9c697ce774a...
==========

[commit-bot: I haz the power, 2017-02-17 23:48:33]

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