Currently factories generated by make_names are protected by "ENABLE(SVG)" which should not be hardcoded but left as a command line option.
Created attachment 20990 [details] First try
Comment on attachment 20990 [details] First try Looks good to me. You can avoid checking guardFactory against "" everywhere by just inverting the if. I think that "" is false in perl. If its not, then you can do the "" check at the top, and assign guardFactory to some value which is false, like null. it's OK as is, or with the further "" check cleanup. I don't need to see the patch again before you land. r=me.
> You can avoid checking guardFactory against "" everywhere by just inverting the > if. I think that "" is false in perl. If its not, then you can do the "" > check at the top, and assign guardFactory to some value which is false, like > null. > I got confused by using 'unless' which is why I needed to check against the empty string. I switched to 'if' and cleaned up the patch as Eric suggested. Committed in r32989.