I don't like this being in the public class.
I know that we (currently) don't offer FacebookAlbumInterface (or any of the other classes) as public C++ API, but we may wish to in the future (in fact, I definitely think we should, eventually).  So, we should keep as much of the implementation detail in the private class as possible, I think.