2007637 - heinnie.com - Products do not load on the page intermittently

Status: REOPENED

(Web Compatibility :: Site Reports, defect, P1)

Description

[Calin Tanase]

Environment:
Operating system: Linux/Windows 10
Firefox version: Firefox 146.0/148

Steps to reproduce:

1. Go to https://heinnie.com/search.php?search_query=overland&section=content
2. Observe the page. (if the products get shown, refresh the page a couple of times or open the page in a new window)

Expected Behavior:
The products get shown every time.

Actual Behavior:
The products are missing. (sometimes it might load)

Notes:

• Reproduces regardless of the status of ETP
• Reproduces in firefox-nightly, and firefox-release
• Does not reproduce in chrome

Created from https://github.com/webcompat/web-bugs/issues/196774

Comment 1

[Calin Tanase]

Attachment: FF vs Chrome.png

Comment 2

[Alice0775 White]

Regression window:
https://hg-edge.mozilla.org/integration/autoland/pushloghtml?fromchange=7de8dcc3a2e176149c35bd9c7e83662c6746655e&tochange=9ec7e6a99e488e2df7de24aea690efde6a940aaf

Comment 3

[BugBot [:suhaib / :marco/ :calixte]]

Set release status flags based on info from the regressing bug 1982924

:jrmuizel, since you are the author of the regressor, bug 1982924, could you take a look? Also, could you set the severity field?

For more information, please visit BugBot documentation.

Comment 4

[Jeff Muizelaar [:jrmuizel]]

I haven't been able to reproduce this. Does it still reproduce for others?

Comment 5

[Calin Tanase]

Could reproduce after closing the initial tab and re-opening on a new tab.

Comment 6

[Ksenia Berezina [:ksenia]]

I could reproduce only on the first page load

Comment 7

[Dianna Smith [:diannaS]]

I can also reproduce this on a Mac and in 147 beta by:

1. Opening a new window
2. pasting https://heinnie.com/search.php?search_query=overland&section=content
3. open new tab : pasting https://heinnie.com/search.php?search_query=overland&section=content
4. repeat until products do not display

I can reproduce this on the second tab sometimes on fourth try (fourth tab)

Comment 8

[Jeff Muizelaar [:jrmuizel]]

I still haven't been able to reproduce this. Can someone who can try getting a profile with the "Debug" preset?

Comment 9

[Dianna Smith [:diannaS]]

I ran profiler if that helps?

otherwise let me know what steps to take

Comment 10

[Kagami Rosylight [:saschanaz] (they/them)]

This is intermittent, I can sometimes see it when keep duplicating the tab.

Comment 11

[Kagami Rosylight [:saschanaz] (they/them)]

The list is loaded, it's just that the right CSS class is not added to the container. https://js.klevu.com/theme/default/v2/quick-landing-search-theme.js

e.dom.helpers.initializeTabSelection = function (t, a, s, n) {
    var o;
    e.each(
      e.dom.find('.kuTab', s),
      (
        function (a, s) {
          var r = n.getElement('active');
          if (
            s.classList.remove('kuTabSelected'),
            r &&
            s.dataset &&
            s.dataset.section &&
            r == s.dataset.section
          ) {
            s.classList.add('kuTabSelected');
            var i = e.dom.helpers.getClosest(s, '.klevuWrap');
            if (null === i) return;
            t.context.activeQueryId = s.dataset.section,
            t.context.section = s.dataset.section,
            i.classList.add(s.dataset.section + 'Active'),
            o = !0
          }
        }
      )
    ),

This adds productListActive to klevuWrap and makes the list visible. The call stack is:

initializeTabSelection quick-landing-search-theme.js:formatted:770
    ee klevu.js:1
    initializeTabSelection quick-landing-search-theme.js:formatted:765
    getStorageAndTarget quick-landing-search-theme.js:formatted:4552
    fire quick-landing-search-theme.js:formatted:12969
    a klevu.js:1
    fireWith klevu.js:1
    fire klevu.js:1
    fire klevu.js:1
    a klevu.js:1
    fireWith klevu.js:1
    fire klevu.js:1
    success klevu.js:1
    r klevu.js:1
    anonymous unknown:0

The event is fired in: https://js.klevu.com/core/v2/klevu.js

  t.getScope().chains.request.fetch.send.add({
    name: 'sendRequest',
    fire: function (t, n) {
      'searchAjax' !== t.context.eventAction &&
      (
        t.scope = n,
        t.context.doSearch ? (
          t.context.requestObject = {
            url: t.request.settings.url,
            type: 'FETCH',
            method: 'POST',
            mimeType: 'application/json; charset=UTF-8',
            contentType: 'application/json; charset=utf-8',
            dataType: 'json',
            crossDomain: !0,
            success: function (t, n, s, o) {
              var a = e.getObjectPath(n.scope.kScope, 'chains.response.success');
              e.isUndefined(a) ||
              0 === a.list().length ||
              (
                a.setScope(n.scope.kElem),
                e.setObjectPath(n, 'response.data', t),
                e.setObjectPath(n, 'context.status', s),
                e.setObjectPath(n, 'context.isSuccess', o),
                a.setData(n),
                a.fire()
              )
            },
            error: function (t, n, s) {
              var o = e.getObjectPath(t.scope.kScope, 'chains.response.fail');
              e.isUndefined(o) ||
              0 === o.list().length ||
              (
                o.setScope(t.scope.kElem),
                e.setObjectPath(t, 'response.data', {
                }),
                e.setObjectPath(t, 'context.status', n),
                e.setObjectPath(t, 'context.isSuccess', s),
                o.setData(t),
                o.fire()
              )
            },
            data: JSON.stringify(t.request.settings.object)
          },
          t.context.requestDetails = e.extend(!0, {
          }, t)
        ) : e.support.hook(
          ['core.logger'],
          (
            function () {
              e.logger.log(
                'search.core',
                'warn',
                'Fetch makeRequest - Request object is not present or request was forced stopped.'
              ),
              e.logger.log('search.core', 'debug', t)
            }
          )
        )
      )
    }
  }),

But even if a.fire() is called, it doesn't seem to always reach to the listener, even though the listener is always added before it dispatches. (It's not standard DOM event, fwiw)

Comment 12

[Jeff Muizelaar [:jrmuizel]]

Does setting dom.timeout.defer_during_load=false fix it?

Comment 13

[Dianna Smith [:diannaS]]

yes, setting dom.timeout.defer_during_load=false fixes it for me. Makes it no longer reproducible.

Comment 14

[Kagami Rosylight [:saschanaz] (they/them)]

Same. This needs an intervention.

Comment 15

[Justin Link]

I am able to reproduce this pretty consistently on my machine. Here is a profile that I took of it happening several times in a row. (I was using Ctrl+R to refresh.)

For me it often seemed to load correctly once or twice when I first visited the page, but then it would consistently fail after that.

Comment 16

[Jeff Muizelaar [:jrmuizel]]

Attachment: Bug 2007637. Add heinnie.com to dom.timeout.defer_during_load.

Verified experimentally that this improves things. It also makes
sense that bug 1982924 would have made this worse.

Comment 17

[Pulsebot]

Pushed by [e-mail]:
https://github.com/mozilla-firefox/firefox/commit/6c7d0968c84c
https://hg.mozilla.org/integration/autoland/rev/7162199d7e42
Add heinnie.com to dom.timeout.defer_during_load.

Comment 18

[Jeff Muizelaar [:jrmuizel]]

Comment on attachment 9539584 [details]
Bug 2007637. Add heinnie.com to dom.timeout.defer_during_load.

Beta/Release Uplift Approval Request

• User impact if declined/Reason for urgency: Fixes loading of heinnie.com
• Is this code covered by automated tests?: No
• Has the fix been verified in Nightly?: No
• Needs manual test from QE?: No
• If yes, steps to reproduce:
• List of other uplifts needed: None
• Risk to taking this patch: Low
• Why is the change risky/not risky? (and alternatives if risky): Just adds hennie.com to a list of sites where we have more compatibility behaviour
• String changes made/needed:
• Is Android affected?: No

Comment 19

[Cristina Horotan [:chorotan]]

https://hg.mozilla.org/mozilla-central/rev/7162199d7e42

Comment 20

[Donal Meehan [:dmeehan]]

Comment on attachment 9539584 [details]
Bug 2007637. Add heinnie.com to dom.timeout.defer_during_load.

Approved for 148.0b7

Comment 21

[Pulsebot]

https://github.com/mozilla-firefox/firefox/commit/afb9c9146e18
https://hg.mozilla.org/releases/mozilla-beta/rev/f8c49aaea194

Comment 22

[Ryan VanderMeulen [:RyanVM]]

Comment on attachment 9539584 [details]
Bug 2007637. Add heinnie.com to dom.timeout.defer_during_load.

Approved for 147.0.2

Comment 23

[Pulsebot]

https://github.com/mozilla-firefox/firefox/commit/83a42e73fdf5
https://hg.mozilla.org/releases/mozilla-release/rev/8ecf6f93b732

Comment 24

[Catalin Sasca, Desktop Test Engineering [:csasca]]

Attachment: heinnie issue.png

I was able to reproduce the issue on Firefox 148.0a1 (2025-12-23) on Windows 10 by using the link provided in Comment 0.

Unfortunately this is still reproducible on Firefox 147.0.2, Firefox 148.0b7 and Firefox 149.0a1 (2026-01-26) on the same machine. Attached a screenshot with 147.0.2 and 148.0b7 in view.

Comment 25

[Bernadett Farkas]

Attachment: Products do not load RC 148.0 / Nightly 1490a1

Verified, the issue can be reproduced with the RC v148. Products do not load on the page intermittently.

Tested with:

• Browser / Version: Firefox 148.0-candidate build 1 / Firefox Nightly 149.0a1 (2026-02-17)
• Operating System: Desktop

Comment 26

[Kagami Rosylight [:saschanaz] (they/them)]

Disabling the pref does fix the problem. Wrong domain? Not sure *.heinnie.com actually includes heinnie.com. NI already exists to Jeff, so can't add another.

Comment 27

[Kagami Rosylight [:saschanaz] (they/them)]

I'm not seeing how setTimeout is involved here, so keeping the flag.

Comment 28

[Kagami Rosylight [:saschanaz] (they/them)]

In comment #11, n.scope.kScope.recentCurrencySymbol is set when the site is not broken, and unset when site is broken. Not sure how this is set though? There is this inline script:

                        if (
                            typeof klevu.search.modules.kmcInputs.base
                                .getPriceFormatterObject == "function" &&
                            productListResults &&
                            productListResults.length
                        ) {
                            var currencyFormatter = klevu.search.modules.kmcInputs.base.getPriceFormatterObject(
                                scope.data.template.query.productList.result[0].currency
                            );
                            if (currencyFormatter) {
                                currSymbol = currencyFormatter.string;
                                var currSymbolHtml = document.createElement("textarea");
                                currSymbolHtml.innerHTML = currSymbol;
                                currSymbol = currSymbolHtml.value;
                                scope.recentCurrencySymbol = currSymbol;
                                currSymbolAtEnd = currencyFormatter.atEnd
                                    ? currencyFormatter.atEnd
                                    : false;
                            }
                        } else {
                            if (
                                currSymbol == "" &&
                                typeof scope.recentCurrencySymbol != "undefined" &&
                                scope.recentCurrencySymbol
                            ) {
                                currSymbol = scope.recentCurrencySymbol;
                            }
                        }

But this is called only after comment #11.

Edit: Actually never mind, it is set later but I was logging the live object, so sure the value is set when I see the object. Not when the callback is called though.