Replace -webkit-transform with transform in Chrome UI

Replace -webkit-transform with transfrom in Chrome UI

BUG=455397
Committed: https://crrev.com/95c81ffa5138408c2bad11a3e6732c3ff7a2657c
Cr-Commit-Position: refs/heads/master@{#384565}

[Deokjin Kim, 2015-12-24 13:21:07]

Description was changed from

==========
Replace -webkit-transform with transfrom in Chrome UI

BUG=455397
==========

to

==========
Replace -webkit-transform with transfrom in Chrome UI

BUG=455397
==========

[Deokjin Kim, 2015-12-24 13:21:07]

deokjin81.kim@samsung.com changed reviewers:
+ dstockwell@chromium.org, sadrul@chromium.org

[Deokjin Kim, 2015-12-24 13:22:02]

sadrul and dstcokwell, PTAL.

[sadrul, 2016-01-04 19:51:59]

sadrul@chromium.org changed reviewers:
+ estade@chromium.org

[sadrul, 2016-01-04 19:52:00]

+estade@ as ui/webui owner. I can stamp the rest if webui has your approval.

[Dan Beam, 2016-01-04 20:27:27]

https://codereview.chromium.org/1549903002/diff/1/ui/webui/resources/js/cr/ui...
File ui/webui/resources/js/cr/ui/card_slider.js (right):

https://codereview.chromium.org/1549903002/diff/1/ui/webui/resources/js/cr/ui...
ui/webui/resources/js/cr/ui/card_slider.js:590:
this.container_.style.WebkitTransform = 'translate3d(' + x + 'px, 0, 0)';
you should probably fix \.[wW]ebkitTransform as well

[Deokjin Kim, 2016-01-05 12:13:56]

On 2016/01/04 20:27:27, Dan Beam wrote:
>
https://codereview.chromium.org/1549903002/diff/1/ui/webui/resources/js/cr/ui...
> File ui/webui/resources/js/cr/ui/card_slider.js (right):
> 
>
https://codereview.chromium.org/1549903002/diff/1/ui/webui/resources/js/cr/ui...
> ui/webui/resources/js/cr/ui/card_slider.js:590:
> this.container_.style.WebkitTransform = 'translate3d(' + x + 'px, 0, 0)';
> you should probably fix \.[wW]ebkitTransform as well
I replaced [wW]ebkitTransform with transform. Would you review my patch?

[dstockwell, 2016-02-02 11:59:22]

On 2016/01/05 at 12:13:56, deokjin81.kim wrote:
> On 2016/01/04 20:27:27, Dan Beam wrote:
> >
https://codereview.chromium.org/1549903002/diff/1/ui/webui/resources/js/cr/ui...
> > File ui/webui/resources/js/cr/ui/card_slider.js (right):
> > 
> >
https://codereview.chromium.org/1549903002/diff/1/ui/webui/resources/js/cr/ui...
> > ui/webui/resources/js/cr/ui/card_slider.js:590:
> > this.container_.style.WebkitTransform = 'translate3d(' + x + 'px, 0, 0)';
> > you should probably fix \.[wW]ebkitTransform as well
> I replaced [wW]ebkitTransform with transform. Would you review my patch?

I'm not an owner of any of this code, but from the point of view of replacing
-webkit-transform with transform this lgtm.

[Deokjin Kim, 2016-02-02 14:33:32]

On 2016/02/02 11:59:22, dstockwell wrote:
> I'm not an owner of any of this code, but from the point of view of replacing
> -webkit-transform with transform this lgtm.
Thank you for your review. May I land this CL?

[dstockwell, 2016-02-22 11:15:09]

On 2016/02/02 at 14:33:32, deokjin81.kim wrote:
> On 2016/02/02 11:59:22, dstockwell wrote:
> > I'm not an owner of any of this code, but from the point of view of
replacing
> > -webkit-transform with transform this lgtm.
> Thank you for your review. May I land this CL?

@estade, @sadrul, ping?

[Evan Stade, 2016-02-22 23:50:32]

I think I asked Dan to take a look? he's a better webui reviewer than me.

[Evan Stade, 2016-02-22 23:50:44]

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

[Dan Beam, 2016-02-23 00:27:31]

lgtm but:
a) I only own ui/webui
b) why not do this in the rest of chrome as well?

[dstockwell, 2016-03-23 05:59:38]

Any plans to land this patch?

[Deokjin Kim, 2016-03-23 06:07:23]

webui was reviewed by Dan Beam, but the rest is not reviewed yet.
So I'm waiting for sadrul's review. I guess he is out of office until Mar.24.

[mtomasz, 2016-03-23 06:16:01]

mtomasz@chromium.org changed reviewers:
+ mtomasz@chromium.org

[mtomasz, 2016-03-23 06:16:01]

ui/file_manager/* lgtm

[sadrul, 2016-03-30 14:29:43]

stamp lgtm

[Deokjin Kim, 2016-03-30 22:54:12]

The CQ bit was checked by deokjin81.kim@samsung.com

[commit-bot: I haz the power, 2016-03-30 22:54:42]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1549903002/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1549903002/40001

[commit-bot: I haz the power, 2016-03-30 22:56:59]

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

[commit-bot: I haz the power, 2016-03-30 22:57:00]

Try jobs failed on following builders:
  ios_dbg_simulator_gn on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios_dbg_simulator...)
  ios_dbg_simulator_ninja on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios_dbg_simulator...)
  ios_rel_device_gn on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios_rel_device_gn...)
  ios_rel_device_ninja on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios_rel_device_ni...)
  mac_chromium_compile_dbg_ng on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)
  mac_chromium_rel_ng on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[Deokjin Kim, 2016-03-31 12:28:09]

I rebased this CL. May I land this patch?

[Dan Beam, 2016-03-31 14:42:29]

still lgtm

[Deokjin Kim, 2016-04-01 00:54:07]

The CQ bit was checked by deokjin81.kim@samsung.com

[Deokjin Kim, 2016-04-01 00:54:07]

The patchset sent to the CQ was uploaded after l-g-t-m from
dstockwell@chromium.org, sadrul@chromium.org, mtomasz@chromium.org
Link to the patchset: https://codereview.chromium.org/1549903002/#ps60001
(title: "Rebase CL")

[commit-bot: I haz the power, 2016-04-01 00:54:46]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1549903002/60001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1549903002/60001

[commit-bot: I haz the power, 2016-04-01 06:56:20]

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

[commit-bot: I haz the power, 2016-04-01 06:56:22]

Try jobs failed on following builders:
  mac_chromium_rel_ng on tryserver.chromium.mac (JOB_TIMED_OUT,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[Deokjin Kim, 2016-04-01 12:16:12]

The CQ bit was checked by deokjin81.kim@samsung.com

[commit-bot: I haz the power, 2016-04-01 12:16:26]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1549903002/60001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1549903002/60001

[commit-bot: I haz the power, 2016-04-01 12:44:02]

Description was changed from

==========
Replace -webkit-transform with transfrom in Chrome UI

BUG=455397
==========

to

==========
Replace -webkit-transform with transfrom in Chrome UI

BUG=455397
==========

[commit-bot: I haz the power, 2016-04-01 12:44:03]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2016-04-01 12:45:38]

Description was changed from

==========
Replace -webkit-transform with transfrom in Chrome UI

BUG=455397
==========

to

==========
Replace -webkit-transform with transfrom in Chrome UI

BUG=455397

Committed: https://crrev.com/95c81ffa5138408c2bad11a3e6732c3ff7a2657c
Cr-Commit-Position: refs/heads/master@{#384565}
==========

[commit-bot: I haz the power, 2016-04-01 12:45:40]

Patchset 4 (id:??) landed as
https://crrev.com/95c81ffa5138408c2bad11a3e6732c3ff7a2657c
Cr-Commit-Position: refs/heads/master@{#384565}