RenderObject debug_name was a char* that sometimes pointed to temporary buffers, change to std::string to ensure proper ownership.

Bug found using ASAN.

std::string can be assigned a const char*, so external code that just
sets the attribute to a null-terminated string should still work
unchanged.