Commit af81bf7d authored by darin@apple.com's avatar darin@apple.com
Browse files

2008-04-01 Darin Adler <darin@apple.com>

        Reviewed by Mitz.

        - fix <rdar://problem/5833351> REGRESSION: Leak in JSCustomXPathNSResolver::create
        - fix <rdar://problem/5833216> REGRESSION: Leak in CSSStyleSelector::addViewportDependentMediaQueryResult

        * bindings/js/JSCustomXPathNSResolver.cpp:
        (WebCore::JSCustomXPathNSResolver::create): Added a missing adoptRef.
        * css/CSSStyleSelector.cpp:
        (WebCore::CSSStyleSelector::~CSSStyleSelector): Added a missing call to deleteAllValues.



git-svn-id: http://svn.webkit.org/repository/webkit/trunk@31530 268f45cc-cd09-0410-ab3c-d52691b4dbfc
parent 432b13ef
2008-04-01 Darin Adler <darin@apple.com>
Reviewed by Mitz.
- fix <rdar://problem/5833351> REGRESSION: Leak in JSCustomXPathNSResolver::create
- fix <rdar://problem/5833216> REGRESSION: Leak in CSSStyleSelector::addViewportDependentMediaQueryResult
* bindings/js/JSCustomXPathNSResolver.cpp:
(WebCore::JSCustomXPathNSResolver::create): Added a missing adoptRef.
* css/CSSStyleSelector.cpp:
(WebCore::CSSStyleSelector::~CSSStyleSelector): Added a missing call to deleteAllValues.
2008-04-01 Darin Adler <darin@apple.com>
 
Reviewed by Sam.
......@@ -53,7 +53,7 @@ PassRefPtr<JSCustomXPathNSResolver> JSCustomXPathNSResolver::create(KJS::ExecSta
return 0;
}
return new JSCustomXPathNSResolver(resolverObject, toJSDOMWindow(exec->dynamicGlobalObject())->impl()->frame());
return adoptRef(new JSCustomXPathNSResolver(resolverObject, toJSDOMWindow(exec->dynamicGlobalObject())->impl()->frame()));
}
JSCustomXPathNSResolver::JSCustomXPathNSResolver(JSObject* customResolver, Frame* frame)
......
......@@ -313,6 +313,7 @@ CSSStyleSelector::~CSSStyleSelector()
::delete m_rootDefaultStyle;
delete m_authorStyle;
delete m_userStyle;
deleteAllValues(m_viewportDependentMediaQueryResults);
}
static CSSStyleSheet* parseUASheet(const char* characters, unsigned size)
......
Supports Markdown
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment