Bug 41374
| Summary: | Rename releaseRef to leakRef | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Darin Adler <darin> |
| Component: | Web Template Framework | Assignee: | Darin Adler <darin> |
| Status: | RESOLVED FIXED | ||
| Severity: | Normal | CC: | ddkilzer |
| Priority: | P2 | ||
| Version: | 528+ (Nightly build) | ||
| Hardware: | All | ||
| OS: | All | ||
| Bug Depends on: | 68672, 71422, 71423 | ||
| Bug Blocks: | 41321 | ||
Darin Adler
Rename the releaseRef function in PassRefPtr to leakRef. Might also want to add a leakRef function to RefPtr.
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
David Kilzer (:ddkilzer)
There is also a releaseRef() method defined in RetainPtr.h.
Darin Adler
JSRetainPtr.h, PassRefPtr.h, RetainPtr.h, and COMPtr.h all now have leakRef functions.
But many old call sites still are calling releaseRef, so we still have releaseRef functions in all of these classes that call over to leakRef.
Remaining work: Switch all the call sites to leakRef, and delete releaseRef functions entirely.