My favorites | Sign in
Logo
             
New issue | Search
for
| Advanced search | Search tips
Issue 19677: Browser crashes occasionally when playing a lot of videos
1 person starred this issue and may be notified of changes. Back to list
 
Reported by xlyuan@chromium.org, Aug 18, 2009
This is found during the video hardware configuration testing.

Build: 3.0.197.11 (Official Build 22553)
   OS: Windows XP

Other Browsers:
Webkit nightly: N/A
    Firefox3.5: N/A
           IE8: N/A

Steps:
1. Launch Chrome
2. Go to http://fbarchard-v64.ad/testmatrix/
3. Randomly play some video
4. Observe 

Result:
Browser crashes ("Aw, Snap!" page shows up) occasionally, hmm, it happens 3 
or 4 times this afternoon.

Expected:
Browser should not crash

The machine configrations are:
CPU: Pentium III 
Speed: 1.00 GHz 
RAM: 512 MB 
Graphics: Intel 82815 
Sound: Analog Devices AD1885 AC'97 

Comment 1 by scherkus@chromium.org, Aug 19, 2009
Was this over remote desktop?  Do you know if it's caused by any particular video? 
Are you watching many different videos, or simply letting a single video play?
Status: Available
Labels: HTML5
Comment 2 by xlyuan@chromium.org, Aug 19, 2009
1. Yes, it was over remote desktop
2. It seems the crash could happen to any video, not just one particular video
3. I played the videos on the testing page one by one, so there was only one video 
playing at the same time

I just tried to replay all the XXX2.mp4 files, XXX2.ogv files and XXX6.mp4 files, but 
I didn't get any luck to reproduce it :-(


Comment 3 by scherkus@chromium.org, Sep 01, 2009
Touching base again.. was it a whole browser crash or was it only a tab?
Comment 4 by xlyuan@chromium.org, Sep 01, 2009
hey, when the crash happened, it was the single tab not the whole browser.
Comment 5 by scherkus@chromium.org, Sep 01, 2009
Ok I have a patch that'll land soon that *might* fix the crash.  I couldn't get it
reproducing, but I found some code that could lead to a crash if there was a full
moon and the stars were aligned, etc... :)

But running over remote desktop has a tendency to trigger the worst-case scenarios,
so you never know!
Comment 6 by bugdroid1@chromium.org, Sep 01, 2009
The following revision refers to this bug:
    http://src.chromium.org/viewvc/chrome?view=rev&revision=25125 

------------------------------------------------------------------------
r25125 | scherkus@chromium.org | 2009-09-01 17:11:38 -0700 (Tue, 01 Sep 2009) | 9 lines
Changed paths:
   M http://src.chromium.org/viewvc/chrome/trunk/src/webkit/api/src/WebMediaPlayerClientImpl.cpp?r1=25125&r2=25124

Fixes crash when switching to new tabs containing audio/video elements.

Cause was we're calling platformContext(), one of the few GraphicsContext operations that asserts if painting is disabled.

BUG=19677,20138
TEST=open a video in a new background tab, wait a bit, then switch to it -- shouldn't crash

Review URL: http://codereview.chromium.org/183038

------------------------------------------------------------------------

Comment 7 by scherkus@chromium.org, Sep 01, 2009
Alright the fix is in.. not in time for this week's dev channel but if you run a
nightly build with ogg content you can look for crashes :)

I'm going to mark this as fixed but feel free to re-open or file a new bug if you run
into any more crashes.
Status: Fixed
Comment 8 by bugdroid1@chromium.org, Sep 01, 2009
The following revision refers to this bug:
    http://src.chromium.org/viewvc/chrome?view=rev&revision=25136 

------------------------------------------------------------------------
r25136 | laforge@chromium.org | 2009-09-01 17:56:56 -0700 (Tue, 01 Sep 2009) | 13 lines
Changed paths:
   M http://src.chromium.org/viewvc/chrome/branches/195/src/webkit/api/src/WebMediaPlayerClientImpl.cpp?r1=25136&r2=25135

Merge 25125 - Fixes crash when switching to new tabs containing audio/video elements.

Cause was we're calling platformContext(), one of the few GraphicsContext operations that asserts if painting is disabled.

BUG=19677,20138
TEST=open a video in a new background tab, wait a bit, then switch to it  shouldn't crash

Review URL: http://codereview.chromium.org/183038


TBR=scherkus@chromium.org

Review URL: http://codereview.chromium.org/172095
------------------------------------------------------------------------

Comment 9 by mal.chromium, Dec 18, 2009
(No comment was entered for this change.)
Labels: -Area-BrowserBackend Area-Internals
Comment 10 by mal.chromium, Dec 18, 2009
(No comment was entered for this change.)
Labels: Internals-Video
Sign in to add a comment

Powered by Google Project Hosting