[CodeHealth] Use base::OnceClosure in FontAccessChooserController.

While here, use QuitClosure as a more concise way of calling
RunLoop::Quit.

No functional change intended.

Bug: 1007635
Change-Id: I25de4378122baa59fdab936b1602656533063106
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2600025
Commit-Queue: Olivier Yiptong <oyiptong@chromium.org>
Auto-Submit: Jeremy Roman <jbroman@chromium.org>
Reviewed-by: Olivier Yiptong <oyiptong@chromium.org>
Cr-Commit-Position: refs/heads/master@{#838940}

chrome/browser/ui/font_access/font_access_chooser_controller.h

@@ -5,6 +5,7 @@
 #ifndef CHROME_BROWSER_UI_FONT_ACCESS_FONT_ACCESS_CHOOSER_CONTROLLER_H_
 #define CHROME_BROWSER_UI_FONT_ACCESS_FONT_ACCESS_CHOOSER_CONTROLLER_H_
 
+#include "base/callback.h"
 #include "chrome/browser/chooser_controller/chooser_controller.h"
 #include "content/public/browser/font_access_chooser.h"
 #include "content/public/browser/render_frame_host.h"
@@ -46,15 +47,15 @@ class FontAccessChooserController : public ChooserController {
   void Close() override;
   void OpenHelpCenterUrl() const override;
 
-  void SetReadyCallbackForTesting(base::Callback<void()> callback) {
-    ready_callback_for_testing_ = callback;
+  void SetReadyCallbackForTesting(base::OnceClosure callback) {
+    ready_callback_for_testing_ = std::move(callback);
   }
 
  private:
   void DidFindAllFonts(blink::mojom::FontEnumerationStatus status,
                        std::vector<blink::mojom::FontMetadata> fonts);
   content::FontAccessChooser::Callback callback_;
-  base::Callback<void()> ready_callback_for_testing_;
+  base::OnceClosure ready_callback_for_testing_;
 
   std::map<std::string, blink::mojom::FontMetadata> font_metadata_map_;
   // An ordered list of font names that determines the order of items in the

chrome/browser/ui/font_access/font_access_chooser_controller_unittest.cc

@@ -41,8 +41,7 @@ TEST_F(FontAccessChooserControllerTest, MultiSelectTest) {
                       }));
 
   base::RunLoop readiness_loop;
-  controller.SetReadyCallbackForTesting(
-      base::BindLambdaForTesting([&]() { readiness_loop.Quit(); }));
+  controller.SetReadyCallbackForTesting(readiness_loop.QuitClosure());
   readiness_loop.Run();
 
   controller.set_view(mock_font_chooser_view_.get());
@@ -65,8 +64,7 @@ TEST_F(FontAccessChooserControllerTest, CancelTest) {
           }));
 
   base::RunLoop readiness_loop;
-  controller.SetReadyCallbackForTesting(
-      base::BindLambdaForTesting([&]() { readiness_loop.Quit(); }));
+  controller.SetReadyCallbackForTesting(readiness_loop.QuitClosure());
   readiness_loop.Run();
 
   controller.set_view(mock_font_chooser_view_.get());
@@ -87,8 +85,7 @@ TEST_F(FontAccessChooserControllerTest, CloseTest) {
           }));
 
   base::RunLoop readiness_loop;
-  controller.SetReadyCallbackForTesting(
-      base::BindLambdaForTesting([&]() { readiness_loop.Quit(); }));
+  controller.SetReadyCallbackForTesting(readiness_loop.QuitClosure());
   readiness_loop.Run();
 
   controller.set_view(mock_font_chooser_view_.get());
@@ -112,8 +109,7 @@ TEST_F(FontAccessChooserControllerTest, DestructorTest) {
               }));
 
   base::RunLoop readiness_loop;
-  controller->SetReadyCallbackForTesting(
-      base::BindLambdaForTesting([&]() { readiness_loop.Quit(); }));
+  controller->SetReadyCallbackForTesting(readiness_loop.QuitClosure());
   readiness_loop.Run();
 
   controller.reset();