[Webkit-unassigned] [Bug 41842] Add feature detection support for NRWT

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=41842





--- Comment #8 from Gabor Rapcsanyi <rgabor at inf.u-szeged.hu>  2010-08-06 04:09:32 PST ---
(In reply to comment #7)
> (From update of attachment 61033 [details])
> Maybe I'm missing something, but I still don't think you need the copy on line 241 of webkit.py. There's no case where you would want to delete the directories from skipped_directories and then add them back in, and so there's no point in looking at the keys in directories_for_symbol over and over again if you've deleted them. There's no correctness issue, but it's unnecessary (and slower).
> 
> Otherwise, LGTM (but I'm not a reviewer).

Without of this .copy() the python throws this:
File "WebKitTools/Scripts/webkitpy/layout_tests/port/webkit.py", line 241, in directories_to_skip
    for feature_symbol in directories_for_symbol:
RuntimeError: dictionary changed size during iteration

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.