Adding browsertest to verify that updating forced-installed extensions is possible.

chrome/browser/policy/cloud/test_request_interceptor.h

Index: chrome/browser/policy/cloud/test_request_interceptor.h
diff --git a/chrome/browser/policy/cloud/test_request_interceptor.h b/chrome/browser/policy/cloud/test_request_interceptor.h
index 4ec5978cab3cf8e64c9fbceeb2ab0209fb460208..c0bffb8396445f005aa9525acf7cd03a434f6c41 100644
--- a/chrome/browser/policy/cloud/test_request_interceptor.h
+++ b/chrome/browser/policy/cloud/test_request_interceptor.h
@@ -9,6 +9,7 @@
 
 #include "base/basictypes.h"
 #include "base/callback.h"
+#include "base/files/file_path.h"

[Joao da Silva, 2013/05/28 08:27:31]

FilePath can be forward declared:

namespace base {
class FilePath;
}

[Joao da Silva, 2013/05/28 11:59:34]

Done.

[csorba, 2013/05/28 12:00:39]

Done.

 #include "chrome/browser/policy/proto/cloud/device_management_backend.pb.h"
 
 namespace net {
@@ -56,6 +57,9 @@ class TestRequestInterceptor {
       enterprise_management::DeviceRegisterRequest::Type expected_type,
       bool expect_reregister);
 
+  // Returns a JobCallback that will create an URLRequestMockHTTPJob.
+  static JobCallback MockJob(const base::FilePath& file_path);

[Joao da Silva, 2013/05/28 08:27:31]

This JobCallback will serve the contents of |file_path|. So a better name is
"FileJob" or "SendFileJob", and the comment should say that it just sends the
contents of the file back.

The URLRequestMockHTTPJob is an implementation detail, and doesn't have to be
mentioned here.

[csorba, 2013/05/28 12:00:39]

Done. (Choosed FileJob)

+
  private:
   class Delegate;