Terrain/RasterWeatherCache: remove the RWLock

Locking has never worked in this class, because GetMap() returns a
pointer to an object that needs protection.  However only the pointer
attribute is protected by the lock.  Let's not pretend that this
works.