Oilpan: Added gyp flag to dump points-to graph information from the Blink GC plugin.

Source/core/xml/XSLImportRule.h

 /*
  * This file is part of the XSL implementation.
  *
  * Copyright (C) 2004, 2006, 2008 Apple Inc. All rights reserved.
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
  * License as published by the Free Software Foundation; either
  * version 2 of the License, or (at your option) any later version.
  *
@@ skipping 45 matching lines @@
 private:
     XSLImportRule(XSLStyleSheet* parentSheet, const String& href);
 
     virtual void setXSLStyleSheet(const String& href, const KURL& baseURL, const String& sheet) OVERRIDE;
 
     // FIXME: Oilpan: This raw pointer is safe and has to be kept raw at this
     // point to avoid cycles. When XSLImportRule is moved to the oilpan heap
     // it should be a Member and be traced.
     XSLStyleSheet* m_parentStyleSheet;
     String m_strHref;
+
+    // FIXME: Oilpan: This persistent causes a direct cycle, but should not leak
+    // because XSLStyleSheet::m_children should only contain child import rules.
+    GC_PLUGIN_IGNORE_CYCLE

[haraken, 2014/03/21 10:31:40]

It's a bit inconsistent that we have a bug number on GC_PLUGIN_IGNORE but we
have a FIXME on GC_PLUGIN_IGNORE_CYCLE. (I'd prefer a FIXME though :-)

[zerny-chromium, 2014/03/21 10:46:19]

This only ignores the field with respect to the cycle detection code, so it has
limited impact (It does not signal a code issue or a plugin bug, just silences a
false positive). The other ignore is much more serious as simply acts as if the
field does not exist. Hopefully the use of GC_PLUGIN_IGNORE will be more
uncommon and the issue quickly resolved.

     RefPtrWillBePersistent<XSLStyleSheet> m_styleSheet;
     ResourcePtr<XSLStyleSheetResource> m_resource;
     bool m_loading;
 };
 
 } // namespace WebCore
 
 #endif // XSLImportRule_h