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

Issue 12847005: Convert Chromoting to use the new, non-deprecated flexbox (Closed)

Created:
7 years, 9 months ago by cbiesinger
Modified:
7 years, 8 months ago
Reviewers:
Jamie
CC:
chromium-reviews, jamiewalch+watch_chromium.org, dcaiafa+watch_chromium.org, simonmorris+watch_chromium.org, hclam+watch_chromium.org, wez+watch_chromium.org, amit, sanjeevr, garykac+watch_chromium.org, lambroslambrou+watch_chromium.org, rmsousa+watch_chromium.org, alexeypa+watch_chromium.org, sergeyu+watch_chromium.org
Base URL:
https://chromium.googlesource.com/chromium/src.git@inspector
Visibility:
Public.

Description

Convert Chromoting to use the new, non-deprecated flexbox This is basically a search and replace for -webkit-box -> -webkit-flex, and some related properties. I have tested that the UI still looks correct, although I couldn't find out where connection-history-options is used. R=jamiewalch@chromium.org BUG= Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=192972

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+8 lines, -8 lines) Patch
M remoting/webapp/connection_history.css View 1 chunk +1 line, -1 line 0 comments Download
M remoting/webapp/main.css View 5 chunks +6 lines, -6 lines 0 comments Download
M remoting/webapp/toolbar.css View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 4 (0 generated)
cbiesinger
7 years, 8 months ago (2013-04-04 23:18:40 UTC) #1
Jamie
lgtm
7 years, 8 months ago (2013-04-04 23:50:19 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/cbiesinger@chromium.org/12847005/1
7 years, 8 months ago (2013-04-08 19:11:42 UTC) #3
commit-bot: I haz the power
7 years, 8 months ago (2013-04-09 00:43:31 UTC) #4
Message was sent while issue was closed.
Change committed as 192972

Powered by Google App Engine
This is Rietveld 408576698