Supervised users: Include extension version in update requests

chrome/browser/supervised_user/child_accounts/permission_request_creator_apiary.cc

Index: chrome/browser/supervised_user/child_accounts/permission_request_creator_apiary.cc
diff --git a/chrome/browser/supervised_user/child_accounts/permission_request_creator_apiary.cc b/chrome/browser/supervised_user/child_accounts/permission_request_creator_apiary.cc
index 668fe4a1d642da52bba3ec697bd56eb7c11a1e92..d06e5728499adef82bbe1e5247d64765c4a43cc3 100644
--- a/chrome/browser/supervised_user/child_accounts/permission_request_creator_apiary.cc
+++ b/chrome/browser/supervised_user/child_accounts/permission_request_creator_apiary.cc
@@ -11,6 +11,7 @@
 #include "base/logging.h"
 #include "base/strings/stringprintf.h"
 #include "base/values.h"
+#include "base/version.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/signin/profile_oauth2_token_service_factory.h"
 #include "chrome/browser/signin/signin_manager_factory.h"
@@ -118,8 +119,10 @@ void PermissionRequestCreatorApiary::CreateURLAccessRequest(
 
 void PermissionRequestCreatorApiary::CreateExtensionUpdateRequest(
     const std::string& extension_id,
+    const base::Version& version,
     const SuccessCallback& callback) {
-  CreateRequest(kNamespaceUpdateRequest, extension_id, callback);
+  std::string object_ref = extension_id + ":" + version.GetString();

[Bernhard Bauer, 2015/03/23 16:18:39]

BTW, is there a reason why this is part of the PermissionRequestCreator
interface and therefore has to be repeated with every implementation, instead of
moving it to SupervisedUserService?

[Marc Treib, 2015/03/23 16:26:46]

I'd say it's only by coincidence that both PRC implementations handle this the
same way.

[Bernhard Bauer, 2015/03/23 16:51:48]

Really? IMO it's fair to say that what we are abstracting away behind the PRC
interface is *how* we talk to the permission request server, not the server
itself, so all PRC implementations can use a shared format to encode their
requests. 

[Marc Treib, 2015/03/24 14:48:03]

Done.

+  CreateRequest(kNamespaceUpdateRequest, object_ref, callback);
 }
 
 GURL PermissionRequestCreatorApiary::GetApiUrl() const {