The header includes the whole world even though it doesn't have any definitions. Almost all of the includes could be turned into forward declarations.
Created attachment 272516 [details] the patch
Comment on attachment 272516 [details] the patch r=me
I will wait for EWS. I made the bet that SlowPathThingyThing.h can be Project not Private. That might make some Mac builds sad.
(In reply to comment #3) > I will wait for EWS. I made the bet that SlowPathThingyThing.h can be > Project not Private. That might make some Mac builds sad. Hmmm, I forgot: new a header file probably means that you'll have to update the VC++ project files as well.
(In reply to comment #4) > (In reply to comment #3) > > I will wait for EWS. I made the bet that SlowPathThingyThing.h can be > > Project not Private. That might make some Mac builds sad. > > Hmmm, I forgot: new a header file probably means that you'll have to update > the VC++ project files as well. Oh, I didn't know that! I'll fix it.
OK, landing.
Lnaded in http://trac.webkit.org/changeset/197365