Bug 481624 - nsTextAccessibleWrap::get_fontFamily leaks, r=marcoz
authorAlexander Surkov <surkov.alexander@gmail.com>
Thu, 05 Mar 2009 21:02:48 +0800
changeset 25766 416767d733ad061eb4fbba957589e3fbec6f5f0f
parent 25765 563c82941f41ce401b1b8dcc8007dc5c5b01c3d4
child 25767 7b7c24f7684a576c0c81465ddd4f93aad236d82c
push id5732
push usersurkov.alexander@gmail.com
push dateThu, 05 Mar 2009 13:03:14 +0000
treeherdermozilla-central@416767d733ad [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersmarcoz
bugs481624
milestone1.9.2a1pre
Bug 481624 - nsTextAccessibleWrap::get_fontFamily leaks, r=marcoz
accessible/src/msaa/nsTextAccessibleWrap.cpp
--- a/accessible/src/msaa/nsTextAccessibleWrap.cpp
+++ b/accessible/src/msaa/nsTextAccessibleWrap.cpp
@@ -271,18 +271,18 @@ STDMETHODIMP nsTextAccessibleWrap::get_f
   }
 
   nsCOMPtr<nsIDeviceContext> deviceContext;
   rc->GetDeviceContext(*getter_AddRefs(deviceContext));
   if (!deviceContext) {
     return E_FAIL;
   }
 
-  nsIFontMetrics *fm;
-  rc->GetFontMetrics(fm);
+  nsCOMPtr<nsIFontMetrics> fm;
+  rc->GetFontMetrics(*getter_AddRefs(fm));
   if (!fm) {
     return E_FAIL;
   }
 
   nsAutoString fontFamily;
   deviceContext->FirstExistingFont(fm->Font(), fontFamily);
   if (fontFamily.IsEmpty())
     return S_FALSE;