GitLab

Following the discussion in:
https://chromium-review.googlesource.com/c/chromium/src/+/1827280/16/content/public/browser/back_forward_cache.h#51

The previous API was very verbose to use. A second one was added in:
https://chromium-review.googlesource.com/c/chromium/src/+/1827280

This CL removes update the first one to match the second.

Design for the new content public API
https://docs.google.com/document/d/1brlmAs1mMumbGwgwdE_M_VGR81YbArzeViI0P-Uak14

Some background on why we sometimes need to disable bfcache:
https://docs.google.com/document/d/1NjZeusdS1kyEkZyfLggndU1A6qVt0Y1sa-LRUxnMoK8

TBR=dominikn@chromium.org (for the name change in AppBanner)

Bug: 1001087
Change-Id: Idd39dff370bd45f7006318b2a1bbad9fec1baa24
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1835617
Commit-Queue: Arthur Sonzogni <arthursonzogni@chromium.org>
Reviewed-by: Arthur Sonzogni <arthursonzogni@chromium.org>
Reviewed-by: Camille Lamy <clamy@chromium.org>
Reviewed-by: Alexander Timin <altimin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#703727}