Don't create cookies when disabled (#18935)

* Rebuilding

* Empty commit

* rebuilt piwik.js

* Need to track a pageview before checking cookie

* more fixes for tests where not setting visitorId cookie until tracking request first sent

* a few more test fixes

* allow a little longer for beforeUnloadHandler now it does more by default...

* bump it a little higher

* track a page view so the referer tests work

* undo last

* testing

* give more info

* removed test code

* undo test changes, set a variable to store initial visitor id while the cookie isn't set

* one more test removal

* Track a pageview to create cookie before testing matching visitorIds

* rebuilt piwik.js

* trackpageview so some more visitorid tests pass

* one more tracking request now

* allow more time for beforeUnloadHandler

* Instead of storing a temporary variable we just create the cookie when getVisitorId is called, if it is not existing

* rebuilt piwik.js

* Removed unnecessary changes

* rebuilt piwik.js

* minor fixes

* different approach

* rebuilt piwik.js

* set visitor id cookie in getVisitorInfo also

* rebuilt piwik.js

* remove unnecessary configCookiesDisabled check

* rebuilt piwik.js

Co-authored-by: justinvelluppillai <justinvelluppillai@users.noreply.github.com>

1 files changed, 1 insertions, 1 deletions

diff --git a/tests/javascript/index.php b/tests/javascript/index.php
index 0d77932c0f..5a8c081c66 100644
--- a/tests/javascript/index.php
+++ b/tests/javascript/index.php
@@ -5208,7 +5208,7 @@ if ($mysql) {
         tracker.hook.test._beforeUnloadHandler();
         stopTime = new Date();
         var msSinceStarted = (stopTime.getTime() - startTime.getTime());
-        ok( msSinceStarted < 510, 'beforeUnloadHandler(): ' + msSinceStarted + ' was greater than 510 ' );
+        ok( msSinceStarted < 520, 'beforeUnloadHandler(): ' + msSinceStarted + ' was greater than 520 ' );
 
         tracker.disableAlwaysUseSendBeacon();
         tracker.setLinkTrackingTimer(2000);