Minor fixes in ChooserContextBase.

Fix one comment to not mention non-existing methods, and change a
parameter from a non-const reference to a const reference, as I
assume it was supposed to be.

Bug: None
Change-Id: I38d6cef800958dc0c4cf22af16e2cb2a9e4f58ac
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2231543
Commit-Queue: Marijn Kruisselbrink <mek@chromium.org>
Commit-Queue: Balazs Engedy <engedy@chromium.org>
Auto-Submit: Marijn Kruisselbrink <mek@chromium.org>
Reviewed-by: Balazs Engedy <engedy@chromium.org>
Cr-Commit-Position: refs/heads/master@{#775285}

components/permissions/chooser_context_base.cc

@@ -156,7 +156,7 @@ void ChooserContextBase::GrantObjectPermission(
 void ChooserContextBase::UpdateObjectPermission(
     const url::Origin& requesting_origin,
     const url::Origin& embedding_origin,
-    base::Value& old_object,
+    const base::Value& old_object,
     base::Value new_object) {
   base::Value setting =
       GetWebsiteSetting(requesting_origin, embedding_origin, /*info=*/nullptr);

components/permissions/chooser_context_base.h

@@ -105,14 +105,14 @@ class ChooserContextBase : public KeyedService {
   // |host_content_settings_map_|.
   void UpdateObjectPermission(const url::Origin& requesting_origin,
                               const url::Origin& embedding_origin,
-                              base::Value& old_object,
+                              const base::Value& old_object,
                               base::Value new_object);
 
   // Revokes |requesting_origin|'s permission to access |object| when embedded
   // within |embedding_origin|.
   //
   // This method may be extended by a subclass to revoke permission to access
-  // objects returned by GetPreviouslyChosenObjects but not stored in
+  // objects returned by GetGrantedObjects but not stored in
   // |host_content_settings_map_|.
   virtual void RevokeObjectPermission(const url::Origin& requesting_origin,
                                       const url::Origin& embedding_origin,