Read later: capture bookmark icon metric when read later is enabled.

This is a preexisting metric capturing the way the bookmark icon was
activated that we want to continue to capture when read later is
enabled.

Bug: 1163938, 1131547
Change-Id: I0c397d7f534c6c52726978d3a85c9fa2a046a6a5
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2616418
Commit-Queue: Caroline Rising <corising@chromium.org>
Reviewed-by: Charlene Yan <cyan@chromium.org>
Cr-Commit-Position: refs/heads/master@{#841546}

chrome/browser/ui/views/location_bar/star_view.cc

@@ -74,6 +74,7 @@ void StarView::OnExecuting(PageActionIconView::ExecuteSource execute_source) {
 }
 
 void StarView::ExecuteCommand(ExecuteSource source) {
+  OnExecuting(source);
   if (base::FeatureList::IsEnabled(reading_list::switches::kReadLater)) {
     menu_model_ = std::make_unique<StarMenuModel>(
         this, active(), chrome::CanMoveActiveTabToReadLater(browser_),
@@ -85,7 +86,6 @@ void StarView::ExecuteCommand(ExecuteSource source) {
                             views::MenuAnchorPosition::kTopRight,
                             ui::MENU_SOURCE_NONE);
   } else {
-    OnExecuting(source);
     chrome::BookmarkCurrentTab(browser_);
   }
 }