Commit a26b5fc4 authored by Alice Wang's avatar Alice Wang Committed by Commit Bot

[Signin][Android][UI] Adjust dark account picker |expand more| icon

This CL adjusts the account picker |expand more| icon in dark mode
according to the UI mock.

Bug: 1107383
Change-Id: I90532fee7b9bd12884e0a4f36c91b6eb17e55665
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2320168Reviewed-by: default avatarTheresa  <twellington@chromium.org>
Commit-Queue: Alice Wang <aliceywang@chromium.org>
Cr-Commit-Position: refs/heads/master@{#793682}
parent 8dc01e76
...@@ -11,9 +11,9 @@ ...@@ -11,9 +11,9 @@
<path <path
android:strokeWidth="1" android:strokeWidth="1"
android:pathData="M23.5,11.9999C23.5,18.3512 18.3513,23.4999 12,23.4999C5.6487,23.4999 0.5,18.3512 0.5,11.9999C0.5,5.6486 5.6487,0.4999 12,0.4999C18.3513,0.4999 23.5,5.6486 23.5,11.9999Z" android:pathData="M23.5,11.9999C23.5,18.3512 18.3513,23.4999 12,23.4999C5.6487,23.4999 0.5,18.3512 0.5,11.9999C0.5,5.6486 5.6487,0.4999 12,0.4999C18.3513,0.4999 23.5,5.6486 23.5,11.9999Z"
android:strokeColor="@color/default_chip_outline_color_light"/> android:strokeColor="@color/hairline_stroke_color"/>
<path <path
android:pathData="M12.5892,14.7487C12.4383,14.9039 12.23,14.9999 12,14.9999C11.77,14.9999 11.5617,14.9039 11.4108,14.7487L7.2442,10.463C7.0933,10.3079 7,10.0936 7,9.857C7,9.3839 7.3733,8.9999 7.8333,8.9999C8.0633,8.9999 8.2717,9.0959 8.4225,9.251L12,12.9307L15.5775,9.251C15.7283,9.0959 15.9367,8.9999 16.1667,8.9999C16.6267,8.9999 17,9.3839 17,9.857C17,10.0936 16.9067,10.3079 16.7558,10.463L12.5892,14.7487Z" android:pathData="M12.5892,14.7487C12.4383,14.9039 12.23,14.9999 12,14.9999C11.77,14.9999 11.5617,14.9039 11.4108,14.7487L7.2442,10.463C7.0933,10.3079 7,10.0936 7,9.857C7,9.3839 7.3733,8.9999 7.8333,8.9999C8.0633,8.9999 8.2717,9.0959 8.4225,9.251L12,12.9307L15.5775,9.251C15.7283,9.0959 15.9367,8.9999 16.1667,8.9999C16.6267,8.9999 17,9.3839 17,9.857C17,10.0936 16.9067,10.3079 16.7558,10.463L12.5892,14.7487Z"
android:fillColor="@color/default_chip_outline_color_dark" android:fillColor="@color/default_control_color_normal"
android:fillType="evenOdd"/> android:fillType="evenOdd"/>
</vector> </vector>
...@@ -38,6 +38,7 @@ import org.chromium.base.test.util.Feature; ...@@ -38,6 +38,7 @@ import org.chromium.base.test.util.Feature;
import org.chromium.chrome.R; import org.chromium.chrome.R;
import org.chromium.chrome.browser.flags.ChromeFeatureList; import org.chromium.chrome.browser.flags.ChromeFeatureList;
import org.chromium.chrome.browser.flags.ChromeSwitches; import org.chromium.chrome.browser.flags.ChromeSwitches;
import org.chromium.chrome.browser.night_mode.ChromeNightModeTestUtils;
import org.chromium.chrome.browser.signin.account_picker.AccountPickerBottomSheetCoordinator; import org.chromium.chrome.browser.signin.account_picker.AccountPickerBottomSheetCoordinator;
import org.chromium.chrome.browser.signin.account_picker.AccountPickerDelegate; import org.chromium.chrome.browser.signin.account_picker.AccountPickerDelegate;
import org.chromium.chrome.test.ChromeJUnit4ClassRunner; import org.chromium.chrome.test.ChromeJUnit4ClassRunner;
...@@ -130,6 +131,18 @@ public class AccountPickerBottomSheetTest { ...@@ -130,6 +131,18 @@ public class AccountPickerBottomSheetTest {
mCoordinator.getBottomSheetViewForTesting(), "collapsed_sheet_with_account"); mCoordinator.getBottomSheetViewForTesting(), "collapsed_sheet_with_account");
} }
@Test
@MediumTest
@Feature("RenderTest")
public void testCollapsedSheetWithAccountViewDarkMode() throws IOException {
TestThreadUtils.runOnUiThreadBlocking(
() -> { ChromeNightModeTestUtils.setUpNightModeForChromeActivity(true); });
mRenderTestRule.setNightModeEnabled(true);
buildAndShowCollapsedBottomSheet();
mRenderTestRule.render(
mCoordinator.getBottomSheetViewForTesting(), "collapsed_sheet_with_account_dark");
}
@Test @Test
@MediumTest @MediumTest
public void testExpandedSheet() { public void testExpandedSheet() {
......
...@@ -40,6 +40,7 @@ ...@@ -40,6 +40,7 @@
<color name="ripple_color_blue">@color/ripple_color_blue_light</color> <color name="ripple_color_blue">@color/ripple_color_blue_light</color>
<!-- Colors used for Widgets (checkboxes, switches, buttons, etc)--> <!-- Colors used for Widgets (checkboxes, switches, buttons, etc)-->
<color name="default_control_color_normal">@color/default_control_color_normal_dark</color>
<color name="default_control_color_active">@color/default_control_color_active_dark</color> <color name="default_control_color_active">@color/default_control_color_active_dark</color>
<!-- Chip colors --> <!-- Chip colors -->
......
...@@ -74,6 +74,7 @@ ...@@ -74,6 +74,7 @@
<color name="ripple_color_blue">@color/ripple_color_blue_dark</color> <color name="ripple_color_blue">@color/ripple_color_blue_dark</color>
<!-- Colors used for Widgets (checkboxes, switches, buttons, etc)--> <!-- Colors used for Widgets (checkboxes, switches, buttons, etc)-->
<color name="default_control_color_normal" tools:ignore="UnusedResources">@color/default_control_color_normal_light</color>
<color name="default_control_color_active" tools:ignore="UnusedResources">@color/default_control_color_active_light</color> <color name="default_control_color_active" tools:ignore="UnusedResources">@color/default_control_color_active_light</color>
<!-- Chip colors --> <!-- Chip colors -->
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment