150722 2015-10-30 11:05:52 -0700 Minor CGColor leaks seen on bots allocated in WebSystemBackdropLayer.mm 2015-10-30 12:10:07 -0700 1 1 1 Unclassified WebKit WebCore Misc. WebKit Nightly Build Unspecified Unspecified RESOLVED FIXED P2 Normal --- 1 joepeck joepeck commit-queue dino simon.fraser oldest_to_newest 1137897 0 joepeck 2015-10-30 11:05:52 -0700 Minor CGColor leaks seen on bots allocated in WebSystemBackdropLayer.mm * LEAK Call stack: [thread 0x11decb300]: | 0x2 | start | main DumpRenderTreeMain.mm:30 | DumpRenderTreeMain(int, char const**) DumpRenderTree.mm:1430 | dumpRenderTree(int, char const**) DumpRenderTree.mm:1295 | runTestingServerLoop() DumpRenderTree.mm:1186 | runTest(std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char> > const&) DumpRenderTree.mm:2036 | CFRunLoopRunSpecific | __CFRunLoopRun | __CFRunLoopDoSources0 | __CFRUNLOOP_IS_CALLING_OUT_TO_A_SOURCE0_PERFORM_FUNCTION__ | MultiplexerSource::_perform(void*) | MultiplexerSource::perform() | RunloopBlockContext::perform() | CFArrayApplyFunction | RunloopBlockContext::_invoke_block(void const*, void*) | ___ZN27URLConnectionClient_Classic18_withDelegateAsyncEPKcU13block_pointerFvP16_CFURLConnectionPK33CFURLConnectionClientCurrent_VMaxE_block_invoke_2 | ___ZN27URLConnectionClient_Classic26_delegate_didFinishLoadingEU13block_pointerFvvE_block_invoke | -[NSURLConnectionInternal _withActiveConnectionAndDelegate:] | -[NSURLConnectionInternal _withConnectionAndDelegate:onlyActive:] | __65-[NSURLConnectionInternal _withConnectionAndDelegate:onlyActive:]_block_invoke | -[WebCoreResourceHandleAsDelegate connectionDidFinishLoading:] WebCoreResourceHandleAsDelegate.mm:258 | WebCore::ResourceLoader::didFinishLoading(WebCore::ResourceHandle*, double) ResourceLoader.cpp:639 | WebCore::SubresourceLoader::didFinishLoading(double) SubresourceLoader.cpp:374 | WebCore::CachedRawResource::finishLoading(WebCore::SharedBuffer*) CachedRawResource.cpp:104 | WebCore::CachedResource::finishLoading(WebCore::SharedBuffer*) CachedResource.cpp:314 | WebCore::CachedResource::checkNotify() CachedResource.cpp:296 | WebCore::DocumentLoader::notifyFinished(WebCore::CachedResource*) DocumentLoader.cpp:385 | WebCore::DocumentLoader::finishedLoading(double) DocumentLoader.cpp:438 | WebCore::DocumentWriter::end() DocumentWriter.cpp:247 | WebCore::HTMLDocumentParser::finish() HTMLDocumentParser.cpp:452 | WebCore::HTMLDocumentParser::attemptToEnd() HTMLDocumentParser.cpp:424 | WebCore::HTMLDocumentParser::prepareToStopParsing() HTMLDocumentParser.cpp:133 | WebCore::HTMLDocumentParser::attemptToRunDeferredScriptsAndEnd() HTMLDocumentParser.cpp:412 | WebCore::HTMLDocumentParser::end() HTMLDocumentParser.cpp:403 | WebCore::HTMLTreeBuilder::finished() HTMLTreeBuilder.cpp:2939 | WebCore::HTMLConstructionSite::finishedParsing() HTMLConstructionSite.cpp:404 | WebCore::Document::finishedParsing() Document.cpp:4897 | WebCore::FrameLoader::finishedParsing() FrameLoader.cpp:758 | WebCore::FrameLoader::checkCompleted() FrameLoader.cpp:837 | WebCore::FrameLoader::checkCallImplicitClose() FrameLoader.cpp:890 | WebCore::Document::implicitClose() Document.cpp:2663 | WebCore::Document::dispatchWindowLoadEvent() Document.cpp:4067 | WebCore::DOMWindow::dispatchLoadEvent() DOMWindow.cpp:1884 | WebCore::DOMWindow::dispatchEvent(WTF::PassRefPtr<WebCore::Event>, WTF::PassRefPtr<WebCore::EventTarget>) DOMWindow.cpp:1926 | WebCore::EventTarget::fireEventListeners(WebCore::Event*) EventTarget.cpp:208 | WebCore::EventTarget::fireEventListeners(WebCore::Event*, WebCore::EventTargetData*, WTF::Vector<WebCore::RegisteredEventListener, 1ul, WTF::CrashOnOverflow, 16ul>&) EventTarget.cpp:256 | WebCore::JSEventListener::handleEvent(WebCore::ScriptExecutionContext*, WebCore::Event*) JSEventListener.cpp:130 | WebCore::JSMainThreadExecState::call(JSC::ExecState*, JSC::JSValue, JSC::CallType, JSC::CallData const&, JSC::JSValue, JSC::ArgList const&, WTF::NakedPtr<JSC::Exception>&) JSMainThreadExecState.h:56 | JSC::call(JSC::ExecState*, JSC::JSValue, JSC::CallType, JSC::CallData const&, JSC::JSValue, JSC::ArgList const&, WTF::NakedPtr<JSC::Exception>&) CallData.cpp:44 | JSC::call(JSC::ExecState*, JSC::JSValue, JSC::CallType, JSC::CallData const&, JSC::JSValue, JSC::ArgList const&) CallData.cpp:39 | JSC::Interpreter::executeCall(JSC::ExecState*, JSC::JSObject*, JSC::CallType, JSC::CallData const&, JSC::JSValue, JSC::ArgList const&) Interpreter.cpp:1024 | JSC::JITCode::execute(JSC::VM*, JSC::ProtoCallFrame*) JITCode.cpp:80 | vmEntryToJavaScript | llint_entry | 0x47e9f4601028 | WebCore::jsInternalsPrototypeFunctionLayerTreeAsText(JSC::ExecState*) JSInternals.cpp:2779 | WebCore::Internals::layerTreeAsText(WebCore::Document*, int&) const Internals.cpp:1861 | WebCore::Internals::layerTreeAsText(WebCore::Document*, unsigned int, int&) const Internals.cpp:1884 | WebCore::Frame::layerTreeAsText(unsigned int) const Frame.cpp:929 | WebCore::RenderLayerCompositor::layerTreeAsText(unsigned int) RenderLayerCompositor.cpp:1860 | WebCore::RenderLayerCompositor::flushPendingLayerChanges(bool) RenderLayerCompositor.cpp:486 | WebCore::GraphicsLayerCA::flushCompositingState(WebCore::FloatRect const&, bool) GraphicsLayerCA.cpp:1100 | WebCore::GraphicsLayerCA::recursiveCommitChanges(WebCore::GraphicsLayerCA::CommitState const&, WebCore::TransformState const&, float, WebCore::FloatPoint const&, bool) GraphicsLayerCA.cpp:1387 | WebCore::GraphicsLayerCA::recursiveCommitChanges(WebCore::GraphicsLayerCA::CommitState const&, WebCore::TransformState const&, float, WebCore::FloatPoint const&, bool) GraphicsLayerCA.cpp:1387 | WebCore::GraphicsLayerCA::recursiveCommitChanges(WebCore::GraphicsLayerCA::CommitState const&, WebCore::TransformState const&, float, WebCore::FloatPoint const&, bool) GraphicsLayerCA.cpp:1374 | WebCore::GraphicsLayerCA::commitLayerChangesBeforeSublayers(WebCore::GraphicsLayerCA::CommitState&, float, WebCore::FloatPoint const&) GraphicsLayerCA.cpp:1496 | WebCore::GraphicsLayerCA::changeLayerTypeTo(WebCore::PlatformCALayer::LayerType) GraphicsLayerCA.cpp:3361 | WebCore::GraphicsLayerCA::createPlatformCALayer(WebCore::PlatformCALayer::LayerType, WebCore::PlatformCALayerClient*) GraphicsLayerCA.cpp:332 | WebCore::PlatformCALayerCocoa::create(WebCore::PlatformCALayer::LayerType, WebCore::PlatformCALayerClient*) PlatformCALayerCocoa.mm:76 | WebCore::PlatformCALayerCocoa::PlatformCALayerCocoa(WebCore::PlatformCALayer::LayerType, WebCore::PlatformCALayerClient*) PlatformCALayerCocoa.mm:278 | WebCore::PlatformCALayerCocoa::PlatformCALayerCocoa(WebCore::PlatformCALayer::LayerType, WebCore::PlatformCALayerClient*) PlatformCALayerCocoa.mm:270 | -[WebLightSystemBackdropLayer init] WebSystemBackdropLayer.mm:54 | create_color | CGTypeCreateInstance | _CFRuntimeCreateInstance | malloc_zone_malloc 1137898 1 264403 joepeck 2015-10-30 11:06:48 -0700 Created attachment 264403 [PATCH] Proposed Fix 1137916 2 264403 commit-queue 2015-10-30 12:10:03 -0700 Comment on attachment 264403 [PATCH] Proposed Fix Clearing flags on attachment: 264403 Committed r191804: <http://trac.webkit.org/changeset/191804> 1137917 3 commit-queue 2015-10-30 12:10:07 -0700 All reviewed patches have been landed. Closing bug. 264403 2015-10-30 11:06:48 -0700 2015-10-30 12:10:03 -0700 [PATCH] Proposed Fix leak-1.patch text/plain 1649 joepeck ZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZyBiL1NvdXJjZS9XZWJDb3JlL0No YW5nZUxvZwppbmRleCAzMTYxNTA4Li4zNGVlNjMwIDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViQ29y ZS9DaGFuZ2VMb2cKKysrIGIvU291cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMTQg QEAKKzIwMTUtMTAtMzAgIEpvc2VwaCBQZWNvcmFybyAgPHBlY29yYXJvQGFwcGxlLmNvbT4KKwor ICAgICAgICBNaW5vciBDR0NvbG9yIGxlYWtzIHNlZW4gb24gYm90cyBhbGxvY2F0ZWQgaW4gV2Vi U3lzdGVtQmFja2Ryb3BMYXllci5tbQorICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9z aG93X2J1Zy5jZ2k/aWQ9MTUwNzIyCisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BT ISkuCisKKyAgICAgICAgKiBwbGF0Zm9ybS9ncmFwaGljcy9jYS9jb2NvYS9XZWJTeXN0ZW1CYWNr ZHJvcExheWVyLm1tOgorICAgICAgICAoLVtXZWJMaWdodFN5c3RlbUJhY2tkcm9wTGF5ZXIgaW5p dF0pOgorICAgICAgICAoLVtXZWJEYXJrU3lzdGVtQmFja2Ryb3BMYXllciBpbml0XSk6CisKIDIw MTUtMTAtMjkgIEFsZXggQ2hyaXN0ZW5zZW4gIDxhY2hyaXN0ZW5zZW5Ad2Via2l0Lm9yZz4KIAog ICAgICAgICBDTWFrZSBidWlsZCBmaXguCmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViQ29yZS9wbGF0 Zm9ybS9ncmFwaGljcy9jYS9jb2NvYS9XZWJTeXN0ZW1CYWNrZHJvcExheWVyLm1tIGIvU291cmNl L1dlYkNvcmUvcGxhdGZvcm0vZ3JhcGhpY3MvY2EvY29jb2EvV2ViU3lzdGVtQmFja2Ryb3BMYXll ci5tbQppbmRleCA0ZDEzMDgxLi41ZjdhNTVmIDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViQ29yZS9w bGF0Zm9ybS9ncmFwaGljcy9jYS9jb2NvYS9XZWJTeXN0ZW1CYWNrZHJvcExheWVyLm1tCisrKyBi L1NvdXJjZS9XZWJDb3JlL3BsYXRmb3JtL2dyYXBoaWNzL2NhL2NvY29hL1dlYlN5c3RlbUJhY2tk cm9wTGF5ZXIubW0KQEAgLTUxLDcgKzUxLDcgQEAgdXNpbmcgbmFtZXNwYWNlIFdlYkNvcmU7CiAj ZW5kaWYKIAogICAgIENHRmxvYXQgY29tcG9uZW50c1s0XSA9IHsgMC44LCAwLjgsIDAuOCwgMC44 IH07Ci0gICAgW3N1cGVyIHNldEJhY2tncm91bmRDb2xvcjpDR0NvbG9yQ3JlYXRlKGNhY2hlZENH Q29sb3JTcGFjZShDb2xvclNwYWNlU1JHQiksIGNvbXBvbmVudHMpXTsKKyAgICBbc3VwZXIgc2V0 QmFja2dyb3VuZENvbG9yOmFkb3B0Q0YoQ0dDb2xvckNyZWF0ZShjYWNoZWRDR0NvbG9yU3BhY2Uo Q29sb3JTcGFjZVNSR0IpLCBjb21wb25lbnRzKSkuZ2V0KCldOwogCiAgICAgcmV0dXJuIHNlbGY7 CiB9CkBAIC03Nyw3ICs3Nyw3IEBAIHVzaW5nIG5hbWVzcGFjZSBXZWJDb3JlOwogI2VuZGlmCiAK ICAgICBDR0Zsb2F0IGNvbXBvbmVudHNbNF0gPSB7IDAuMiwgMC4yLCAwLjIsIDAuOCB9OwotICAg IFtzdXBlciBzZXRCYWNrZ3JvdW5kQ29sb3I6Q0dDb2xvckNyZWF0ZShjYWNoZWRDR0NvbG9yU3Bh Y2UoQ29sb3JTcGFjZVNSR0IpLCBjb21wb25lbnRzKV07CisgICAgW3N1cGVyIHNldEJhY2tncm91 bmRDb2xvcjphZG9wdENGKENHQ29sb3JDcmVhdGUoY2FjaGVkQ0dDb2xvclNwYWNlKENvbG9yU3Bh Y2VTUkdCKSwgY29tcG9uZW50cykpLmdldCgpXTsKIAogICAgIHJldHVybiBzZWxmOwogfQo=