gl: Avoid calling gl functions without a context even if the object is GL_NONE

- While calling glDestroyXXXX with GL_NONE is a no-op, calling it without a context will crash some drivers.

rpcs3/Emu/RSX/GL/glutils/program.h

@@ -35,16 +35,16 @@ namespace gl
 
 			~shader()
 			{
-				if (created())
+				remove();
-				{
-					remove();
-				}
 			}
 
 			void remove()
 			{
-				glDeleteShader(m_id);
+				if (m_id)
-				m_id = GL_NONE;
+				{
+					glDeleteShader(m_id);
+					m_id = GL_NONE;
+				}
 			}
 
 			void create(::glsl::program_domain type_, const std::string& src);