Skip to content
This repository has been archived by the owner on Dec 11, 2019. It is now read-only.

Adds media publishers support #11851

Closed
bbondy opened this issue Nov 8, 2017 · 0 comments
Closed

Adds media publishers support #11851

bbondy opened this issue Nov 8, 2017 · 0 comments
Assignees
@NejcZdovc
Labels
QA/checked-Linux QA/checked-macOS QA/checked-Win64 QA/test-plan-specified release-notes/include
Milestone
0.19.x Hotfix 5 (...

Comments

@bbondy
Copy link
Member

bbondy commented Nov 8, 2017 •
edited by srirambv
Loading

Test Plan

  • Visit any YouTube video in a normal/session tab and ensure the video publisher name is listed in ledger table
  • Visit any YouTube video in a private tab and ensure the video publisher name is not listed in ledger table
  • Visit any live YouTube video and ensure the time spent is shown under ledger table
  • Visit any embeded YouTube video and ensure the video publisher name is listed in ledger table
  • Ensure total time spent is correctly calculated for each publisher video
  • Ensure total time spent is correctly calculated when switching to YouTube video from an embeded video
  • Ensure YouTube publishers are not listed when Allow contributions to video is disabled in adavanced settings
  • Ensure existing YouTube publishers are not lost when Allow contributions to video is disabled in adavanced settings
  • Ensure YouTube publishers is listed but not included when auto-include is disabled
  • Update Advanced settings to different time/visit value and ensure YouTube videos are added to ledger table once criteria is met
@bbondy bbondy mentioned this issue Nov 8, 2017
Merged
8 tasks
NejcZdovc added a commit to NejcZdovc/browser-laptop that referenced this issue Nov 10, 2017
@NejcZdovc
Adds media publishers support
78252fa 
Resolves brave#11851

Auditors:

Test Plan:
NejcZdovc added a commit to NejcZdovc/browser-laptop that referenced this issue Nov 10, 2017
@NejcZdovc
Adds media publishers support
476b2f7 
Resolves brave#11851

Auditors:

Test Plan:
@kjozwiak kjozwiak mentioned this issue Nov 16, 2017
Closed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@NejcZdovc NejcZdovc

Labels
QA/checked-Linux QA/checked-macOS QA/checked-Win64 QA/test-plan-specified release-notes/include
Projects
None yet
Milestone
0.19.x Hotfix 5 (Release channel)
Development

No branches or pull requests
6 participants
@bbondy @kjozwiak @bsclifton @NejcZdovc @srirambv @LaurenWags