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

Issue 23530019: Remove new dialog style command-line flags, etc. (Closed)

Created:
7 years, 3 months ago by msw
Modified:
7 years, 3 months ago
Reviewers:
sky
CC:
chromium-reviews, sadrul, yusukes+watch_chromium.org, ben+watch_chromium.org, tfarina, jam, penghuang+watch_chromium.org, joi+watch-content_chromium.org, nona+watch_chromium.org, darin-cc_chromium.org, James Su, Mike Wittman, James Cook
Visibility:
Public.

Description

Remove new dialog style command-line flags, etc. Remove --[en|dis]able-new-dialog-style command-line flags. Remove the corresponding strings and about:flags entry. The new dialog style has been on-by-default since M-29. Retain DialogDelegate::UseNewStyle() for Views/Win Textfield. (new dialogs can't host the off-by-default Win Textfields) (this will go away once I remove the Views Textfield flags) Retain split DialogDelegate::UseNewStyleForThisDialog() codepaths. (we explicitly want the old style for some dialogs see Issue 246777) (task manager, about:ipc dialog, extension (cros save/open) dialog) (this could go away if we made those windows not use dialog code) BUG=166075, 282698 TEST=New dialog style is used on Windows/CrOS everywhere except for the Task Manager, about:ipc dialog (Win only), and Extension (CrOS save/open) dialogs. The old dialog style is used with --disable-views-textfields for technical reasons. R=sky@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=221292

Patch Set 1 #

Patch Set 2 : Remove strings; tweak comment. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+3 lines, -44 lines) Patch
M ash/wm/system_modal_container_layout_manager.cc View 1 1 chunk +1 line, -2 lines 0 comments Download
M chrome/app/generated_resources.grd View 1 chunk +0 lines, -6 lines 0 comments Download
M chrome/browser/about_flags.cc View 1 chunk +0 lines, -8 lines 0 comments Download
M content/browser/renderer_host/render_process_host_impl.cc View 2 chunks +0 lines, -2 lines 0 comments Download
M ui/base/ui_base_switches.h View 1 chunk +0 lines, -2 lines 0 comments Download
M ui/base/ui_base_switches.cc View 2 chunks +0 lines, -6 lines 0 comments Download
M ui/base/ui_base_switches_util.h View 1 chunk +0 lines, -2 lines 0 comments Download
M ui/base/ui_base_switches_util.cc View 1 chunk +0 lines, -9 lines 0 comments Download
M ui/views/controls/textfield/textfield.cc View 2 chunks +0 lines, -4 lines 0 comments Download
M ui/views/window/dialog_delegate.cc View 2 chunks +2 lines, -3 lines 0 comments Download

Messages

Total messages: 4 (0 generated)
msw
Hey Scott, please take a look; thanks!
7 years, 3 months ago (2013-09-04 20:18:58 UTC) #1
sky
LGTM
7 years, 3 months ago (2013-09-04 21:01:42 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/msw@chromium.org/23530019/3001
7 years, 3 months ago (2013-09-04 21:05:48 UTC) #3
commit-bot: I haz the power
7 years, 3 months ago (2013-09-04 23:02:05 UTC) #4
Message was sent while issue was closed.
Change committed as 221292

Powered by Google App Engine
This is Rietveld 408576698