Change locale modifications in test to use ScopedLocale |
||
Issue description
,
Nov 9 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/990e612ebcd677c133a6ffe5e5b4bc7eae1762b3 commit 990e612ebcd677c133a6ffe5e5b4bc7eae1762b3 Author: Alex Newcomer <newcomer@chromium.org> Date: Thu Nov 09 18:41:03 2017 cros replace home-baked ScopedRTL with ScopedRestoreICUDefaultLocale Replace home-baked ScopedRTL and other Scoped RTL objects with the more commonly used version. Modify ctor of ScopedRestoreICUDefaultLocale to optionally accept a locale code to set. Bug: 777997 Change-Id: I23bebd574b008b1f7e8d16f2be5dda780d46ba63 Reviewed-on: https://chromium-review.googlesource.com/742442 Reviewed-by: danakj <danakj@chromium.org> Reviewed-by: Michael Wasserman <msw@chromium.org> Reviewed-by: Sadrul Chowdhury <sadrul@chromium.org> Commit-Queue: Alex Newcomer <newcomer@chromium.org> Cr-Commit-Position: refs/heads/master@{#515222} [modify] https://crrev.com/990e612ebcd677c133a6ffe5e5b4bc7eae1762b3/ash/shelf/shelf_view_unittest.cc [modify] https://crrev.com/990e612ebcd677c133a6ffe5e5b4bc7eae1762b3/base/test/icu_test_util.cc [modify] https://crrev.com/990e612ebcd677c133a6ffe5e5b4bc7eae1762b3/base/test/icu_test_util.h [modify] https://crrev.com/990e612ebcd677c133a6ffe5e5b4bc7eae1762b3/ui/views/view_unittest.cc
,
Nov 9 2017
|
||
►
Sign in to add a comment |
||
Comment 1 by newcomer@chromium.org
, Oct 24 2017