Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Follow up of # PR 18834 - Incorrect response for the metrics Brave.VerticalTabs.GroupTabs when tabs are group and ngroup on the vertical tab view #31366

Closed
GeetaSarvadnya opened this issue Jun 28, 2023 · 2 comments

Comments

@GeetaSarvadnya
Copy link

Description

Found while testing brave/brave-core#18834
Follow up of # PR 18834 - Incorrect response for the metrics Brave.VerticalTabs.OpenTabs when tabs are group and ngroup on the vertical tab view

Steps to Reproduce

  1. Clean profile 1.55.17
  2. Open brave://local-state file, right click on the tab and select Use vertical tabs context menu
  3. Open a bunch of sites
  4. Ceate 6 tab groups
  5. Reload the bave://local-state file
  6. Look for the metrics Brave.VerticalTabs.GroupTabs
  7. Confirmed that the metrics Brave.VerticalTabs.GroupTabs response value is 2 as expected
  8. Remove 6th tab group
  9. Reload the bave://local-state file
  10. The metrics Brave.VerticalTabs.GroupTabs response value is 2 instead of 1

Actual result:

Incorrect response for the metrics Brave.VerticalTabs.GroupTabs when tabs are group and ngroup on the vertical tab view

Expected result:

The metrics should be correct when tabs are ungroup

Reproduces how often:

Easy

Brave version (brave://version info)

Brave 1.55.17 Chromium: 115.0.5790.40 (Official Build) nightly (64-bit)
Revision 071c9ddea889c3c7887daf4eac13fed72d4fff62-refs/branch-heads/5790@{#979}
OS Windows 10 Version 22H2 (Build 19045.3086)

Version/Channel Information:

  • Can you reproduce this issue with the current release? NA
  • Can you reproduce this issue with the beta channel? NA
  • Can you reproduce this issue with the nightly channel? Yes

Other Additional Information:

  • Does the issue resolve itself when disabling Brave Shields? NA
  • Does the issue resolve itself when disabling Brave Rewards? NA
  • Is the issue reproducible on the latest version of Chrome? NA

Miscellaneous Information:

cc: @brave/qa-team @DJAndries @mattmcalister

@DJAndries
Copy link
Collaborator

This is expected. The metric is supposed to report the max amount of groups for the past week: "As a user with vertical tabs mode enabled and had at least one tab group this week, what is the max amount of tab groups this week?"

@GeetaSarvadnya
Copy link
Author

GeetaSarvadnya commented Jul 6, 2023

As a user with vertical tabs mode enabled and had at least one tab group this week, what is the max amount of tab groups this week?

The max tab groups for the week is considered for the metrics response calculation hence the value 2 is expected for the 1st week, if we set the sys clock ahead by 8 days in the 2nd week the response value is 1 as expected. Hence closing the issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants