OpenX Adapter: Video Support Update

[jimee02]

Type of change

• Bugfix
• Feature
• Refactoring (no functional changes, no api changes)

Description of change

Bugfix

• Fixed bug in video where request is always in http instead of using current protocol
  https://github.com/prebid/Prebid.js/pull/2342/files#diff-2051ef6e55abda4fed56905b5982884aL320
  vs
  https://github.com/prebid/Prebid.js/pull/2342/files#diff-2051ef6e55abda4fed56905b5982884aR276

Feature

• Added beacon tracking for video requests via user sync
  • https://github.com/prebid/Prebid.js/pull/2342/files#diff-2051ef6e55abda4fed56905b5982884aR359
• Merged both Banner and Video Beacons into a single function
  • https://github.com/prebid/Prebid.js/pull/2342/files#diff-2051ef6e55abda4fed56905b5982884aR367

Refactors

• Added test coverage for user syncing and beaconing for banner and video responses
• Removed unnecessary query parameter 'ef'
• Switched custom functions with utility functions
• Refactored codebase for easier readability

[mkendall07]

This pull request introduces 1 alert when merging d902f9a6101cda08cc3c5194eeab1020ed12b022 into 216f01c - view on lgtm.com

new alerts:

• 1 for Superfluous trailing arguments

---

Comment posted by lgtm.com

[mkendall07]

This pull request introduces 1 alert and fixes 1 when merging aa779679c25ed5d5c600f51ce0f24606f2b8068b into 216f01c - view on lgtm.com

new alerts:

• 1 for Syntax error

fixed alerts:

• 1 for Useless assignment to local variable

---

Comment posted by lgtm.com