[Sync] Enable Compression from Client to Server by experiment

components/sync/engine_impl/commit.cc

 // Copyright 2012 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 "components/sync/engine_impl/commit.h"
 
 #include <utility>
 
 #include "base/metrics/sparse_histogram.h"
+#include "base/rand_util.h"
 #include "base/trace_event/trace_event.h"
 #include "components/sync/base/data_type_histogram.h"
 #include "components/sync/engine/events/commit_request_event.h"
 #include "components/sync/engine/events/commit_response_event.h"
+#include "components/sync/engine/net/http_bridge.h"
 #include "components/sync/engine_impl/commit_processor.h"
 #include "components/sync/engine_impl/commit_util.h"
 #include "components/sync/engine_impl/cycle/sync_cycle.h"
 #include "components/sync/engine_impl/syncer.h"
 #include "components/sync/engine_impl/syncer_proto_util.h"
 
 namespace syncer {
 
 Commit::Commit(ContributionMap contributions,
                const sync_pb::ClientToServerMessage& message,
@@ skipping 25 matching lines @@
   // Give up if no one had anything to commit.
   if (contributions.empty())
     return NULL;
 
   sync_pb::ClientToServerMessage message;
   message.set_message_contents(sync_pb::ClientToServerMessage::COMMIT);
   message.set_share(account_name);
 
   sync_pb::CommitMessage* commit_message = message.mutable_commit();
   commit_message->set_cache_guid(cache_guid);
+  if (base::FeatureList::IsEnabled(syncer::kSyncClientToServerCompression)) {
+    commit_message->set_padding(base::RandBytesAsString(256));

[Nicolas Zea, 2016/10/18 20:23:08]

prefer having the length defined at the top of file as a const size_t. As part
of that definition, comment why 256 was chosen.

[Gang Wu, 2016/10/19 00:06:13]

Done.

+  }
 
   // Set extensions activity if bookmark commits are present.
   ExtensionsActivity::Records extensions_activity_buffer;
   ContributionMap::const_iterator it = contributions.find(BOOKMARKS);
   if (it != contributions.end() && it->second->GetNumEntries() != 0) {
     commit_util::AddExtensionsActivityToMessage(
         extensions_activity, &extensions_activity_buffer, commit_message);
   }
 
   // Set the client config params.
@@ skipping 106 matching lines @@
 
 void Commit::CleanUp() {
   for (ContributionMap::const_iterator it = contributions_.begin();
        it != contributions_.end(); ++it) {
     it->second->CleanUp();
   }
   cleaned_up_ = true;
 }
 
 }  // namespace syncer