[Autofill Assistant] Do not implicitly hide progress bar at 100%.

See b/118735405.

Change-Id: Ia759ad8bab73d9479d832aa7dd187b51337773fc
Reviewed-on: https://chromium-review.googlesource.com/c/1312479Reviewed-by: Stephane Zermatten <szermatt@chromium.org>
Commit-Queue: Jordan Demeulenaere <jdemeulenaere@chromium.org>
Cr-Commit-Position: refs/heads/master@{#604589}

components/autofill_assistant/browser/actions/show_progress_bar_action.cc

@@ -24,11 +24,11 @@ ShowProgressBarAction::~ShowProgressBarAction() {}
 void ShowProgressBarAction::InternalProcessAction(
     ActionDelegate* delegate,
     ProcessActionCallback callback) {
-  int progress =
-      std::min(100, std::max(0, proto_.show_progress_bar().progress()));
-  if (proto_.show_progress_bar().done() || progress == 100) {
+  if (proto_.show_progress_bar().done()) {
     delegate->HideProgressBar();
   } else {
+    int progress =
+        std::min(100, std::max(0, proto_.show_progress_bar().progress()));
     delegate->ShowProgressBar(progress, proto_.show_progress_bar().message());
   }
 

components/autofill_assistant/browser/service.proto

@@ -378,7 +378,6 @@ message ShowProgressBarProto {
 
   // Value between 0 and 100 indicating the current progress. Values above 100
   // will be capped to 100, values below 0 will be capped to 0 by the client.
-  // NOTE: Setting |progress| to 100 is an equivalent of setting |done| to true.
   optional int32 progress = 6;
 }