Closed Bug 427853 Opened 14 years ago Closed 14 years ago
IAb Manager::get Directory
As part of the refactoring, I now want to define nsIAbManager::getDirectory, for now this will enable us to "hide" the GetResource that the rdf service uses. This patch defines the function, taking the URI of the rdf resource (as that is currently the only identifier we have), and uses it in all the unit tests. I'll do actual code usages in a separate bug as I know there's at least one patch that should be coming soon that I'll bitrot if I do the code changes now.
New version to incorporate test_mailList1.js that I checked in earlier today.
Comment on attachment 314601 [details] [diff] [review] The fix v2 >+ nsCOMPtr<nsIRDFResource> parentResource; >+ rv = rdfService->GetResource(aURI, >+ getter_AddRefs(parentResource)); >+ NS_ENSURE_SUCCESS(rv, rv); What's it the parent of? Perhaps just call it dirResource? >+ nsCOMPtr<nsIAbDirectory> dir(do_QueryInterface(parentResource, &rv)); >+ NS_ENSURE_SUCCESS(rv, rv); >+ >+ dir.swap(*aResult); >+ return NS_OK; return CallQueryInterface(dirResource, aResult);
Patch checked in with requested changes. Fixed.
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Responsible for increasing trace_malloc_leaks count on Linux nye Dep bloat?
(In reply to comment #4) > Responsible for increasing trace_malloc_leaks count on Linux nye Dep bloat? > Highly unlikely considering this code isn't actually called anywhere within the app yet (only called from unit tests so far).
You need to log in before you can comment on or make changes to this bug.