Chromium Code Reviews
DescriptionRevert of Site Settings Android: A few polish items. (patchset #6 id:180001 of https://codereview.chromium.org/1947493002/ )
Reason for revert:
Caused crbug.com/610576
Original issue's description:
> Site Settings Android: A few polish items.
>
> From issue 601918:
> - Change "+ Add site" blue action link to be "+ Add site exception"
> - Change dialog text to be: "Allow [JavaScript/the permission in question etc.] for a specific site."
>
> From issue 467788:
> - Update the individual site settings page toggles with verbs that match the permissions bubbles
>
> From issue 604753 (partial fix):
> - Improve alignment of Site Settings list a bit (move icon a bit to the right).
> - Color the Site List heading blue.
>
> Also add an updated image for Protected Content (slightly larger) and the Media icon, which is for
> the container menu that will house Protected Content and Autoplay.
>
> BUG=601918, 467788, 604753
>
> Committed: https://crrev.com/2d4101ec17e44ff061d5e65bc52f386d9985e881
> Cr-Commit-Position: refs/heads/master@{#392363}
TBR=twellington@chromium.org,dfalcantara@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=601918, 467788, 604753
Committed: https://crrev.com/04e90ff9a7f58302c97fe4a0c14dd12e41f89951
Cr-Commit-Position: refs/heads/master@{#392612}
Patch Set 1 #Messages
Total messages: 5 (1 generated)
|