Don't log search events when not in search mode (82c9033a) · Commits · e / os / android_packages_apps_Contacts

src/com/android/contacts/activities/PeopleActivity.java

+3 −5

Original line number	Diff line number	Diff line
		@@ -560,8 +560,7 @@ public class PeopleActivity extends ContactsActivity implements
		break;
		case ActionBarAdapter.Listener.Action.START_SEARCH_MODE:
		if (!mIsRecreatedInstance) {
		Logger.getInstance().logScreenView(
		ScreenEvent.SEARCH, this, ScreenEvent.TAG_SEARCH);
		Logger.logScreenView(ScreenEvent.SEARCH, this, ScreenEvent.TAG_SEARCH);
		}
		startSearchOrSelectionMode();
		break;
		@@ -1367,9 +1366,8 @@ public class PeopleActivity extends ContactsActivity implements
		if (mAllFragment.wasSearchResultClicked()) {
		mAllFragment.resetSearchResultClicked();
		} else {
		Logger.getInstance().logScreenView(
		ScreenEvent.SEARCH_EXIT, this, ScreenEvent.TAG_SEARCH_EXIT);
		Logger.getInstance().logSearchEventImpl(mAllFragment.createSearchState());
		Logger.logScreenView(ScreenEvent.SEARCH_EXIT, this, ScreenEvent.TAG_SEARCH_EXIT);
		Logger.logSearchEvent(mAllFragment.createSearchState());
		}
		} else {
		super.onBackPressed();

+4 −2

Original line number	Diff line number	Diff line
		@@ -183,8 +183,10 @@ public class MultiSelectContactsListFragment extends DefaultContactBrowseListFra
		getAdapter().toggleSelectionOfContactId(Long.valueOf(contactId));
		}
		} else {
		if (isSearchMode()) {
		mSearchResultClicked = true;
		Logger.logSearchEvent(createSearchStateForSearchResultClick(position));
		}
		super.onItemClick(position, id);
		}
		if (mCheckBoxListListener != null && getAdapter().getSelectedContactIds().size() == 0) {