Bug 25596 - Chromium build bustage caused by r43317 (making RefCountedBase::m_refCount private)
Summary: Chromium build bustage caused by r43317 (making RefCountedBase::m_refCount pr...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Platform (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Darin Fisher (:fishd, Google)
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-05-06 14:00 PDT by Darin Fisher (:fishd, Google)
Modified: 2009-05-06 16:53 PDT (History)
1 user (show)

See Also:


Attachments
v1 patch (1.15 KB, patch)
2009-05-06 14:04 PDT, Darin Fisher (:fishd, Google)
dglazkov: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Darin Fisher (:fishd, Google) 2009-05-06 14:00:09 PDT
Chromium build bustage caused by r43317 (making RefCountedBase::m_refCount private)

See http://trac.webkit.org/changeset/43317

Our AccessibilityObjectWrapper still needs the old semantics of RefCounted (when it was zero initialized).  We haven't fixed that yet because we are waiting to get our WebKit layer side into svn.webkit.org.  That side uses COM style reference counting, which leads to needing the object to be initialized with a ref count of 0.  This is all super annoyingly ugly, and I really want it to be fixed.  But, I want the WebKit API to be upstreamed more than that, and so I am putting off this :-(

Ugly workaround patch coming up...
Comment 1 Darin Fisher (:fishd, Google) 2009-05-06 14:04:20 PDT
Created attachment 30063 [details]
v1 patch
Comment 2 Dimitri Glazkov (Google) 2009-05-06 14:06:21 PDT
Comment on attachment 30063 [details]
v1 patch

Eek.
Comment 3 Darin Fisher (:fishd, Google) 2009-05-06 14:07:35 PDT
Landed as:  http://trac.webkit.org/changeset/43322
Comment 4 Darin Adler 2009-05-06 15:10:57 PDT
Another way to fix this would be to not use RefCounted until you can match its normal semantics. RefPtr works fine as long as there are ref and deref functions; they don't have to come from the RefCounted class template.
Comment 5 Darin Fisher (:fishd, Google) 2009-05-06 16:53:19 PDT
> Another way to fix this would be to not use RefCounted until you can match its
> normal semantics. RefPtr works fine as long as there are ref and deref
> functions; they don't have to come from the RefCounted class template.

That's a very good suggestion!