I guess this is a reference to the parallel thread about the dual uses of landuse=industrial and other landuse tags:
Your approach works fine, except that factories aren’t the only kind of industrial estate/complex/site. For some industrial sites, we have a verifiable name and well-defined property boundary but no specific top-level tag to explain what it is. Part of the original controversy in this thread was that industrial got hoisted up to a top-level feature tag as a solution to this problem. I think we’ll always need some kind of catch-all for industrial facilities that don’t have an obvious type word in English, if not landuse=industrial or industrial=* then something else.