Don't open the eye dropper if it is already opened.

This CL adds a check to ensure that a new eye dropper is not opened
while an eye dropper is already opened.

Bug: 1137328
Change-Id: I14a134a5cb6ec8d3923956d6f3a1bbf8b5604c0c
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2481072Reviewed-by: Mason Freed <masonfreed@chromium.org>
Commit-Queue: Ionel Popescu <iopopesc@microsoft.com>
Cr-Commit-Position: refs/heads/master@{#818658}

third_party/blink/renderer/core/html/forms/color_chooser_popup_ui_controller.cc

@@ -276,7 +276,10 @@ void ColorChooserPopupUIController::EyeDropperResponseHandler(bool success,
 }
 
 void ColorChooserPopupUIController::OpenEyeDropper() {
-  if (!LocalFrame::HasTransientUserActivation(frame_))
+  // Don't open the eye dropper without user activation or if it is already
+  // opened.
+  if (!LocalFrame::HasTransientUserActivation(frame_) ||
+      eye_dropper_chooser_.is_bound())
     return;
 
   frame_->GetBrowserInterfaceBroker().GetInterface(