Fixed a glitch in the ComponentCloudPolicyUpdater where a protobuf field was read before being vali…

chrome/browser/policy/component_cloud_policy_updater_unittest.cc

 // Copyright (c) 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/policy/component_cloud_policy_updater.h"
 
 #include <string>
 
 #include "base/basictypes.h"
 #include "base/files/scoped_temp_dir.h"
@@ skipping 410 matching lines @@
   fetcher->set_response_code(500);
   fetcher->delegate()->OnURLFetchComplete(fetcher);
 
   // Now the second job was invalidated, and the third job (for the same
   // extension) is the next one.
   fetcher = GetCurrentFetcher();
   ASSERT_TRUE(fetcher);
   EXPECT_EQ(GURL(kTestDownload3), fetcher->GetOriginalURL());
 }
 
+TEST_F(ComponentCloudPolicyUpdaterTest, NoPolicy) {
+  // Start a fetch with a valid download url.
+  updater_->UpdateExternalPolicy(CreateResponse());
+  net::TestURLFetcher* fetcher = GetCurrentFetcher();
+  ASSERT_TRUE(fetcher);
+
+  // Now update the policy fetch response before the fetch completes. The new
+  // data does not have a download url.
+  builder_.payload().Clear();
+  updater_->UpdateExternalPolicy(CreateResponse());
+
+  // The download has been cancelled.
+  fetcher = GetCurrentFetcher();
+  ASSERT_FALSE(fetcher);
+}
+
 }  // namespace policy