http://git.collabora.co.uk/?p=user/cassidy/telepathy-glib;a=shortlog;h=refs/heads/protocol-avatar-36049

Note that I wasn't able to test this with a real life CM as none seems to implement this interface atm.

Created attachment 45535 [details] [review]
TpProtocol: add API to get avatars requirements (fdo #36049)

Review of attachment 45535 [details] [review]:

this looks fine.

::: tests/dbus/protocol-objects.c
@@ +345,3 @@
+  g_assert_cmpuint (req->maximum_width, ==, 96);
+  g_assert_cmpuint (req->maximum_height, ==, 96);
+        "image/jpeg"));

Where did this number come from?

(In reply to comment #3)
> Review of attachment 45535 [details] [review]:
> 
> this looks fine.
> 
> ::: tests/dbus/protocol-objects.c
> @@ +345,3 @@
> +  g_assert_cmpuint (req->maximum_width, ==, 96);
> +  g_assert_cmpuint (req->maximum_height, ==, 96);
> +        "image/jpeg"));
> 
> Where did this number come from?

Grr splinter. The number I was referring to was the massive number of maximum bytes.

(In reply to comment #4)
> (In reply to comment #3)
> > Review of attachment 45535 [details] [review] [details]:
> > 
> > this looks fine.
> > 
> > ::: tests/dbus/protocol-objects.c
> > @@ +345,3 @@
> > +  g_assert_cmpuint (req->maximum_width, ==, 96);
> > +  g_assert_cmpuint (req->maximum_height, ==, 96);
> > +        "image/jpeg"));
> > 
> > Where did this number come from?
> 
> Grr splinter. The number I was referring to was the massive number of maximum
> bytes.

That's the number already used in examples/cm/echo-message-parts/protocol.c

Merged to master; will be in 0.15.6