Release Oilpan heap singletons prior to LSan leak detection.

third_party/WebKit/Source/core/css/CSSDefaultStyleSheets.cpp

 /*
  * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
  *           (C) 2004-2005 Allan Sandfeld Jensen (kde@carewolf.com)
  * Copyright (C) 2006, 2007 Nicholas Shanks (webkit@nickshanks.com)
  * Copyright (C) 2005, 2006, 2007, 2008, 2009, 2010, 2011, 2012 Apple Inc. All rights reserved.
  * Copyright (C) 2007 Alexey Proskuryakov <ap@webkit.org>
  * Copyright (C) 2007, 2008 Eric Seidel <eric@webkit.org>
  * Copyright (C) 2008, 2009 Torch Mobile Inc. All rights reserved. (http://www.torchmobile.com/)
  * Copyright (c) 2011, Code Aurora Forum. All rights reserved.
  * Copyright (C) Research In Motion Limited 2011. All rights reserved.
@@ skipping 50 matching lines @@
     DEFINE_STATIC_LOCAL(const MediaQueryEvaluator, staticPrintEval, ("print"));
     return staticPrintEval;
 }
 
 static PassRefPtrWillBeRawPtr<StyleSheetContents> parseUASheet(const String& str)
 {
     RefPtrWillBeRawPtr<StyleSheetContents> sheet = StyleSheetContents::create(CSSParserContext(UASheetMode, 0));
     sheet->parseString(str);
     // User Agent stylesheets are parsed once for the lifetime of the renderer
     // process and are intentionally leaked.
-    WTF_ANNOTATE_LEAKING_OBJECT_PTR(sheet.get());
+    LEAK_SANITIZER_IGNORE_OBJECT(sheet.get());

[haraken, 2015/12/07 14:46:10]

This is the only use case of WTF_ANNOTATE_LEAKING_OBJECT_PTR. Shall we just
replace it with LEAK_SANITIZER_DISABLED_SCOPE (and remove
WTF_ANNOTATE_LEAKING_OBJECT_PTR)?

[sof, 2015/12/07 15:09:28]

It is also used by Persistent*::registerAsStaticReference(), but those can be
avoided.

Actually, I'm not so sure we need to mark this object as individually ignored
here as all of these UA sheet objects belong to the
CSSDefaultStyleSheet::instance() singleton. Will investigate.

[sof, 2015/12/07 16:59:16]

Ah, this is just right as it is. The StyleSheetContents objects are reachable by
other objects, hence marking them as ignored is what's wanted. And marking the
object itself as ignored, rather than disabling LSan just for the duration of
its (initial) construction, will also let us not flag leaks against anything
that the object exclusively refers to at shutdown.

Same goes for Persistent*::registerAsStaticReference() uses, they're also
needed. Hence leaving this as it is.

     return sheet.release();
 }
 
 CSSDefaultStyleSheets::CSSDefaultStyleSheets()
     : m_defaultStyle(nullptr)
     , m_defaultMobileViewportStyle(nullptr)
     , m_defaultQuirksStyle(nullptr)
     , m_defaultPrintStyle(nullptr)
     , m_defaultViewSourceStyle(nullptr)
     , m_defaultXHTMLMobileProfileStyle(nullptr)
@@ skipping 109 matching lines @@
     visitor->trace(m_defaultStyleSheet);
     visitor->trace(m_mobileViewportStyleSheet);
     visitor->trace(m_quirksStyleSheet);
     visitor->trace(m_svgStyleSheet);
     visitor->trace(m_mathmlStyleSheet);
     visitor->trace(m_mediaControlsStyleSheet);
     visitor->trace(m_fullscreenStyleSheet);
 }
 
 } // namespace blink