[MERGE] Remove too-early checks for revisions adding unnecessary round trips, at the cost of actually reading revision data when pulling (because we currently don't have a hint as about whats local for fetch).

[Robert Collins]

We were querying the smart server multiple times when a single check
will suffice due to some 'shortcut' code paths. Those paths are much
less relevant for packs (partial index loading) and smart servers (no
VFS involved at all).

Removing them drops the round trips for pushing onto existing content
(which the stacked code path exercisses) lower.

-Rob
-------------- next part --------------
A non-text attachment was scrubbed...
Name: push-roundtrips-4312.patch
Type: text/x-patch
Size: 11445 bytes
Desc: not available
Url : https://lists.ubuntu.com/archives/bazaar/attachments/20090429/e13059f3/attachment-0001.bin 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 197 bytes
Desc: This is a digitally signed message part
Url : https://lists.ubuntu.com/archives/bazaar/attachments/20090429/e13059f3/attachment-0001.pgp