• Matt Giuca's avatar
    Revert "Implemented fluent style for select multiple" · af9106b5
    Matt Giuca authored
    This reverts commit 6881d10f.
    
    Reason for revert: Introduces a use of uninitialized value.
    
    NativeThemeBase::PaintInnerSpinButton passes |arrow| to PaintArrowButton without
    initializing its |right_to_left| member.
    
    See https://crbug.com/990672
    
    Original change's description:
    > Implemented fluent style for select multiple
    > 
    > -Styled the in-page select, options, and optgroups via css.
    > -Modified the scrollbar to have rounded corners
    > (had to plumb the zoom factor through the Scrollbar so that the native
    > theme aura code can get the rounding correct at paint time)
    > 
    > Bug: 987292
    > Change-Id: Id11d1a47fe3d5b2be01aa578aa066f80577a9bbd
    > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1715553
    > Commit-Queue: Allison Pastewka <alpastew@microsoft.com>
    > Reviewed-by: Elly Fong-Jones <ellyjones@chromium.org>
    > Reviewed-by: Kent Tamura <tkent@chromium.org>
    > Reviewed-by: Kentaro Hara <haraken@chromium.org>
    > Reviewed-by: Mason Freed <masonfreed@chromium.org>
    > Cr-Commit-Position: refs/heads/master@{#683728}
    
    TBR=ellyjones@chromium.org,chrishtr@chromium.org,haraken@chromium.org,tkent@chromium.org,masonfreed@chromium.org,alpastew@microsoft.com
    
    # Not skipping CQ checks because original CL landed > 1 day ago.
    
    Bug: 987292, 990672
    Change-Id: I778b09437c79061b23de1d5e8335c8f0cf124e12
    Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1735548Reviewed-by: default avatarMatt Giuca <mgiuca@chromium.org>
    Reviewed-by: default avatarKent Tamura <tkent@chromium.org>
    Commit-Queue: Matt Giuca <mgiuca@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#683912}
    af9106b5
native_theme_base.h 8.23 KB