Clears out external protocol data when cookies and site data is cleared.

chrome/browser/extensions/api/browsing_data/browsing_data_api.h

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 // Defines the Chrome Extensions BrowsingData API functions, which entail
 // clearing browsing data, and clearing the browser's cache (which, let's be
 // honest, are the same thing), as specified in the extension API JSON.
 
 #ifndef CHROME_BROWSER_EXTENSIONS_API_BROWSING_DATA_BROWSING_DATA_API_H_
 #define CHROME_BROWSER_EXTENSIONS_API_BROWSING_DATA_BROWSING_DATA_API_H_
@@ skipping 22 matching lines @@
 extern const char kFileSystemsKey[];
 extern const char kFormDataKey[];
 extern const char kHistoryKey[];
 extern const char kIndexedDBKey[];
 extern const char kPluginDataKey[];
 extern const char kLocalStorageKey[];
 extern const char kPasswordsKey[];
 extern const char kServiceWorkersKey[];
 extern const char kCacheStorageKey[];
 extern const char kWebSQLKey[];
+extern const char kExternalProtocolKey[];
 
 // Option keys.
 extern const char kExtensionsKey[];
 extern const char kOriginTypesKey[];
 extern const char kProtectedWebKey[];
 extern const char kSinceKey[];
 extern const char kUnprotectedWebKey[];
 
 // Errors!
 extern const char kBadDataTypeDetails[];
@@ skipping 249 matching lines @@
   DECLARE_EXTENSION_FUNCTION("browsingData.removeWebSQL",
                              BROWSINGDATA_REMOVEWEBSQL)
 
  protected:
   ~BrowsingDataRemoveWebSQLFunction() override {}
 
   // BrowsingDataRemoverFunction:
   bool GetRemovalMask(int* removal_mask) override;
 };
 
+class BrowsingDataRemoveExternalProtocolDataFunction

[Devlin, 2017/02/15 15:56:46]

I don't see anywhere in the bug or CL description that indicates this should be
adding a new extension API function, nor anywhere that defines this function so
it could be exposed to an extension.  Is this intentional/necessary?

[ramyasharma, 2017/02/16 03:29:47]

This API was changed because test suite browsing_data_test.cc was failing. And I
was not sure if there is a way to work around these failures. But as msramek@
confirmed it is not necessary to add to the extension API, and there is another
way to make the tests pass, I am reverting these changes, and not implementing
this API change. Sorry for the confusion, I was waiting to hear from msramek@
before updating the cl.

+    : public BrowsingDataRemoverFunction {
+ public:
+  DECLARE_EXTENSION_FUNCTION("browsingData.removeExternalProtocolData",
+                             BROWSINGDATA_REMOVEEXTERNALPROTOCOLDATA)
+
+ protected:
+  ~BrowsingDataRemoveExternalProtocolDataFunction() override {}
+
+  // BrowsingDataRemoverFunction:
+  bool GetRemovalMask(int* removal_mask) override;
+};
+
 #endif  // CHROME_BROWSER_EXTENSIONS_API_BROWSING_DATA_BROWSING_DATA_API_H_