pgsql: Allow pg_dump to dump non-extension members of an extension-owne

Allow pg_dump to dump non-extension members of an extension-owned schema.

Previously, if a schema was created by an extension, a normal pg_dump run
(not --binary-upgrade) would summarily skip every object in that schema.
In a case where an extension creates a schema and then users create other
objects within that schema, this does the wrong thing: we want pg_dump
to skip the schema but still create the non-extension-owned objects.

There's no easy way to fix this pre-9.6, because in earlier versions the
"dump" status for a schema is just a bool and there's no way to distinguish
"dump me" from "dump my members".  However, as of 9.6 we do have enough
state to represent that, so this is a simple correction of the logic in
selectDumpableNamespace.

In passing, make some cosmetic fixes in nearby code.

Martín Marqués, reviewed by Michael Paquier

Discussion: <99581032-71de-6466-c325-069861f1947d@2ndquadrant.com>

Branch
------
REL9_6_STABLE

Details
-------
http://git.postgresql.org/pg/commitdiff/31eb14504de311c98db2d1306ac61427bcdad863

Modified Files
--------------
src/bin/pg_dump/pg_dump.c                          |  28 ++-
src/test/modules/test_pg_dump/t/001_base.pl        | 206 ++++++++++++++++++++-
.../modules/test_pg_dump/test_pg_dump--1.0.sql     |  24 +++
3 files changed, 247 insertions(+), 11 deletions(-)