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

tabClassification is the same for all tabs #2955

Closed
btlechowski opened this issue Jan 14, 2019 · 2 comments
Closed

tabClassification is the same for all tabs #2955

btlechowski opened this issue Jan 14, 2019 · 2 comments
Assignees
Labels
feature/ads priority/P2 A bad problem. We might uplift this to the next planned release. QA/Test-Plan-Specified QA/Yes

Comments

@btlechowski
Copy link

btlechowski commented Jan 14, 2019

Steps to Reproduce

  1. Run brave with logging
  2. Enable rewards and ads
  3. In new tab open https://www.interia.pl/ and make sure tabClassification is set
  4. In new tab open https://www.booking.com/ and check tabClassification
  5. Keep https://www.booking.com/ active till tabClassification changes
  6. In new tab open https://techcrunch.com and wait for tabClassification to change

Actual result:

After step 4: All tabs have "tabClassification":["health & fitness","nutrition"]

[12252:1212:0111/164417.206:INFO:ads_service_impl.cc(958)] AdsService Event Log: {"data":{"type":"load","stamp":"2019-01-11T16:44:17Z","tabId":9,"tabType":"click","tabUrl":"https://www.interia.pl/","tabClassification":["health & fitness","nutrition"]}
[12252:1212:0111/164417.208:INFO:ads_service_impl.cc(958)] AdsService Event Log: {"data":{"type":"blur","stamp":"2019-01-11T16:44:17Z","tabId":9}
[12252:1212:0111/164417.209:INFO:ads_service_impl.cc(958)] AdsService Event Log: {"data":{"type":"load","stamp":"2019-01-11T16:44:17Z","tabId":10,"tabType":"click","tabUrl":"https://www.booking.com/","tabClassification":["health & fitness","nutrition"]}

After step 5: All tabs have "tabClassification":["travel","hotels"]

[12252:1212:0111/165829.671:INFO:ads_service_impl.cc(958)] AdsService Event Log: {"data":{"type":"load","stamp":"2019-01-11T16:58:29Z","tabId":10,"tabType":"click","tabUrl":"https://www.booking.com/","tabClassification":["travel","hotels"]}
[12252:1212:0111/165829.680:INFO:ads_service_impl.cc(958)] AdsService Event Log: {"data":{"type":"blur","stamp":"2019-01-11T16:58:29Z","tabId":10}
[12252:1212:0111/165829.683:INFO:ads_service_impl.cc(958)] AdsService Event Log: {"data":{"type":"load","stamp":"2019-01-11T16:58:29Z","tabId":9,"tabType":"click","tabUrl":"https://www.interia.pl/","tabClassification":["travel","hotels"]}

After step 6: All tabs have "tabClassification":["personal finance","investing"]

[12252:1212:0111/170854.790:INFO:ads_service_impl.cc(958)] AdsService Event Log: {"data":{"type":"load","stamp":"2019-01-11T17:08:54Z","tabId":9,"tabType":"click","tabUrl":"https://www.interia.pl/","tabClassification":["personal finance","investing"]}
[12252:1212:0111/170854.790:INFO:ads_service_impl.cc(958)] AdsService Event Log: {"data":{"type":"blur","stamp":"2019-01-11T17:08:54Z","tabId":9}
[12252:1212:0111/170854.791:INFO:ads_service_impl.cc(958)] AdsService Event Log: {"data":{"type":"load","stamp":"2019-01-11T17:08:54Z","tabId":10,"tabType":"click","tabUrl":"https://www.booking.com/","tabClassification":["personal finance","investing"]}
[12252:1212:0111/170854.799:INFO:ads_service_impl.cc(958)] AdsService Event Log: {"data":{"type":"blur","stamp":"2019-01-11T17:08:54Z","tabId":10}
[12252:1212:0111/170854.800:INFO:ads_service_impl.cc(958)] AdsService Event Log: {"data":{"type":"load","stamp":"2019-01-11T17:08:54Z","tabId":17,"tabType":"click","tabUrl":"https://techcrunch.com/?guccounter=1","tabClassification":["personal finance","investing"]}

Expected result:

tabCassification is set individually per tab as name suggests, It is not global.

Reproduces how often:

always

Brave version (brave://version info)

Brave 0.60.11 Chromium: 72.0.3626.28 (Official Build) dev (64-bit)
Revision 997b1040b63bac324e815797ba52be0cd8f616ed-refs/branch-heads/3626@{#461}
OS Windows 7

cc @jsecretan @brave/legacy_qa

@btlechowski btlechowski added this to the 1.x Backlog milestone Jan 14, 2019
@mrose17 mrose17 added the priority/P2 A bad problem. We might uplift this to the next planned release. label Jan 15, 2019
@mrose17 mrose17 self-assigned this Jan 17, 2019
@mrose17
Copy link
Member

mrose17 commented Jan 21, 2019

Fixed by brave-intl/bat-native-ads#106

@srirambv
Copy link
Contributor

@mrose17 is this targeted for 0.61.x ? The linked PR's are not having milestone.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature/ads priority/P2 A bad problem. We might uplift this to the next planned release. QA/Test-Plan-Specified QA/Yes
Projects
None yet
Development

No branches or pull requests

4 participants