Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(245)

Issue 1414903008: Elim CSS transitions causing distortion in the user image (Closed)

Created:
5 years, 1 month ago by stevenjb
Modified:
5 years, 1 month ago
Reviewers:
achuithb, Dan Beam
CC:
chromium-reviews, dbeam+watch-options_chromium.org, michaelpg+watch-options_chromium.org, arv+watch_chromium.org, oshima+watch_chromium.org
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Elim CSS transitions causing distortion in the user image Going back to at least R45, the CSS transistions of -webkit-transform for the camera image select UI are non functional and occasionaly distort. It is unclear what the root cause might be, but the simplest short term fix is to eliminate the transitions entirely. BUG=548621 Committed: https://crrev.com/df3a7e240ad25cab39b0084e83f982ffd4ac42da Cr-Commit-Position: refs/heads/master@{#358612}

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+0 lines, -7 lines) Patch
M chrome/browser/resources/options/chromeos/change_picture_options.css View 2 chunks +0 lines, -7 lines 0 comments Download

Messages

Total messages: 12 (4 generated)
stevenjb
5 years, 1 month ago (2015-11-06 22:50:00 UTC) #2
achuithb
lgtm
5 years, 1 month ago (2015-11-06 23:58:42 UTC) #3
commit-bot: I haz the power
Dry run: CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1414903008/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1414903008/1
5 years, 1 month ago (2015-11-07 00:00:15 UTC) #5
Dan Beam
lgtm
5 years, 1 month ago (2015-11-07 00:34:05 UTC) #6
commit-bot: I haz the power
Dry run: This issue passed the CQ dry run.
5 years, 1 month ago (2015-11-07 00:42:48 UTC) #8
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1414903008/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1414903008/1
5 years, 1 month ago (2015-11-09 17:34:44 UTC) #10
commit-bot: I haz the power
Committed patchset #1 (id:1)
5 years, 1 month ago (2015-11-09 18:28:41 UTC) #11
commit-bot: I haz the power
5 years, 1 month ago (2015-11-09 18:29:22 UTC) #12
Message was sent while issue was closed.
Patchset 1 (id:??) landed as
https://crrev.com/df3a7e240ad25cab39b0084e83f982ffd4ac42da
Cr-Commit-Position: refs/heads/master@{#358612}

Powered by Google App Engine
This is Rietveld 408576698