Expectation:
\nvar scrollable = Ti.UI.createScrollableView();
\nscrollable.views = [set of views]; // initial
\nscrollable.views = [new set of views]; // further in flow;\nrelease previous set of views
\nSteps to recreate:
\n1) Run on device and load Instruments (all processes)
\n2) Tap 'Generate Section' button to confirm memory is being\nreleased on recreation.
\n3) Tap 'Update Views' button to confirm memory is not being\nreleased. Application will send memory notifications and eventually\ncrash on approx. 15-20 attempts.
\nvar win = Ti.UI.createWindow({ backgroundColor:'#fff', top:0, right:0, bottom:0, left:0 });\n\nvar contentContainer = Ti.UI.createView({ backgroundColor:'#000' });\nvar sectionContainer = Ti.UI.createView({ backgroundColor:'#f00' });\n\nvar scrollable = Ti.UI.createScrollableView({ top:0, right:0, left:0, bottom:50, backgroundColor:'#ff0' });\nvar button1 = Ti.UI.createButton({ bottom:0, left:0, width:'40%', height:50, title:'GENERATE SECTION' });\nvar button2 = Ti.UI.createButton({ bottom:0, right:0, width:'40%', height:50, title:'UPDATE VIEWS' });\n\nvar sectionState = false;\n\nvar currentImages = [];\n\nfunction createButton() {\n var button = Ti.UI.createButton({ left:0, right:0, height:30, bottom:0, title:'Button' });\n \n button.addEventListener('click', function() {\n Ti.API.info('Tapped button.');\n });\n \n return button;\n}\n\n// ### Border Radius Performance - toImage improves this - larger memory footprint\nfunction generateItem(img) {\n var itemContainer = Ti.UI.createView({ width:200, height:200, top:5, left:5, bottom:5, right:5, backgroundColor:'#0f0' });\n var image = Ti.UI.createImageView({ width:200, height:100, image:img, borderRadius:5 }); // slow performance\n var comp = Ti.UI.createImageView({ width:200, height:100 });\n var title = Ti.UI.createLabel({ touchEnabled:'false', text:'Title', textAlign:'center', top:0, height:'auto' });\n var button = createButton();\n \n itemContainer.add(image);\n itemContainer.add(title);\n itemContainer.add(button);\n \n itemContainer.addEventListener('singletap', function() {\n Ti.API.info('Tapped view.');\n });\n\n image.addEventListener('load', function() {\n comp.image = image.toImage();\n itemContainer.remove(image);\n itemContainer.add(comp);\n });\n \n return itemContainer;\n}\n\nfunction createCustomView(images) {\n var view = Ti.UI.createView({ top:0, left:0, right:0, bottom:0, layout:'horizontal', backgroundColor:'#f00' });\n \n var item;\n \n for (var i = 0; i < 12; i++) {\n item = generateItem(images[i]);\n view.add(item);\n }\n \n item.addEventListener('singletap', function() {\n Ti.API.info('Tapped view.');\n });\n \n return view;\n}\n\n// SUCCESS ON RELEASE FROM MEMORY\nfunction generateSection() { \n if (sectionState) {\n contentContainer.remove(sectionContainer);\n }\n \n setTimeout(function() {\n var images = [];\n \n for (var i = 0; i < 12; i++) {\n images.push('image.png');\n }\n \n currentImages = images; \n \n sectionContainer = Ti.UI.createView({ backgroundColor:'#f00' });\n contentContainer.add(sectionContainer);\n\n scrollable = Ti.UI.createScrollableView({ top:0, right:0, left:0, bottom:50, backgroundColor:'#ff0' });\n\n scrollable.views = [\n createCustomView(images),\n createCustomView(images),\n createCustomView(images),\n createCustomView(images)\n ];\n\n sectionContainer.add(scrollable);\n\n sectionState = true;\n }, 500);\n}\n\n// MEMORY ISSUES\nfunction refreshViews() { \n // works on simulator, crash on device (log below) - perhaps this should be done in a reverse order (remove last, first)\n // for (var i = 0, sl = scrollable.views.length; i < sl; i++) {\n // scrollable.removeView(scrollable.views[i]);\n // }\n // [ERROR] The application has crashed with an unhandled exception. Stack trace:\n // \n // 0 CoreFoundation 0x3173463d __exceptionPreprocess + 96\n // 1 libobjc.A.dylib 0x3642dc5d objc_exception_throw + 24\n // 2 CoreFoundation 0x31734491 +[NSException raise:format:arguments:] + 68\n // 3 CoreFoundation 0x317344cb +[NSException raise:format:] + 34\n // 4 QuartzCore 0x309ed61d _ZL18CALayerSetPositionP7CALayerRKN2CA4Vec2IdEEb + 140\n // 5 QuartzCore 0x309ed58b -[CALayer setPosition:] + 38\n // 6 UIKit 0x32e1136b -[UIView(Geometry) setCenter:] + 22\n // 7 scrollabletest2 0x00043115 -[TiViewProxy relayout] + 632\n // 8 scrollabletest2 0x00042673 -[TiViewProxy refreshView:] + 362\n // 9 scrollabletest2 0x00043445 -[TiViewProxy layoutChildrenIfNeeded] + 136\n // 10 scrollabletest2 0x0009d459 performLayoutRefresh + 344\n // 11 CoreFoundation 0x3170ba47 __CFRUNLOOP_IS_CALLING_OUT_TO_A_TIMER_CALLBACK_FUNCTION__ + 14\n // 12 CoreFoundation 0x3170decb __CFRunLoopDoTimer + 850\n // 13 CoreFoundation 0x3170e845 __CFRunLoopRun + 1088\n // 14 CoreFoundation 0x3169eec3 CFRunLoopRunSpecific + 230\n // 15 CoreFoundation 0x3169edcb CFRunLoopRunInMode + 58\n // 16 GraphicsServices 0x3288741f GSEventRunModal + 114\n // 17 GraphicsServices 0x328874cb GSEventRun + 62\n // 18 UIKit 0x32e07d69 -[UIApplication _run] + 404\n // 19 UIKit 0x32e05807 UIApplicationMain + 670\n // 20 scrollabletest2 0x00003bb3 main + 70\n // 21 scrollabletest2 0x000036a8 start + 40\n // \n // \n // 2011-04-13 09:19:00.904 scrollabletest2[8820:707] *** Terminating app due to uncaught exception 'CALayerInvalidGeometry', reason: 'CALayer position contains NaN: [nan 4.0726e-10]'\n // *** Call stack at first throw:\n // (\n // 0 CoreFoundation 0x3173464f __exceptionPreprocess + 114\n // 1 libobjc.A.dylib 0x3642dc5d objc_exception_throw + 24\n // 2 CoreFoundation 0x31734491 +[NSException raise:format:arguments:] + 68\n // 3 CoreFoundation 0x317344cb +[NSException raise:format:] + 34\n // 4 QuartzCore 0x309ed61d _ZL18CALayerSetPositionP7CALayerRKN2CA4Vec2IdEEb + 140\n // 5 QuartzCore 0x309ed58b -[CALayer setPosition:] + 38\n // 6 UIKit 0x32e1136b -[UIView(Geometry) setCenter:] + 22\n // 7 scrollabletest2 0x00043115 -[TiViewProxy relayout] + 632\n // 8 scrollabletest2 0x00042673 -[TiViewProxy refreshView:] + 362\n // 9 scrollabletest2 0x00043445 -[TiViewProxy layoutChildrenIfNeeded] + 136\n // 10 scrollabletest2 0x0009d459 performLayoutRefresh + 344\n // 11 CoreFoundation 0x3170ba47 __CFRUNLOOP_IS_CALLING_OUT_TO_A_TIMER_CALLBACK_FUNCTION__ + 14\n // 12 CoreFoundation 0x3170decb __CFRunLoopDoTimer + 850\n // 13 CoreFoundation 0x3170e845 __CFRunLoopRun + 1088\n // 14 CoreFoundation 0x3169eec3 CFRunLoopRunSpecific + 230\n // 15 CoreFoundation 0x3169edcb CFRunLoopRunInMode + 58\n // 16 GraphicsServices 0x3288741f GSEventRunModal + 114\n // 17 GraphicsServices 0x328874cb GSEventRun + 62\n // 18 UIKit 0x32e07d69 -[UIApplication _run] + 404\n // 19 UIKit 0x32e05807 UIApplicationMain + 670\n // 20 scrollabletest2 0x00003bb3 main + 70\n // 21 scrollabletest2 0x000036a8 start + 40\n // )\n // terminate called after throwing an instance of 'NSException'\n \n // this seems to work, but memory jumps on regenerating section\n // for (var i = 0, sl = scrollable.views.length; i < sl; i++) {\n // scrollable.views[i] = createCustomView(currentImages);\n // }\n \n // previous views do not seem to be released from memory\n scrollable.views = [\n createCustomView(currentImages),\n createCustomView(currentImages),\n createCustomView(currentImages),\n createCustomView(currentImages)\n ];\n}\n\nbutton1.addEventListener('click', function(e) {\n generateSection();\n});\n\nbutton2.addEventListener('click', function(e) {\n refreshViews();\n});\n\nwin.add(contentContainer);\nwin.add(button1);\nwin.add(button2);\n\nwin.open();\n\ngenerateSection();\nImage used for example.
ios_proxy_registration
\ncommit: 6c9e23fb5ae2b9d838e841217193964ef4bd274a
Right now setViews is rather broken in that it doesn't release\nthe old proxies, it's true. Added to that, addView and removeView\n(Despite handling proxy retention properly) don't update the views\nproperty. Unfortunately, the cause lies somewhat deeper, involving\nthe proxies owned by the view instead of the scrollableviewproxy,\nwhich makes fixing this sticky.
\nIn the mean time, there's two solutions:
\n1) Treat setViews as a write-once action, and don't recycle a\nscrollableView to use one collection, then run over and use a\ndifferent collection; use multiple scrollableViews.
2) If the former is not an option, the following can be done as\na workaround until scrollableView is fixed, given scrollableView\nfoo:
\n\nvar oldViews = foo.views;\nfor(var i=0; i<oldViews.length; i++){\n foo.removeView(oldViews[i]);\n}\nfoo.views = [...newviews...];\n