Bug 68204

Summary:

[chromium] Add temporary diagnostics for LayerTreeHost::commitTo crash

Product:

WebKit

Reporter:

Adrienne Walker <enne>

Component:

New Bugs

Assignee:

Adrienne Walker <enne>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

enne, jamesr, nduca

Priority:

Version:

528+ (Nightly build)

Hardware:

Unspecified

OS:

Unspecified

Bug Depends on:

68814

Bug Blocks:

Attachments:

Description	Flags
Patch	jamesr: review+

Description Adrienne Walker 2011-09-15 18:18:57 PDT

[chromium] Add temporary diagnostics for LayerTreeHost::commitTo crash

Comment 1 Adrienne Walker 2011-09-15 18:24:20 PDT

Created attachment 107579 [details]
Patch

Comment 2 Adrienne Walker 2011-09-15 18:25:49 PDT

I'm not quite sure if this is an initialization error (no LayerTreeHost) or use-after-free of a LayerTreeHost (with a cleared out contents texture manager).  I just want to add two diagnostic crash calls which I will revert after I see what it looks like on the canary.

See: http://code.google.com/p/chromium/issues/detail?id=96719

Comment 3 Nat Duca 2011-09-15 18:26:53 PDT

/me likes

Comment 4 James Robinson 2011-09-15 18:34:57 PDT

Comment on attachment 107579 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=107579&action=review

R=me

> Source/WebCore/ChangeLog:10
> +        No new tests. (OOPS!)

presubmit will reject this

Comment 5 Adrienne Walker 2011-09-15 18:53:40 PDT

Committed r95256: <http://trac.webkit.org/changeset/95256>