[Sync] Pass ModelTypeSet by value everywhere

[Sync] Pass ModelTypeSet by value everywhere

ModelTypeSet is small enough to not need to be passed around by
const reference.

BUG=145986
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=154595

[akalin, 2012-08-31 20:45:11]

+rlarocque for review

[rlarocque, 2012-08-31 20:51:49]

LGTM.

There's a small chance that this will conflict with
http://codereview.chromium.org/10832286/, which is currently in the CQ.  I'd
recommend waiting until that patch lands before sending this one through the
trybots/CQ.

[akalin, 2012-09-01 00:33:02]

On 2012/08/31 20:51:49, rlarocque wrote:
> LGTM.
> 
> There's a small chance that this will conflict with
> http://codereview.chromium.org/10832286/, which is currently in the CQ.  I'd
> recommend waiting until that patch lands before sending this one through the
> trybots/CQ.

Done, putting through CQ now.

[commit-bot: I haz the power, 2012-09-01 00:48:45]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/akalin@chromium.org/10905045/7001

[commit-bot: I haz the power, 2012-09-01 03:50:31]

Change committed as 154595