Only connect to BufferInterface::aboutToBeDestroyed once
It is allowed to call surface_attach multiple times with the same buffer. We want to only connect for cleanup once.
Found whilst debugging a freeze, going through the list of BufferInterface connections I had multiple connections to the same SurfaceInterface.
CCBUG: 428361