Cleanup: Remove double semicolons.

Found with: git grep -l ';;$' | egrep '\.(cc|h)$' | grep -v third_party

Review URL: https://codereview.chromium.org/1558633002

Cr-Commit-Position: refs/heads/master@{#367361}

base/metrics/statistics_recorder_unittest.cc

@@ -74,7 +74,7 @@ TEST_F(StatisticsRecorderTest, NotInitialized) {
   DeleteHistogram(histogram);
 
   // RegisterOrDeleteDuplicateRanges is a no-op.
-  BucketRanges* ranges = new BucketRanges(3);;
+  BucketRanges* ranges = new BucketRanges(3);
   ranges->ResetChecksum();
   EXPECT_EQ(ranges,
             StatisticsRecorder::RegisterOrDeleteDuplicateRanges(ranges));
@@ -85,9 +85,9 @@ TEST_F(StatisticsRecorderTest, NotInitialized) {
 TEST_F(StatisticsRecorderTest, RegisterBucketRanges) {
   std::vector<const BucketRanges*> registered_ranges;
 
-  BucketRanges* ranges1 = new BucketRanges(3);;
+  BucketRanges* ranges1 = new BucketRanges(3);
   ranges1->ResetChecksum();
-  BucketRanges* ranges2 = new BucketRanges(4);;
+  BucketRanges* ranges2 = new BucketRanges(4);
   ranges2->ResetChecksum();
 
   // Register new ranges.
@@ -111,7 +111,7 @@ TEST_F(StatisticsRecorderTest, RegisterBucketRanges) {
   EXPECT_EQ(0, ranges1->range(2));
 
   // Register ranges with same values.
-  BucketRanges* ranges3 = new BucketRanges(3);;
+  BucketRanges* ranges3 = new BucketRanges(3);
   ranges3->ResetChecksum();
   EXPECT_EQ(ranges1,  // returning ranges1
             StatisticsRecorder::RegisterOrDeleteDuplicateRanges(ranges3));

base/process/launch_win.cc

@@ -300,7 +300,7 @@ Process LaunchProcess(const string16& cmdline,
     DestroyEnvironmentBlock(enviroment_block);
     if (!launched) {
       DPLOG(ERROR) << "Command line:" << std::endl << UTF16ToUTF8(cmdline)
-                   << std::endl;;
+                   << std::endl;
       return Process();
     }
   } else {
@@ -309,7 +309,7 @@ Process LaunchProcess(const string16& cmdline,
                        inherit_handles, flags, NULL, NULL,
                        startup_info, &temp_process_info)) {
       DPLOG(ERROR) << "Command line:" << std::endl << UTF16ToUTF8(cmdline)
-                   << std::endl;;
+                   << std::endl;
       return Process();
     }
   }

cc/layers/ui_resource_layer_impl_unittest.cc

@@ -70,7 +70,7 @@ TEST(UIResourceLayerImplTest, VerifyDrawQuads) {
 
   // Make sure we're appending quads when there are valid values.
   gfx::Size bitmap_size(100, 100);
-  gfx::Size layer_size(100, 100);;
+  gfx::Size layer_size(100, 100);
   size_t expected_quad_size = 1;
   bool opaque = true;
   UIResourceId uid = 1;
@@ -117,7 +117,7 @@ TEST(UIResourceLayerImplTest, VerifySetOpaqueOnSkBitmap) {
   host_impl.InitializeRenderer(output_surface.get());
 
   gfx::Size bitmap_size(100, 100);
-  gfx::Size layer_size(100, 100);;
+  gfx::Size layer_size(100, 100);
   bool opaque = false;
   UIResourceId uid = 1;
   scoped_ptr<UIResourceLayerImpl> layer = GenerateUIResourceLayer(&host_impl,

chrome/browser/extensions/api/passwords_private/passwords_private_apitest.cc

@@ -57,7 +57,7 @@ class TestDelegate : public PasswordsPrivateDelegate {
   TestDelegate() : observers_(new base::ObserverListThreadSafe<Observer>()) {
     // Create mock data.
     for (size_t i = 0; i < kNumMocks; i++) {
-      current_entries_.push_back(CreateEntry(i));;
+      current_entries_.push_back(CreateEntry(i));
       current_exceptions_.push_back(CreateException(i));
     }
   }

chrome/browser/sync_file_system/drive_backend/sync_task_manager.h

@@ -206,7 +206,7 @@ class SyncTaskManager {
   scoped_refptr<base::SequencedWorkerPool> worker_pool_;
   base::SequenceChecker sequence_checker_;
 
-  base::WeakPtrFactory<SyncTaskManager> weak_ptr_factory_;;
+  base::WeakPtrFactory<SyncTaskManager> weak_ptr_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(SyncTaskManager);
 };

cloud_print/service/win/chrome_launcher.cc

@@ -13,7 +13,6 @@
 #include "base/json/json_reader.h"
 #include "base/json/json_writer.h"
 #include "base/process/process.h"
-#include "base/process/process.h"
 #include "base/values.h"
 #include "base/win/registry.h"
 #include "base/win/scoped_handle.h"
@@ -201,8 +200,8 @@ void ChromeLauncher::Run() {
   const base::TimeDelta default_time_out = base::TimeDelta::FromSeconds(1);
   const base::TimeDelta max_time_out = base::TimeDelta::FromHours(1);
 
-  for (base::TimeDelta time_out = default_time_out;;
-       time_out = std::min(time_out * 2, max_time_out)) {
+  base::TimeDelta time_out = default_time_out;
+  while (1) {
     base::FilePath chrome_path =
         chrome_launcher_support::GetAnyChromePath(false /* is_sxs */);
 
@@ -254,6 +253,8 @@ void ChromeLauncher::Run() {
     }
     if (stop_event_.TimedWait(time_out))
       break;
+
+    time_out = std::min(time_out * 2, max_time_out);
   }
 }
 

components/gcm_driver/instance_id/instance_id_driver_unittest.cc

@@ -143,7 +143,7 @@ base::Time InstanceIDDriverTest::GetCreationTime(InstanceID* instance_id) {
 
 InstanceID::Result InstanceIDDriverTest::DeleteID(InstanceID* instance_id) {
   async_operation_completed_ = false;
-  result_ = InstanceID::UNKNOWN_ERROR;;
+  result_ = InstanceID::UNKNOWN_ERROR;
   instance_id->DeleteID(base::Bind(&InstanceIDDriverTest::DeleteIDCompleted,
                         base::Unretained(this)));
   WaitForAsyncOperation();
@@ -157,7 +157,7 @@ std::string InstanceIDDriverTest::GetToken(
     const std::map<std::string, std::string>& options) {
   async_operation_completed_ = false;
   token_.clear();
-  result_ = InstanceID::UNKNOWN_ERROR;;
+  result_ = InstanceID::UNKNOWN_ERROR;
   instance_id->GetToken(
       authorized_entity,
       scope,
@@ -173,7 +173,7 @@ InstanceID::Result InstanceIDDriverTest::DeleteToken(
     const std::string& authorized_entity,
     const std::string& scope) {
   async_operation_completed_ = false;
-  result_ = InstanceID::UNKNOWN_ERROR;;
+  result_ = InstanceID::UNKNOWN_ERROR;
   instance_id->DeleteToken(
       authorized_entity,
       scope,
@@ -209,7 +209,7 @@ void InstanceIDDriverTest::DeleteIDCompleted(InstanceID::Result result) {
 }
 
 void InstanceIDDriverTest::GetTokenCompleted(
-    const std::string& token, InstanceID::Result result){
+    const std::string& token, InstanceID::Result result) {
   DCHECK(!async_operation_completed_);
   async_operation_completed_ = true;
   token_ = token;
@@ -357,4 +357,4 @@ TEST_F(InstanceIDDriverTest, DeleteToken) {
             GetToken(instance_id, kAuthorizedEntity2, kScope1, options));
 }
 
-}  // instance_id
+}  // namespace instance_id

components/sync_driver/model_association_manager.cc

@@ -364,7 +364,7 @@ void ModelAssociationManager::TypeStartCallback(
     base::TimeDelta association_wait_time =
         std::max(base::TimeDelta(), type_start_time - association_start_time_);
     base::TimeDelta association_time =
-        base::TimeTicks::Now() - type_start_time;;
+        base::TimeTicks::Now() - type_start_time;
     syncer::DataTypeAssociationStats stats =
         BuildAssociationStatsFromMergeResults(local_merge_result,
                                               syncer_merge_result,

components/variations/study_filtering_unittest.cc

@@ -120,7 +120,7 @@ TEST(VariationsStudyFilteringTest, CheckStudyFormFactor) {
     }
 
     if (i < arraysize(form_factors)) {
-      const int index = arraysize(form_factors) - i - 1;;
+      const int index = arraysize(form_factors) - i - 1;
       filter.add_form_factor(form_factors[index]);
       form_factor_added[index] = true;
     }

content/browser/accessibility/browser_accessibility_win_unittest.cc

@@ -1082,11 +1082,11 @@ TEST_F(BrowserAccessibilityTest, TestCaretAndSelectionInSimpleFields) {
   LONG selection_end = -2;
 
   // Test get_caretOffset.
-  HRESULT hr = combo_box_accessible->get_caretOffset(&caret_offset);;
+  HRESULT hr = combo_box_accessible->get_caretOffset(&caret_offset);
   EXPECT_EQ(S_OK, hr);
   EXPECT_EQ(1L, caret_offset);
   // The caret should be at the end of the selection.
-  hr = text_field_accessible->get_caretOffset(&caret_offset);;
+  hr = text_field_accessible->get_caretOffset(&caret_offset);
   EXPECT_EQ(S_OK, hr);
   EXPECT_EQ(2L, caret_offset);
 
@@ -1098,27 +1098,27 @@ TEST_F(BrowserAccessibilityTest, TestCaretAndSelectionInSimpleFields) {
       manager->GetFocus(root_accessible)->ToBrowserAccessibilityWin());
 
   // The caret should not have moved.
-  hr = text_field_accessible->get_caretOffset(&caret_offset);;
+  hr = text_field_accessible->get_caretOffset(&caret_offset);
   EXPECT_EQ(S_OK, hr);
   EXPECT_EQ(2L, caret_offset);
 
   // Test get_nSelections.
-  hr = combo_box_accessible->get_nSelections(&n_selections);;
+  hr = combo_box_accessible->get_nSelections(&n_selections);
   EXPECT_EQ(S_OK, hr);
   EXPECT_EQ(0L, n_selections);
-  hr = text_field_accessible->get_nSelections(&n_selections);;
+  hr = text_field_accessible->get_nSelections(&n_selections);
   EXPECT_EQ(S_OK, hr);
   EXPECT_EQ(1L, n_selections);
 
   // Test get_selection.
   hr = combo_box_accessible->get_selection(
-      0L /* selection_index */, &selection_start, &selection_end);;
+      0L /* selection_index */, &selection_start, &selection_end);
   EXPECT_EQ(E_INVALIDARG, hr); // No selections available.
   // Invalid in_args should not modify out_args.
   EXPECT_EQ(-2L, selection_start);
   EXPECT_EQ(-2L, selection_end);
   hr = text_field_accessible->get_selection(
-      0L /* selection_index */, &selection_start, &selection_end);;
+      0L /* selection_index */, &selection_start, &selection_end);
   EXPECT_EQ(S_OK, hr);
   EXPECT_EQ(1L, selection_start);
   EXPECT_EQ(2L, selection_end);
@@ -1228,7 +1228,7 @@ TEST_F(BrowserAccessibilityTest, TestCaretInContentEditables) {
   hr = div_editable_accessible->get_nSelections(&n_selections);
   EXPECT_EQ(S_OK, hr);
   EXPECT_EQ(0L, n_selections);
-  hr = div_editable_accessible->get_caretOffset(&caret_offset);;
+  hr = div_editable_accessible->get_caretOffset(&caret_offset);
   EXPECT_EQ(S_OK, hr);
   EXPECT_EQ(6L, caret_offset);
 
@@ -1239,10 +1239,10 @@ TEST_F(BrowserAccessibilityTest, TestCaretInContentEditables) {
   EXPECT_EQ(S_OK, hr);
   EXPECT_EQ(0L, n_selections);
 
-  hr = link_accessible->get_caretOffset(&caret_offset);;
+  hr = link_accessible->get_caretOffset(&caret_offset);
   EXPECT_EQ(S_OK, hr);
   EXPECT_EQ(1L, caret_offset);
-  hr = link_text_accessible->get_caretOffset(&caret_offset);;
+  hr = link_text_accessible->get_caretOffset(&caret_offset);
   EXPECT_EQ(S_OK, hr);
   EXPECT_EQ(1L, caret_offset);
 
@@ -1330,46 +1330,46 @@ TEST_F(BrowserAccessibilityTest, TestSelectionInContentEditables) {
   ASSERT_NE(nullptr, link_text_accessible);
 
   // get_nSelections should work on all objects.
-  HRESULT hr = div_editable_accessible->get_nSelections(&n_selections);;
+  HRESULT hr = div_editable_accessible->get_nSelections(&n_selections);
   EXPECT_EQ(S_OK, hr);
   EXPECT_EQ(1L, n_selections);
-  hr = text_accessible->get_nSelections(&n_selections);;
+  hr = text_accessible->get_nSelections(&n_selections);
   EXPECT_EQ(S_OK, hr);
   EXPECT_EQ(1L, n_selections);
-  hr = link_accessible->get_nSelections(&n_selections);;
+  hr = link_accessible->get_nSelections(&n_selections);
   EXPECT_EQ(S_OK, hr);
   EXPECT_EQ(1L, n_selections);
-  hr = link_text_accessible->get_nSelections(&n_selections);;
+  hr = link_text_accessible->get_nSelections(&n_selections);
   EXPECT_EQ(S_OK, hr);
   EXPECT_EQ(1L, n_selections);
 
   // get_selection should be unaffected by focus placement.
   hr = div_editable_accessible->get_selection(
-      0L /* selection_index */, &selection_start, &selection_end);;
+      0L /* selection_index */, &selection_start, &selection_end);
   EXPECT_EQ(S_OK, hr);
   EXPECT_EQ(1L, selection_start);
   // selection_end should be after embedded object character.
   EXPECT_EQ(7L, selection_end);
 
   hr = text_accessible->get_selection(
-      0L /* selection_index */, &selection_start, &selection_end);;
+      0L /* selection_index */, &selection_start, &selection_end);
   EXPECT_EQ(S_OK, hr);
   EXPECT_EQ(1L, selection_start);
   // No embedded character on this object, only the first part of the text.
   EXPECT_EQ(6L, selection_end);
   hr = link_accessible->get_selection(
-      0L /* selection_index */, &selection_start, &selection_end);;
+      0L /* selection_index */, &selection_start, &selection_end);
   EXPECT_EQ(S_OK, hr);
   EXPECT_EQ(0L, selection_start);
   EXPECT_EQ(4L, selection_end);
   hr = link_text_accessible->get_selection(
-      0L /* selection_index */, &selection_start, &selection_end);;
+      0L /* selection_index */, &selection_start, &selection_end);
   EXPECT_EQ(S_OK, hr);
   EXPECT_EQ(0L, selection_start);
   EXPECT_EQ(4L, selection_end);
 
   // The caret should be at the focus (the end) of the selection.
-  hr = div_editable_accessible->get_caretOffset(&caret_offset);;
+  hr = div_editable_accessible->get_caretOffset(&caret_offset);
   EXPECT_EQ(S_OK, hr);
   EXPECT_EQ(7L, caret_offset);
 
@@ -1380,21 +1380,21 @@ TEST_F(BrowserAccessibilityTest, TestSelectionInContentEditables) {
       manager->GetFocus(root_accessible)->ToBrowserAccessibilityWin());
 
   // The caret should not have moved.
-  hr = div_editable_accessible->get_caretOffset(&caret_offset);;
+  hr = div_editable_accessible->get_caretOffset(&caret_offset);
   EXPECT_EQ(S_OK, hr);
   EXPECT_EQ(7L, caret_offset);
 
   // The caret offset should reflect the position of the selection's focus in
   // any given object.
-  hr = link_accessible->get_caretOffset(&caret_offset);;
+  hr = link_accessible->get_caretOffset(&caret_offset);
   EXPECT_EQ(S_OK, hr);
   EXPECT_EQ(4L, caret_offset);
-  hr = link_text_accessible->get_caretOffset(&caret_offset);;
+  hr = link_text_accessible->get_caretOffset(&caret_offset);
   EXPECT_EQ(S_OK, hr);
   EXPECT_EQ(4L, caret_offset);
 
   hr = div_editable_accessible->get_selection(
-      0L /* selection_index */, &selection_start, &selection_end);;
+      0L /* selection_index */, &selection_start, &selection_end);
   EXPECT_EQ(S_OK, hr);
   EXPECT_EQ(1L, selection_start);
   EXPECT_EQ(7L, selection_end);

content/browser/geolocation/mock_location_arbitrator.cc

@@ -15,7 +15,7 @@ MockLocationArbitrator::MockLocationArbitrator()
 }
 
 void MockLocationArbitrator::StartProviders(bool use_high_accuracy) {
-  providers_started_ = true;;
+  providers_started_ = true;
 }
 
 void MockLocationArbitrator::StopProviders() {

content/browser/loader/async_resource_handler_browsertest.cc

@@ -49,10 +49,10 @@ scoped_ptr<net::test_server::HttpResponse> HandlePostAndRedirectURLs(
                        base::CompareCase::SENSITIVE)) {
     http_response->set_code(net::HTTP_TEMPORARY_REDIRECT);
     http_response->AddCustomHeader("Location", kPostPath);
-    EXPECT_EQ(request.content.length(), kPayloadSize);;
+    EXPECT_EQ(request.content.length(), kPayloadSize);
     return std::move(http_response);
-  } else if(base::StartsWith(request.relative_url, kPostPath,
-                             base::CompareCase::SENSITIVE)) {
+  } else if (base::StartsWith(request.relative_url, kPostPath,
+                              base::CompareCase::SENSITIVE)) {
     http_response->set_content("hello");
     http_response->set_content_type("text/plain");
     EXPECT_EQ(request.content.length(), kPayloadSize);

content/browser/media/capture/desktop_capture_device_unittest.cc

@@ -112,7 +112,7 @@ class MockDeviceClient : public media::VideoCaptureDevice::Client {
 // kFakePixelValueFirst, and the rest of the bytes set to kFakePixelValue, for
 // UnpackedFrame and InvertedFrame verification.
 webrtc::BasicDesktopFrame* CreateBasicFrame(const webrtc::DesktopSize& size) {
-  webrtc::BasicDesktopFrame* frame = new webrtc::BasicDesktopFrame(size);;
+  webrtc::BasicDesktopFrame* frame = new webrtc::BasicDesktopFrame(size);
   DCHECK_EQ(frame->size().width() * webrtc::DesktopFrame::kBytesPerPixel,
             frame->stride());
   memset(frame->data(),
@@ -168,6 +168,7 @@ class UnpackedDesktopFrame : public webrtc::DesktopFrame {
     delete[] data_;
   }
 
+ private:
   DISALLOW_COPY_AND_ASSIGN(UnpackedDesktopFrame);
 };
 
@@ -202,7 +203,7 @@ class FakeScreenCapturer : public webrtc::ScreenCapturer {
     }
     frame_index_++;
 
-    webrtc::DesktopFrame* frame = CreateBasicFrame(size);;
+    webrtc::DesktopFrame* frame = CreateBasicFrame(size);
 
     if (generate_inverted_frames_) {
       frame = new InvertedDesktopFrame(frame);

content/browser/service_worker/service_worker_url_request_job_unittest.cc

@@ -749,7 +749,7 @@ TEST_F(ServiceWorkerURLRequestJobTest, StreamResponse_Flush) {
   expected_response.reserve((sizeof(kTestData) - 1) * 1024);
   for (int i = 0; i < 1024; ++i) {
     expected_response += kTestData;
-    stream->AddData(kTestData, sizeof(kTestData) - 1);;
+    stream->AddData(kTestData, sizeof(kTestData) - 1);
     stream->Flush();
     base::RunLoop().RunUntilIdle();
     EXPECT_EQ(expected_response, url_request_delegate_.response_data());

content/common/dwrite_font_platform_win.cc

@@ -933,7 +933,7 @@ bool FontCollectionLoader::IsFileCached(UINT32 font_key) {
   }
   CacheMap::iterator iter = cache_map_.find(
       GetFontNameFromKey(font_key).c_str());
-  return iter != cache_map_.end();;
+  return iter != cache_map_.end();
 }
 
 bool FontCollectionLoader::LoadCacheFile() {

content/public/browser/host_zoom_map.h

@@ -164,7 +164,7 @@ class HostZoomMap {
 
   // Get/Set the default zoom level for pages that don't override it.
   virtual double GetDefaultZoomLevel() const = 0;
-  virtual void SetDefaultZoomLevel(double level) = 0;;
+  virtual void SetDefaultZoomLevel(double level) = 0;
 
   typedef base::Callback<void(const ZoomLevelChange&)> ZoomLevelChangedCallback;
   typedef base::CallbackList<void(const ZoomLevelChange&)>::Subscription

content/renderer/render_frame_impl.cc

@@ -1067,7 +1067,7 @@ void RenderFrameImpl::PepperCancelComposition(
     PepperPluginInstanceImpl* instance) {
   if (instance != render_view_->focused_pepper_plugin())
     return;
-  Send(new InputHostMsg_ImeCancelComposition(render_view_->GetRoutingID()));;
+  Send(new InputHostMsg_ImeCancelComposition(render_view_->GetRoutingID()));
 #if defined(OS_MACOSX) || defined(USE_AURA)
   GetRenderWidget()->UpdateCompositionInfo(true);
 #endif

content/shell/browser/layout_test/layout_test_permission_manager.cc

@@ -114,7 +114,7 @@ void LayoutTestPermissionManager::ResetPermission(
       PermissionDescription(permission, requesting_origin, embedding_origin));
   if (it == permissions_.end())
     return;
-  permissions_.erase(it);;
+  permissions_.erase(it);
 }
 
 PermissionStatus LayoutTestPermissionManager::GetPermissionStatus(

extensions/shell/browser/shell_extensions_browser_client.cc

@@ -84,7 +84,7 @@ BrowserContext* ShellExtensionsBrowserClient::GetOriginalContext(
 #if defined(OS_CHROMEOS)
 std::string ShellExtensionsBrowserClient::GetUserIdHashFromContext(
     content::BrowserContext* context) {
-  return chromeos::LoginState::Get()->primary_user_hash();;
+  return chromeos::LoginState::Get()->primary_user_hash();
 }
 #endif
 

gin/isolate_holder.cc

@@ -85,7 +85,7 @@ void IsolateHolder::Initialize(ScriptMode mode,
 
 void IsolateHolder::AddRunMicrotasksObserver() {
   DCHECK(!task_observer_.get());
-  task_observer_.reset(new RunMicrotasksObserver(isolate_));;
+  task_observer_.reset(new RunMicrotasksObserver(isolate_));
   base::MessageLoop::current()->AddTaskObserver(task_observer_.get());
 }
 

gpu/command_buffer/tests/gl_program_unittest.cc

@@ -177,7 +177,7 @@ TEST_F(GLProgramTest, UniformsInCurrentProgram) {
       uniform vec4 u_color;
       void main()
       {
-          gl_FragColor = u_color;;
+          gl_FragColor = u_color;
       }
   );
 

storage/browser/fileapi/quota/quota_reservation_manager.cc

@@ -85,7 +85,7 @@ scoped_refptr<QuotaReservation> QuotaReservationManager::CreateReservation(
     const GURL& origin,
     FileSystemType type) {
   DCHECK(origin.is_valid());
-  return GetReservationBuffer(origin, type)->CreateReservation();;
+  return GetReservationBuffer(origin, type)->CreateReservation();
 }
 
 }  // namespace storage

sync/internal_api/sync_backup_manager_unittest.cc

@@ -39,7 +39,7 @@ class SyncBackupManagerTest : public syncer::SyncManager::Observer,
                void(const sessions::SyncSessionSnapshot&));
   MOCK_METHOD1(OnConnectionStatusChange, void(ConnectionStatus));
   MOCK_METHOD1(OnActionableError, void(const SyncProtocolError&));
-  MOCK_METHOD1(OnMigrationRequested, void(ModelTypeSet));;
+  MOCK_METHOD1(OnMigrationRequested, void(ModelTypeSet));
   MOCK_METHOD1(OnProtocolEvent, void(const ProtocolEvent&));
   MOCK_METHOD4(OnInitializationComplete,
                void(const WeakHandle<JsBackend>&,

sync/internal_api/sync_rollback_manager_unittest.cc

@@ -85,7 +85,7 @@ class SyncRollbackManagerTest : public testing::Test,
                     const WeakHandle<DataTypeDebugInfoListener>&,
                     bool, ModelTypeSet));
   MOCK_METHOD1(OnActionableError, void(const SyncProtocolError&));
-  MOCK_METHOD1(OnMigrationRequested, void(ModelTypeSet));;
+  MOCK_METHOD1(OnMigrationRequested, void(ModelTypeSet));
   MOCK_METHOD1(OnProtocolEvent, void(const ProtocolEvent&));
 
   void OnConfigDone(bool success) {

ui/views/widget/widget_interactive_uitest.cc

@@ -850,7 +850,7 @@ TEST_F(WidgetTestInteractive, MAYBE_SystemModalWindowReleasesCapture) {
 
   ASSERT_FALSE(focus_listener.focus_changes().empty());
   EXPECT_EQ(top_level_widget.GetNativeView(),
-            focus_listener.focus_changes().back());;
+            focus_listener.focus_changes().back());
 
   EXPECT_FALSE(top_level_widget.HasCapture());
   top_level_widget.SetCapture(NULL);