Bug 1218371
Summary: | [scale] JavaScriptException "elem is null" in Firefox 37 | ||
---|---|---|---|
Product: | Red Hat Enterprise Virtualization Manager | Reporter: | Vojtech Szocs <vszocs> |
Component: | ovirt-engine-webadmin-portal | Assignee: | Greg Sheremeta <gshereme> |
Status: | CLOSED WONTFIX | QA Contact: | Eldad Marciano <emarcian> |
Severity: | high | Docs Contact: | |
Priority: | medium | ||
Version: | 3.5.1 | CC: | ecohen, emarcian, gklein, lsurette, mgoldboi, rbalakri, Rhev-m-bugs, yeylon, ylavi |
Target Milestone: | --- | ||
Target Release: | 3.5.5 | ||
Hardware: | x86_64 | ||
OS: | Linux | ||
Whiteboard: | ux | ||
Fixed In Version: | Doc Type: | Bug Fix | |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2015-08-04 15:46:35 UTC | Type: | Bug |
Regression: | --- | Mount Type: | --- |
Documentation: | --- | CRM: | |
Verified Versions: | Category: | --- | |
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
Cloudforms Team: | --- | Target Upstream Version: | |
Embargoed: | |||
Bug Depends On: | |||
Bug Blocks: | 1223194 |
Description
Vojtech Szocs
2015-05-04 18:05:35 UTC
See bug 1207299 for reference and description of the original issue. Greg, please be in touch with Eldad [emarcian] for reproducing, assistance in testing, etc. Note that this seems to happen only in FF (not in Chrome), and maybe only in WAN, rather than on LAN (strange...). we may want to backport that into 3.5.z. note: this reproduces only for FF versions > 36. I strongly recommend trying to test on FF38, which is about to become the standard FF version provided within RHEL. I'm unable to reproduce on my own, but I don't think I need to since we have the stack trace. I suspect the issue is this line of code in GWT's CompositeCell: public boolean isEditing(Context context, Element parent, C value) { ** Element curChild = getContainerElement(parent).getFirstChildElement(); ** for (HasCell<C, ?> hasCell : hasCells) { ... curChild is never checked for being null later in the method, but we're seeing it be null. (It's probably null in the rare circumstance of two concurrent grid refreshes -- certainly possible over a latent network.) So I'd classify this as a GWT bug. We can easily work around by overriding "isEditing" and safely handling nulls in there. after discussing with Greg: we need to fix this issue for 3.6 first. however, in 3.6 we have made significant changes: (a) upgraded GWT and GWT-related dependencies. (b) refactored a lot of code around table-cells for the PatternFly-tooltips changes. we don't even know whether this issue reproduces in 3.6. @Eldad - do you have a 3.6 environment (similar to your 3.5 environment) on which you can attempt to reproduce the problem? we need to invest some time and effort for this, in progress Met with Eldad today. He is going to try to reproduce on master. He'll follow up in a day or two. Eldad reports the issue cannot be reproduced on master. This is good -- I think that means it was a bug in version of GWT in oVirt 3.5.x, and has since been fixed in the newer version of GWT we're using in oVirt 3.6. Einav / PM can advise on pursuing a workaround for z-stream. (Such a small issue ... doesn't seem like z-stream material to me.) |