2015-06-27 14 views
7

सुनिश्चित नहीं है कि मैंने WebKit में कोई बग मारा है या मैं कुछ गलत कर रहा हूं, लेकिन मुझे में रिसाव के लिए में जो भी मूल्य शामिल है, इसका उपयोग नहीं किया जा सकता है।WKScriptMessageHandler का उपयोग करते समय मेमोरी लीक

मैं इस मुद्दे को अलग करने के लिए एक न्यूनतम मैक प्रोजेक्ट को एक साथ रखने में सक्षम था, लेकिन इसका कोई फायदा नहीं हुआ।

मुख्य दृश्य नियंत्रक में:

class ViewController: NSViewController { 
    var webView: WKWebView? 

    override func viewDidLoad() { 
    super.viewDidLoad() 
    let userContentController = WKUserContentController() 
    userContentController.addScriptMessageHandler(self, name: "handler") 
    let configuration = WKWebViewConfiguration() 
    configuration.userContentController = userContentController 
    webView = WKWebView(frame: CGRectZero, configuration: configuration) 
    view.addSubview(webView!) 

    let path = NSBundle.mainBundle().pathForResource("index", ofType: "html") 
    let url = NSURL(fileURLWithPath: path!)! 
    webView?.loadRequest(NSURLRequest(URL: url)) 
    } 
} 

extension ViewController: WKScriptMessageHandler { 
    func userContentController(userContentController: WKUserContentController, didReceiveScriptMessage message: WKScriptMessage) { 
    print(message.body) 
    } 
} 

और फिर index.html फ़ाइल में:

<html> 
    <head></head> 
    <body> 
    <script type="text/javascript"> 
     webkit.messageHandlers.handler.postMessage("Here's a random number for you: " + Math.random() * 10) 
    </script> 
    </body> 
</html> 

जब मैं परियोजना तो उपकरण में स्मृति डिबगर खोलने चलाने के लिए, मैं रिसाव निम्न देखें:

leak

यदि मैं एक बटन जोड़ता हूं जो अनुरोध को दोबारा लोड करता है, और बहुत कम दर्जन बार करता है, तो ऐप की मेमोरी पदचिह्न बढ़ती रहती है, और एक निश्चित दहलीज के बाद दुर्घटनाग्रस्त हो जाती है। इस न्यूनतम उदाहरण में दुर्घटनाग्रस्त होने में कुछ समय लग सकता है, लेकिन मेरे ऐप में जहां मुझे प्रति सेकंड कई संदेश मिलते हैं, तो इसे क्रैश करने में 10 से कम समय लगता है।

पूरी परियोजना downloaded here हो सकती है।

क्या हो रहा है इसके बारे में कोई विचार?

उत्तर

5

जो आप देख रहे हैं वह एक वेबकिट बग है: https://bugs.webkit.org/show_bug.cgi?id=136140। यह fixed in WebKit trunk a while ago था, लेकिन ऐसा लगता है कि किसी भी वेबकिट अपडेट में विलय नहीं हुआ है।

आप -dealloc से WKScriptMessage जोड़कर इस पर काम कर सकते हैं जो अधिक से अधिक बनाए रखने के लिए क्षतिपूर्ति करता है। यह कुछ इस तरह दिखाई दे सकता है:

// 
// WKScriptMessage+WKScriptMessageLeakFix.m 
// TestWebkitMessages 
// 
// Created by Mark Rowe on 6/27/15. 
// Copyright © Mark Rowe. 
// 
// Permission is hereby granted, free of charge, to any person obtaining a copy of this software and 
// associated documentation files (the "Software"), to deal in the Software without restriction, 
// including without limitation the rights to use, copy, modify, merge, publish, distribute, sublicense, 
// and/or sell copies of the Software, and to permit persons to whom the Software is furnished to do so, 
// subject to the following conditions: 
// 
// The above copyright notice and this permission notice shall be included in all copies or substantial 
// portions of the Software. 
// 
// THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR IMPLIED, INCLUDING BUT NOT 
// LIMITED TO THE WARRANTIES OF MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. 
// IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, 
// WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE 
// SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE. 

#import <mach-o/dyld.h> 
#import <objc/runtime.h> 
#import <WebKit/WebKit.h> 

// Work around <https://webkit.org/b/136140> WKScriptMessage leaks its body 

@interface WKScriptMessage (WKScriptMessageLeakFix) 
@end 

@implementation WKScriptMessage (WKScriptMessageLeakFix) 

+ (void)load 
{ 
    // <https://webkit.org/b/136140> was fixed in WebKit trunk prior to the first v601 build being released. 
    // Enable the workaround in WebKit versions < 601. In the unlikely event that the fix is backported, this 
    // version check will need to be updated. 
    int32_t version = NSVersionOfRunTimeLibrary("WebKit"); 
    int32_t majorVersion = version >> 16; 
    if (majorVersion > 600) 
     return; 

    // Add our -dealloc to WKScriptMessage. If -[WKScriptMessage dealloc] already existed 
    // we'd need to swap implementations instead. 
    Method fixedDealloc = class_getInstanceMethod(self, @selector(fixedDealloc)); 
    IMP fixedDeallocIMP = method_getImplementation(fixedDealloc); 
    class_addMethod(self, @selector(dealloc), fixedDeallocIMP, method_getTypeEncoding(fixedDealloc)); 
} 

- (void)fixedDealloc 
{ 
    // Compensate for the over-retain in -[WKScriptMessage _initWithBody:webView:frameInfo:name:]. 
    [self.body release]; 

    // Call our WKScriptMessage's superclass -dealloc implementation. 
    [super dealloc]; 
} 

@end 

अपनी परियोजना में एक ऑब्जेक्टिव-सी फाइल में ड्रॉप यह, इस फाइल -fno-objc-arc को रोकने के लिए के लिए संकलक निशानी नहीं लगाई है, और यह आप के लिए रिसाव का ध्यान रखना चाहिए।

+0

शानदार! बहुत बहुत धन्यवाद! –

9

मुझे आईओएस 9 एसडीके पर एक ही समस्या का सामना करना पड़ा है।

मैंने देखा userContentController.addScriptMessageHandler(self, name: "handler") हैंडलर का संदर्भ रखेगा। लीक को रोकने के लिए, जब आपको इसकी आवश्यकता नहीं होती है तो बस संदेश हैंडलर को हटा दें। जैसे जब आप कथित नियंत्रक को खारिज करते हैं, तो क्लीन अप विधि को कॉल करें जो removeScriptMessageHandlerForName() पर कॉल करेगा।

आप addScriptMessageHandler() को viewWillAppear पर ले जाने पर विचार कर सकते हैं और viewWillDisappear में संबंधित removeScriptMessageHandlerForName() कॉल जोड़ सकते हैं।

+1

यह स्वीकार्य उत्तर होना चाहिए –

+1

मूल प्रश्न संदेश के निकाय के रूप में पारित मूल्य के बारे में है।यह उत्तर इस तथ्य पर प्रकाश डाला गया है कि 'WKUserContentController' हटाए जाने तक पंजीकृत हैंडलर को बरकरार रखता है, जो जानना महत्वपूर्ण है, लेकिन संदेश निकाय को लीक करने से संबंधित नहीं है। यह एक अच्छा जवाब है, लेकिन यह एक अलग सवाल का जवाब है। – bdash

4

आपके पास यहां एक सतत चक्र है। अपने कोड में, व्यू कंट्रोलर WKWebView को बरकरार रखता है, WKWebView WKWebView कॉन्फ़िगरेशन को बरकरार रखता है, WKWebView कॉन्फ़िगरेशन WKUserContentController को बनाए रखता है और आपका WKUserContentController आपके व्यू कंट्रोलर को बरकरार रखता है। उपर्युक्त टिप्पणी की तरह, आपको अपने व्यू कंट्रोलर को बंद करने से पहले, removeScriptMessageHandlerForName को कॉल करके स्क्रिप्ट हैंडलर को हटाना होगा।

संबंधित मुद्दे