Commit 97abc3d8 authored by gogerald's avatar gogerald Committed by Commit Bot

[StartSurface] Add user actions for the fake box

This CL is based on
https://chromium-review.googlesource.com/c/chromium/src/+/1825165

Bug: 982018
Change-Id: I1cf69ab2aa2ecdaf97d39b92bcfdd690ca4f12ee
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1853826
Commit-Queue: Ganggui Tang <gogerald@chromium.org>
Reviewed-by: default avatarYusuf Ozuysal <yusufo@chromium.org>
Cr-Commit-Position: refs/heads/master@{#706753}
parent dafecd19
...@@ -17,6 +17,7 @@ import android.text.Editable; ...@@ -17,6 +17,7 @@ import android.text.Editable;
import android.text.TextWatcher; import android.text.TextWatcher;
import android.view.View; import android.view.View;
import org.chromium.base.metrics.RecordUserAction;
import org.chromium.chrome.browser.ntp.FakeboxDelegate; import org.chromium.chrome.browser.ntp.FakeboxDelegate;
import org.chromium.chrome.browser.omnibox.LocationBar; import org.chromium.chrome.browser.omnibox.LocationBar;
import org.chromium.chrome.browser.omnibox.LocationBarVoiceRecognitionHandler; import org.chromium.chrome.browser.omnibox.LocationBarVoiceRecognitionHandler;
...@@ -42,6 +43,7 @@ class TasksSurfaceMediator { ...@@ -42,6 +43,7 @@ class TasksSurfaceMediator {
public void onClick(View v) { public void onClick(View v) {
mFakeboxDelegate.setUrlBarFocus( mFakeboxDelegate.setUrlBarFocus(
true, null, LocationBar.OmniboxFocusReason.TASKS_SURFACE_FAKE_BOX_TAP); true, null, LocationBar.OmniboxFocusReason.TASKS_SURFACE_FAKE_BOX_TAP);
RecordUserAction.record("TasksSurface.FakeBox.Tapped");
} }
}); });
mModel.set(FAKE_SEARCH_BOX_TEXT_WATCHER, new TextWatcher() { mModel.set(FAKE_SEARCH_BOX_TEXT_WATCHER, new TextWatcher() {
...@@ -56,6 +58,7 @@ class TasksSurfaceMediator { ...@@ -56,6 +58,7 @@ class TasksSurfaceMediator {
if (s.length() == 0) return; if (s.length() == 0) return;
mFakeboxDelegate.setUrlBarFocus(true, s.toString(), mFakeboxDelegate.setUrlBarFocus(true, s.toString(),
LocationBar.OmniboxFocusReason.TASKS_SURFACE_FAKE_BOX_LONG_PRESS); LocationBar.OmniboxFocusReason.TASKS_SURFACE_FAKE_BOX_LONG_PRESS);
RecordUserAction.record("TasksSurface.FakeBox.LongPressed");
// This won't cause infinite loop since we checked s.length() == 0 above. // This won't cause infinite loop since we checked s.length() == 0 above.
s.clear(); s.clear();
...@@ -66,6 +69,7 @@ class TasksSurfaceMediator { ...@@ -66,6 +69,7 @@ class TasksSurfaceMediator {
public void onClick(View v) { public void onClick(View v) {
mFakeboxDelegate.getLocationBarVoiceRecognitionHandler().startVoiceRecognition( mFakeboxDelegate.getLocationBarVoiceRecognitionHandler().startVoiceRecognition(
LocationBarVoiceRecognitionHandler.VoiceInteractionSource.TASKS_SURFACE); LocationBarVoiceRecognitionHandler.VoiceInteractionSource.TASKS_SURFACE);
RecordUserAction.record("TasksSurface.FakeBox.VoiceSearch");
} }
}); });
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment