Kaleidoscope: Remove deprecated method from data provider

This CL is a followup of crrev.com/c/2365314 which removes the code
that is deprecated by that CL. There is a chrome-internal CL that must
land after that CL and before this CL in order to avoid a broken state.

Bug: b:154517281
Change-Id: I21e0363a0744600b8d91857d53d435841b17145d
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2367934
Commit-Queue: Tommy Steimel <steimel@chromium.org>
Reviewed-by: Alex Gough <ajgo@chromium.org>
Cr-Commit-Position: refs/heads/master@{#801035}

chrome/browser/media/kaleidoscope/kaleidoscope_data_provider_impl.cc

@@ -159,11 +159,6 @@ void KaleidoscopeDataProviderImpl::GetShouldShowFirstRunExperience(
   std::move(cb).Run(false);
 }
 
-void KaleidoscopeDataProviderImpl::SetFirstRunExperienceCompleted() {
-  SetFirstRunExperienceStep(
-      media::mojom::KaleidoscopeFirstRunExperienceStep::kCompleted);
-}
-
 void KaleidoscopeDataProviderImpl::SetFirstRunExperienceStep(
     media::mojom::KaleidoscopeFirstRunExperienceStep step) {
   if (metrics_recorder_)

chrome/browser/media/kaleidoscope/kaleidoscope_data_provider_impl.h

@@ -51,7 +51,6 @@ class KaleidoscopeDataProviderImpl
   void GetCredentials(GetCredentialsCallback cb) override;
   void GetShouldShowFirstRunExperience(
       GetShouldShowFirstRunExperienceCallback cb) override;
-  void SetFirstRunExperienceCompleted() override;
   void SetFirstRunExperienceStep(
       media::mojom::KaleidoscopeFirstRunExperienceStep step) override;
   void GetAllMediaFeeds(GetAllMediaFeedsCallback cb) override;

chrome/browser/media/kaleidoscope/mojom/kaleidoscope.mojom

@@ -61,10 +61,6 @@ interface KaleidoscopeDataProvider {
   // Returns true if the first run experience should be shown.
   GetShouldShowFirstRunExperience() => (bool should_show_first_run);
 
-  // Called when the user successfully completes the first run experience.
-  // DEPRECATED: will be removed as soon as a private-side CL also lands.
-  SetFirstRunExperienceCompleted();
-
   // While the user is on the First Run Experience, this will be called each
   // time they move onto a different step. Used to set prefs and record metrics
   // on what step the user stopped at.