We now hide the parent QFrame while updating the label to work around
the problem where the it wouldn't shrink the widget.

Done with Roberto Raggi

Now that we're showing this thing instantly, we can no longer rely on
the thing hiding cause of a focus change.

(cherry picked from commit 5a0a084c)

Conflicts:

	src/plugins/cpptools/cppcodecompletion.cpp

Now it shows immediately when there is only a single signature of a
given method/constructor.

(cherry picked from commit 8b9dd766)

Conflicts:

	src/plugins/cpptools/cppcodecompletion.cpp

Deals with cases like "QString s(", but for the moment doesn't handle
yet "QString const s(".

Done with Roberto Raggi.

(cherry picked from commit fca3ec1f)

When completing, don't insert "();" characters when they're already
there. Skip them instead.

(cherry picked from commit 86427ecf)

Conflicts:

	src/plugins/cpptools/cppcodecompletion.cpp

Crashed for example in the case of template instantiation. To make sure
the Function* stays valid, we need to not only keep around the Snapshot
but the whole LookupContext.

Done with Roberto Raggi.

Now that we're showing this thing instantly, we can no longer rely on
the thing hiding cause of a focus change.

Now it shows immediately when there is only a single signature of a
given method/constructor.

This reverts commits:

c721304a
885d908e
a0909989
fb4ad59d
0a9a67cf
0d1624d4
d018cfd5
0504fdd0
a2fd10fe

Conflicts:

	src/plugins/cpptools/cppcodecompletion.cpp

Reviewed-by: Roberto Raggi

When completing, don't insert "();" characters when they're already
there. Skip them instead.

Deals with cases like "QString s(", but for the moment doesn't handle
yet "QString const s(".

Done with Roberto Raggi.

Fixes: Do not activate member completion when class names are used as base of a DOT or ARROW expression.

For example,

struct {
  int a, b
} foo;

int main() {
  foo.  <-
}

Reviewed-by: Roberto Raggi

Spaces after the opening brace would cause SIGNAL/SLOT completion to be
disabled along with function completion. Now function completion is
checked at a later stage.

It didn't make sure its Function* stayed valid. Now it does, by keeping
a copy of the list of CppDocuments (the snapshot) around.

Reviewed-by: Roberto Raggi