Merge pull request ampproject#1904 from gduchene/master

Use a proper correlator value for doubleclick.js
jridgewell · Feb 11, 2016 · 63f786b · 63f786b
2 parents b5343b5 + 4d20b1e
commit 63f786b
Showing 1 changed file with 9 additions and 0 deletions.
diff --git a/ads/doubleclick.js b/ads/doubleclick.js
@@ -40,12 +40,21 @@ export function doubleclick(global, data) {
         parseInt(data.overrideWidth || data.width, 10),
         parseInt(data.overrideHeight || data.height, 10)
       ]];
+      const clientId = window.context.clientId;
+      const pageViewId = window.context.pageViewId;
+      let correlator = null;
+      if (clientId != null) {
+        correlator = pageViewId + (clientId.replace(/\D/g, '') % 1e6) * 1e6;
+      } else {
+        correlator = pageViewId;
+      }
       const pubads = googletag.pubads();
       const slot = googletag.defineSlot(data.slot, dimensions, 'c')
           .addService(pubads);
       pubads.enableSingleRequest();
       pubads.markAsAmp();
       pubads.set('page_url', context.canonicalUrl);
+      pubads.setCorrelator(Number(correlator));
       googletag.enableServices();
 
       if (data.targeting) {