1306683 - Unresponsive Script Warning on wellsfargo.com

Status: RESOLVED WORKSFORME

(Core :: JavaScript Engine, defect, P3)

Description

[Erin Lancaster [:elan]]

Attachment: Screen Shot 2016-09-30 at 10.38.22 AM.png

Firefox 50.0b2, Mac OS X v10.11.6, MacBook Pro 2.7 GHz

STR:

1) Launch Firefox and have 10-15 tabs open (possibly more)
2) Allow browser to go idle overnight at least once
3) Visit https://www.wellsfargo.com
4) The site will start to load, then kind of freeze, then sometimes you see the progress spinner (from the OS) and for sure you will see the slow script dialog prompt

As a user, this dialog is meaningless and the various options make it worse. If we can't fix this problem, I'd recommend we at least improve the user experience. 

This issue has become worse as of 49 and 50. It seemed to get better with 48. It was bad in 46 and 47. This is my #1 problem with Firefox and I use it for hours all day, every day. If there are profiles needed, please send me instructions via IRC.

Comment 1

[Erin Lancaster [:elan]]

Actual tab count = 60

Comment 2

[Brad Lassey [:blassey] (use needinfo?)]

Shorlander, seems like we could do better for our release population here. I'd suggest we put the debug option behind a pref. We could also simplify the messaging to something like "wellsfargo.com seems to be hung" and use less technical language than "stop script".

Also note that Android has a similar dialog for hung apps. I think the options it gives are "kill app" and "wait".

Comment 3

[:Gijs (he/him)]

Separate from the point in comment #2 about language and UI/UX, it would still be useful to figure out *why* we're showing the slow script dialog so often on wellsfargo.com. It feels like separate bugs on the two issues would be useful.

Comment 4

[Brad Lassey [:blassey] (use needinfo?)]

Good point, created bug 1307125 to track improving the dialog. We can use this bug to track the problem with WellsFargo

Comment 5

[Larissa Shapiro]

Also experiencing this one.

Comment 6

[Brad Lassey [:blassey] (use needinfo?)]

Naveed, do you have someone who can figure out what's going on here?

Comment 7

[Naveed Ihsanullah [:naveed]]

Arai,

:jandem mentioned you were working on bugs of this class (https://bugzilla.mozilla.org/show_bug.cgi?id=1284511). Would you please take a look?

Comment 8

[Tooru Fujisawa [:arai]]

I'll try reproducing it (so, it may take 1-2 days).
meanwhile, can you provide some more information?

1. does this issue happen everywhere in the website?
   the screenshot looks different page than the URL in the STR
   does it mean this happens when you click some link?
2. do I need to login?  or just open the URL in clean profile?
3. is the filename and line number shown in the unresponsive dialog always same?
   or is it totally random?
4. does the issue happen without other tabs?
   if it only happens when there's other tabs, what kind of tabs do I need to open?
5. does the issue happen in safe mode?

Comment 9

[Tooru Fujisawa [:arai]]

here's pretty print of the code mentioned in the screenshot.

  // https://connect.secure.wellsfargo.com/ATADUN/2.2/w/w-642409/sync/js/
  // line 33
  nds.common.bi.getWebGLInfo = function() {
    for(var a = {},
            b = document.createElement("canvas"),
            c = ["webgl", "experimental-webgl", "moz-webgl", "webkit-3d"],
            d,
            e = 0;
        e < c.length;
        e++)
      try {
        if (d = b.getContext(c[e])) {
          a.ContextName=c[e];
          break
        }
      } catch (f) {
      }

    if (!(d && "getParameter" in d))
      return null;

    b = "VENDOR VERSION RENDERER SHADING_LANGUAGE_VERSION DEPTH_BITS MAX_VERTEX_ATTRIBS MAX_VERTEX_TEXTURE_IMAGE_UNITS MAX_VARYING_VECTORS MAX_VERTEX_UNIFORM_VECTORS MAX_COMBINED_TEXTURE_IMAGE_UNITS MAX_TEXTURE_SIZE MAX_CUBE_MAP_TEXTURE_SIZE NUM_COMPRESSED_TEXTURE_FORMATS MAX_RENDERBUFFER_SIZE MAX_VIEWPORT_DIMS ALIASED_LINE_WIDTH_RANGE ALIASED_POINT_SIZE_RANGE".split(" ");

  // line 34
    for (c = 0; c < b.length; c++)
      e = b[c],
      e in d &&
        (a[e] = d.getParameter(d[e]));

    return a
  };
  // ...

so this might be related to bug 887016, if the alert is shown for same line every time, as there's String#split.

Comment 10

[Tooru Fujisawa [:arai]]

So far I don't see Unresponsive Script Warning.

Comment 11

[Tooru Fujisawa [:arai]]

still not reproducible.

Comment 12

[Erin Lancaster [:elan]]

I'm so sorry for the delay, let me test this again and provide answers asap.

Comment 13

[Erin Lancaster [:elan]]

update: I stopped going to the wells fargo website via Firefox after I hit that dialog all the time. As of re-testing on b7, it's not happening anymore. I can try to go back to previous builds to answer the questions. It will be a couple of days.

Comment 14

[Wayne Mery (:wsmwk)]

Is/was this a Mac issue?   Because I cannot reproduce on win7 nightly build with 500+ tabs.

Comment 15

[Erin Lancaster [:elan]]

Yes, I reported it above with: Firefox 50.0b2, Mac OS X v10.11.6, MacBook Pro 2.7 GHz. I can resolve this as WFM.