[Sync] Made some sync session member functions const

chrome/browser/sync/engine/download_updates_command_unittest.cc

 // Copyright (c) 2011 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/sync/engine/download_updates_command.h"
 #include "chrome/browser/sync/protocol/autofill_specifics.pb.h"
 #include "chrome/browser/sync/protocol/bookmark_specifics.pb.h"
 #include "chrome/browser/sync/protocol/preference_specifics.pb.h"
 #include "chrome/browser/sync/protocol/sync.pb.h"
 #include "chrome/browser/sync/syncable/directory_manager.h"
@@ skipping 24 matching lines @@
     (*mutable_routing_info())[syncable::PREFERENCES] = GROUP_UI;
     SyncerCommandTest::SetUp();
   }
 
   DownloadUpdatesCommand command_;
 
  private:
   DISALLOW_COPY_AND_ASSIGN(DownloadUpdatesCommandTest);
 };
 
-TEST_F(DownloadUpdatesCommandTest, SetRequestedTypes) {
-  {
-    SCOPED_TRACE("Several enabled datatypes, spread out across groups.");
-    syncable::ModelTypeBitSet enabled_types;
-    enabled_types[syncable::BOOKMARKS] = true;
-    enabled_types[syncable::AUTOFILL] = true;
-    enabled_types[syncable::PREFERENCES] = true;
-    sync_pb::EntitySpecifics get_updates_filter;
-    command_.SetRequestedTypes(enabled_types, &get_updates_filter);
-    ProtoExtensionValidator<sync_pb::EntitySpecifics> v(get_updates_filter);
-    v.ExpectHasExtension(sync_pb::autofill);
-    v.ExpectHasExtension(sync_pb::preference);
-    v.ExpectHasExtension(sync_pb::bookmark);
-    v.ExpectNoOtherFieldsOrExtensions();
-  }
-
-  {
-    SCOPED_TRACE("Top level folders.");
-    syncable::ModelTypeBitSet enabled_types;
-    enabled_types[syncable::TOP_LEVEL_FOLDER] = true;
-    enabled_types[syncable::BOOKMARKS] = true;
-    sync_pb::EntitySpecifics get_updates_filter;
-    command_.SetRequestedTypes(enabled_types, &get_updates_filter);
-    ProtoExtensionValidator<sync_pb::EntitySpecifics> v(get_updates_filter);
-    v.ExpectHasExtension(sync_pb::bookmark);
-    v.ExpectNoOtherFieldsOrExtensions();
-  }
-
-  {
-    SCOPED_TRACE("Bookmarks only.");
-    syncable::ModelTypeBitSet enabled_types;
-    enabled_types[syncable::BOOKMARKS] = true;
-    sync_pb::EntitySpecifics get_updates_filter;
-    command_.SetRequestedTypes(enabled_types, &get_updates_filter);
-    ProtoExtensionValidator<sync_pb::EntitySpecifics> v(get_updates_filter);
-    v.ExpectHasExtension(sync_pb::bookmark);
-    v.ExpectNoOtherFieldsOrExtensions();
-  }
-
-  {
-    SCOPED_TRACE("Autofill only.");
-    syncable::ModelTypeBitSet enabled_types;
-    enabled_types[syncable::AUTOFILL] = true;
-    sync_pb::EntitySpecifics get_updates_filter;
-    command_.SetRequestedTypes(enabled_types, &get_updates_filter);
-    ProtoExtensionValidator<sync_pb::EntitySpecifics> v(get_updates_filter);
-    v.ExpectHasExtension(sync_pb::autofill);
-    v.ExpectNoOtherFieldsOrExtensions();
-  }
-
-  {
-    SCOPED_TRACE("Preferences only.");
-    syncable::ModelTypeBitSet enabled_types;
-    enabled_types[syncable::PREFERENCES] = true;
-    sync_pb::EntitySpecifics get_updates_filter;
-    command_.SetRequestedTypes(enabled_types, &get_updates_filter);
-    ProtoExtensionValidator<sync_pb::EntitySpecifics> v(get_updates_filter);
-    v.ExpectHasExtension(sync_pb::preference);
-    v.ExpectNoOtherFieldsOrExtensions();
-  }
-}
-
 TEST_F(DownloadUpdatesCommandTest, ExecuteNoPayloads) {
   ConfigureMockServerConnection();
   mock_server()->ExpectGetUpdatesRequestTypes(
       ModelTypeBitSetFromSet(GetRoutingInfoTypes(routing_info())));
   command_.ExecuteImpl(session());
 }
 
 TEST_F(DownloadUpdatesCommandTest, ExecuteWithPayloads) {
   ConfigureMockServerConnection();
   sessions::SyncSourceInfo source;
@@ skipping 11 matching lines @@
   EXPECT_CALL(*(mock_debug_info_getter()), GetAndClearDebugInfo(_))
       .Times(1);
   command_.AppendClientDebugInfoIfNeeded(session(), &debug_info);
 
   // Now try to add it once more and make sure |GetAndClearDebugInfo| is not
   // called.
   command_.AppendClientDebugInfoIfNeeded(session(), &debug_info);
 }
 
 }  // namespace browser_sync