Bug 58283

Summary:

LayerChromium/CCLayerImpl ownership fix

Product:

WebKit

Reporter:

Alexey Marinichev <amarinichev>

Component:

New Bugs

Assignee:

Alexey Marinichev <amarinichev>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

commit-queue, enne, jamesr

Priority:

Version:

528+ (Nightly build)

Hardware:

Other

OS:

OS X 10.5

Attachments:

Description	Flags
Patch	none
Patch	none

Description Alexey Marinichev 2011-04-11 16:16:55 PDT

LayerChromium/CCLayerImpl ownership fix

Comment 1 Alexey Marinichev 2011-04-11 16:20:57 PDT

Created attachment 89116 [details]
Patch

Comment 2 James Robinson 2011-04-11 16:25:34 PDT

Comment on attachment 89116 [details]
Patch

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

OK

> Source/WebCore/platform/graphics/chromium/cc/CCLayerImpl.cpp:154
> +    // m_owner is reset by LayerChromium's destructor.

comment isn't really helpful here, i would just delete

Comment 3 Alexey Marinichev 2011-04-11 16:27:29 PDT

Created attachment 89119 [details]
Patch

Comment 4 WebKit Commit Bot 2011-04-11 19:22:24 PDT

Comment on attachment 89119 [details]
Patch

Clearing flags on attachment: 89119

Committed r83551: <http://trac.webkit.org/changeset/83551>

Comment 5 WebKit Commit Bot 2011-04-11 19:22:29 PDT

All reviewed patches have been landed.  Closing bug.