Sanity checks to make sure we are not trying to clear Content Licenses when Flash Pepper is not in use.

Added sanity checks to ClearBrowserDataHandler to ensure that when it calls BrowsingDataRemover::Remove, the |remove_mask| will not have the REMOVE_CONTENT_LICENSES flag set when PPAPI Flash is disabled -- even if the corresponding "Deauthorize Content Licenses" checkbox on the "Clear Browsing Data" dialog is checked.

This is required because while removing Content Licenses is only possible with PPAPI Flash, the checked state for the check-box is persisted, and thus even though the check-box gets hidden when PPAPI Flash is disabled, it will still be checked behind the curtains, and would cause the above-mentioned flag to be set.

We opt to keep the persisted checked-state for the check-box though, to provide consistency in the case PPAPI Flash gets re-enabled later.

This is the first of two CLs to address the issue mentioned below. It is in itself enough to fix the issue, but given that the issue actually discovered two bugs, there is an other CL to fix the other bug too. See issue for details.

BUG=144874


Review URL: https://chromiumcodereview.appspot.com/11000022

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@159667 0039d316-1c4b-4281-b951-d872f2087c98
2 files changed