Chromium Code Reviews
DescriptionRevert of Track if a ReadingListEntry has been dismissed (patchset #7 id:120001 of https://codereview.chromium.org/2806213004/ )
Reason for revert:
Findit identified CL at revision 463615 as the culprit for
failures in the build cycles as shown on:
https://findit-for-me.appspot.com/waterfall/culprit?key=ag9zfmZpbmRpdC1mb3ItbWVyRAsSDVdmU3VzcGVjdGVkQ0wiMWNocm9taXVtLzAwNDMwMTMyMjlmZmZhYjM1NTVkYWYyOGM2NGVlMjc1MjEyMDlkN2IM
Original issue's description:
> Track if a ReadingListEntry has been dismissed
>
> This CL adds a field to the ReadingListEntry, saved only on the device, to
> track if the entry has been dismissed from the NTP.
> The only way to change the extra information is to use the model, allowing
> the data to be stored.
>
> BUG=707730
>
> Review-Url: https://codereview.chromium.org/2806213004
> Cr-Commit-Position: refs/heads/master@{#463615}
> Committed: https://chromium.googlesource.com/chromium/src/+/0043013229fffab3555daf28c64ee27521209d7b
TBR=olivierrobin@chromium.org,treib@chromium.org,gambard@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=707730
Patch Set 1 #
Messages
Total messages: 8 (3 generated)
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||