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

Issue 12330177: Use Views Omnibox and Textfields by default on non-Aura Win. (Closed)

Created:
7 years, 9 months ago by msw
Modified:
7 years, 8 months ago
Reviewers:
sky, dcheng
CC:
chromium-reviews, tfarina, James Su, penghuang+watch_chromium.org, yusukes+watch_chromium.org, Ben Goodger (Google), Alexei Svitkine (slow), xji, oshima, gideonwald
Visibility:
Public.

Description

Use Views Omnibox and Textfields by default on non-Aura Win. Enable by default to expand dogfooding for release. Users can --disable-views-textfield via commandline or about:flags. File related Chrome UI (not web content) bugs against msw. Known issues: http://crbug.com/?q=owner=msw@chromium.org BUG=131660 TEST=Textfields work well enough on non-Aura Win. R=sky@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=192702

Patch Set 1 : Sync and rebase. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+0 lines, -1 line) Patch
M ui/views/controls/textfield/textfield.cc View 1 chunk +0 lines, -1 line 0 comments Download

Messages

Total messages: 8 (0 generated)
msw
Hey Scott, please take a look; thanks.
7 years, 9 months ago (2013-02-27 19:55:05 UTC) #1
msw
Actually, hold off while I investigate test failures; sorry.
7 years, 9 months ago (2013-02-27 20:34:47 UTC) #2
msw
Hey Scott, please take a look, tests will pass after this CL lands: https://codereview.chromium.org/13461032/
7 years, 8 months ago (2013-04-05 19:06:56 UTC) #3
sky
LGTM
7 years, 8 months ago (2013-04-05 19:47:07 UTC) #4
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/msw@chromium.org/12330177/23001
7 years, 8 months ago (2013-04-06 02:29:16 UTC) #5
commit-bot: I haz the power
Change committed as 192702
7 years, 8 months ago (2013-04-06 04:51:04 UTC) #6
dcheng
Is there any chance this is causing the test failure on Windows? They started happening ...
7 years, 8 months ago (2013-04-07 02:13:00 UTC) #7
msw
7 years, 8 months ago (2013-04-07 17:45:32 UTC) #8
Message was sent while issue was closed.
On 2013/04/07 02:13:00, dcheng wrote:
> Is there any chance this is causing the test failure on Windows?
Yup, reverting fixed the test; I filed http://crrev.com/227444 and will fix
before re-landing.
Thanks for taking care of that and sorry for the inconvenience.

Powered by Google App Engine
This is Rietveld 408576698