Chromium Code Reviews
DescriptionRevert of Replace AllAppsTileItemView with ArrowExpandView (patchset #2 id:20001 of https://codereview.chromium.org/2959943003/ )
Reason for revert:
Causes memory leak on ASAN LSAN bot, see
https://uberchromegw.corp.google.com/i/chromium.memory/builders/Linux%20Chromium%20OS%20ASan%20LSan%20Tests%20%281%29/builds/22001
Original issue's description:
> Replace AllAppsTileItemView with ArrowExpandView
>
> This CL replace the old "All apps" button in start page with a new arrow
> expand button.
>
> specs:
> https://screenshot.googleplex.com/PM0rC5nzVvE
> https://screenshot.googleplex.com/Qs1T3mHSf2D
>
> screenshot:
> https://screenshot.googleplex.com/RNQS0joM35z
>
> BUG=735502
>
> Review-Url: https://codereview.chromium.org/2959943003
> Cr-Commit-Position: refs/heads/master@{#482767}
> Committed: https://chromium.googlesource.com/chromium/src/+/c6f4af3584484510a05497d4f63d1194a6f553d7
TBR=xiyuan@chromium.org,weidongg@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=735502
Review-Url: https://codereview.chromium.org/2960123002
Cr-Commit-Position: refs/heads/master@{#482875}
Committed: https://chromium.googlesource.com/chromium/src/+/ffbc4fd6a0c2561d40ce890f20975737a7f1262d
Patch Set 1 #
Messages
Total messages: 6 (3 generated)
|
|||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||