Closed
Bug 280922
Opened 20 years ago
Closed 19 years ago
setInterval seems to be interfering with one another between tabs/windows
Categories
(Core :: Widget: Win32, defect)
Tracking
()
RESOLVED
FIXED
People
(Reporter: martijn.martijn, Assigned: jst)
References
Details
(Keywords: regression, testcase, Whiteboard: [ETA ?])
Attachments
(3 files)
(deleted),
text/html
|
Details | |
(deleted),
patch
|
Details | Diff | Splinter Review | |
(deleted),
text/html
|
Details |
See testcase:
- Open this testcase twice in two tabs or windows
They both should act the same, but for me the second one opened doesn't animate.
There seems to be heavy interference from the earlier opened testcase.
I'm seeing more strange behavior.
When opening a new window (of Firefox), my toolbar bookmarks menu is not
expanded, when I have the testcase still open. That seems like another symptom
of this timer related issue.
As I write this,I still get this behavior after I closed the tab of the testcase
and my cpu is still 100%.
I suspect this has got something to do with the issues mentioned bug 277762,
comment 10. Because when I use 10 instead of 15 moving points, I get the
jerky/not smooth behavior when switching tabs between the two identical testcases.
Reporter | ||
Comment 1•20 years ago
|
||
I'm not seeing the bug in:
Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.3a) Gecko/20021212
But I'm seeing the bug in:
Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.3b) Gecko/20030210
Reporter | ||
Comment 2•20 years ago
|
||
Bug 194994 related, perhaps?
Comment 3•20 years ago
|
||
I can't reproduce this -- I just loaded this in 7 or 8 tabs, and it all worked
fine...
Reporter | ||
Comment 4•20 years ago
|
||
Indeed, I tried it now also in Linux, but the problem doesn't exist there. So
probably a Windows only problem.
Comment 5•20 years ago
|
||
Over to widgetry... sounds like an event loop issue?
Assignee: general → win32
Component: DOM: Level 0 → Widget: Win32
Reporter | ||
Comment 6•20 years ago
|
||
I backed out the fix from bug 183604 (with a minor adjustment, I had to add
nsresult rv, otherwise it would not compile), and I can't see the bug with this
backed out build.
I haven't tested this build yet without this back out (that's something I'll do
shortly).
Reporter | ||
Comment 7•20 years ago
|
||
Hmm, false alarm. After using a normal nsGlobalWindow.cpp I still can't see the
bug with my own build (or should I rebuild everything, not just the dom directory?).
Comment 8•20 years ago
|
||
you also need to build layout/build when changing dom/
I see no problem in either my current Mozilla/5.0 (Windows; U; Windows NT 5.1;
en-US; rv:1.8b) Gecko/20050125 Firefox/1.0+ build, or in a downloaded nightly
20040204 Windows Firefox build. All windows simultaneously open animate equally
well, though CPU usage does climb with additional windows
just this bug report open, idle 0%
1 testcase window 20%
1 testcase + wave mouse around 40%
2 testcase windows 45%
3 75%
4 100%
and animation quality degrades in all windows as the CPU strains harder.
Martijn, I hope that two year old build you were testing with in comment 1 is
only an attempt to track down when the problem first appeared.
Hardware: PC → All
Comment 10•20 years ago
|
||
Oops. It was a 20050204 build in the previous comment; today's.
Comment 11•20 years ago
|
||
mmm timers. Brendan, jst, any idea what's up here? Note that each object being
animated has its own interval timer in this testcase. Could we be processing
things slowly enough that we get past the point where the interval timer should
have fired and then fail to fire it? Keep in mind that Martijn is using pretty
slow hardware.
Reporter | ||
Comment 12•20 years ago
|
||
Ok, I've had a new Firefox non-debug build that showed this bug (I checked it)
and then I applied the "What I backed out" 'patch'.
After that I rebuilt dom/ and layout/build/ and after that the Firefox build
didn't show this bug anymore.
So I think the fix from bug bug 183604 definitely has got something to do with
this bug.
I'm testing this on a 800MHz Duron/128MB memory/Win2000.
(In reply to comment #9)
> Martijn, I hope that two year old build you were testing with in comment 1 is
> only an attempt to track down when the problem first appeared.
Yes, exactly. Unfortunately there are no builds available inbetween at
archive.mozilla.org, otherwise I could give a more narrow regression range.
What I backed out was just a wild guess (but seems to be a lucky shot).
Comment 13•20 years ago
|
||
I know it's a long shot since it's been broken since 1.3, but it'd be good to
sort this timer issue out for 1.8...
Flags: blocking1.8b?
Comment 14•20 years ago
|
||
Agreed, let's try to get this solved for b1 or b2.
Flags: blocking1.8b? → blocking1.8b+
Updated•20 years ago
|
Flags: blocking1.8b+ → blocking1.8b2+
Updated•20 years ago
|
Attachment #173419 -
Attachment is patch: true
Comment 15•20 years ago
|
||
Brendan, JST, BZ, any chance something's gonna happen here in the next few days?
If not, can we push this out to 1.8b3 and hope for some work on it then.
Comment 16•20 years ago
|
||
I'm not going to be able to do anything with this; being unable to reproduce (no
Windows) doesn't help. :(
Updated•20 years ago
|
Flags: blocking1.8b2+ → blocking1.8b3+
Comment 17•19 years ago
|
||
Pav, can you take a look at this and see if you can help?
Comment 18•19 years ago
|
||
Johnny, any chance you can look into this? I suspect you're not the right
person, but I'm desperate ;-)
Assignee: win32 → jst
Assignee | ||
Comment 19•19 years ago
|
||
I doubt I'll have any time for this for 1.8, nor do I see it absolutely
necessary to fix it for 1.8
Reporter | ||
Comment 20•19 years ago
|
||
Well, when I have two tabs of the testcase open, the url bar does update itself
when switching tabs or opening a new url only very occassionally (after 20s or
more), so I'm seeing the wrong url in the url bar when switching tabs.
Reporter | ||
Comment 21•19 years ago
|
||
When clicking on the "open paypal" button I can see htpp://www.paypal.com in
the url bar, but I see the text "This is content that doesn't come from
paypal".
Comment 22•19 years ago
|
||
Is this just a symptom of but 291386?
Updated•19 years ago
|
Flags: blocking-aviary1.1+
Whiteboard: [cb] no progress for 1.8b3? (defer?)
Updated•19 years ago
|
Flags: blocking1.8b3+ → blocking1.8b4+
Updated•19 years ago
|
Whiteboard: [cb] no progress for 1.8b3? (defer?) → [ETA ?]
Updated•19 years ago
|
Flags: blocking1.8b4+ → blocking1.8b4-
Updated•19 years ago
|
Flags: blocking-aviary1.5+
Reporter | ||
Updated•19 years ago
|
Flags: blocking1.9a1?
Comment 23•19 years ago
|
||
(In reply to comment #22)
> Is this just a symptom of but 291386?
at least a side effect according to Brendan's setting of the dependency
Reporter | ||
Comment 24•19 years ago
|
||
Darin, you may be interested in this, since you seem to be working on bug 291386.
The testcase might be useful to you or the "What I backed out patch", maybe (otherwise sorry to cc you).
Reporter | ||
Comment 25•19 years ago
|
||
This is working a lot better with the build Darin provided in bug 326273.
Depends on: nsIThreadManager
Reporter | ||
Comment 26•19 years ago
|
||
This is working almost perfect now!
Both testcases are fixed.
With the first testcase I notice a little sluggishness with the ui (basically what I mentioned in the first part of bug 326273, comment 9) when I have approximately 10 tabs of the first testcase open. But that's something I certainly can live with.
Status: NEW → RESOLVED
Closed: 19 years ago
Flags: blocking1.9a1?
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•