Fix unused lambda captures in //device and //services/media_capture.

services/video_capture/test/fake_device_unittest.cc

 // Copyright 2016 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 "base/memory/ref_counted.h"
 #include "base/run_loop.h"
 #include "media/base/video_frame.h"
 #include "media/mojo/common/media_type_converters.h"
 #include "services/video_capture/public/cpp/capture_settings.h"
 #include "services/video_capture/public/interfaces/device_factory.mojom.h"
@@ skipping 14 matching lines @@
   bool is_mappable;
   base::TimeDelta timestamp;
 };
 
 }  // anonymous namespace
 
 namespace video_capture {
 
 TEST_F(FakeDeviceTest, DISABLED_FrameCallbacksArrive) {
   base::RunLoop wait_loop;
-  const int kNumFramesToWaitFor = 3;
+  // These two constants must be static as a workaround
+  // for a MSVC++ bug about lambda captures, see the discussion at
+  // https://social.msdn.microsoft.com/Forums/SqlServer/4abf18bd-4ae4-4c72-ba3e-3b13e7909d5f
+  static const int kNumFramesToWaitFor = 3;
   int num_frames_arrived = 0;
   mojom::ReceiverPtr receiver_proxy;
   MockReceiver receiver(mojo::MakeRequest(&receiver_proxy));
   EXPECT_CALL(receiver, OnIncomingCapturedVideoFramePtr(_))
       .WillRepeatedly(InvokeWithoutArgs(
-          [&wait_loop, &kNumFramesToWaitFor, &num_frames_arrived]() {
+          [&wait_loop, &num_frames_arrived]() {
             num_frames_arrived += 1;
             if (num_frames_arrived >= kNumFramesToWaitFor) {
               wait_loop.Quit();
             }
           }));
 
   fake_device_proxy_->Start(requestable_settings_, std::move(receiver_proxy));
   wait_loop.Run();
 }
 
 // Tests that frames received from a fake capture device match the requested
 // format and have increasing timestamps.
 TEST_F(FakeDeviceTest, DISABLED_ReceiveFramesFromFakeCaptureDevice) {
   base::RunLoop wait_loop;
   mojom::ReceiverPtr receiver_proxy;
-  constexpr int num_frames_to_receive = 2;
+  // These two constants must be static as a workaround
+  // for a MSVC++ bug about lambda captures, see the discussion at
+  // https://social.msdn.microsoft.com/Forums/SqlServer/4abf18bd-4ae4-4c72-ba3e-3b13e7909d5f
+  static const int num_frames_to_receive = 2;
   FrameInfo received_frame_infos[num_frames_to_receive];
   int received_frame_count = 0;
   MockReceiver receiver(mojo::MakeRequest(&receiver_proxy));
   EXPECT_CALL(receiver, OnIncomingCapturedVideoFramePtr(_))
       .WillRepeatedly(Invoke(
-          [&received_frame_infos, &received_frame_count, &num_frames_to_receive,
-           &wait_loop](const media::mojom::VideoFramePtr* frame) {
+          [&received_frame_infos, &received_frame_count, &wait_loop]
+          (const media::mojom::VideoFramePtr* frame) {
             if (received_frame_count >= num_frames_to_receive)
               return;
             auto video_frame = frame->To<scoped_refptr<media::VideoFrame>>();
             auto& frame_info = received_frame_infos[received_frame_count];
             frame_info.pixel_format = video_frame->format();
             frame_info.storage_type = video_frame->storage_type();
             frame_info.is_mappable = video_frame->IsMappable();
             frame_info.size = video_frame->natural_size();
             frame_info.timestamp = video_frame->timestamp();
             received_frame_count += 1;
@@ skipping 16 matching lines @@
     EXPECT_TRUE(frame_info.is_mappable);
     EXPECT_EQ(requestable_settings_.format.frame_size, frame_info.size);
     // Timestamps are expected to increase
     if (i > 0)
       EXPECT_GT(frame_info.timestamp, previous_timestamp);
     previous_timestamp = frame_info.timestamp;
   }
 }
 
 }  // namespace video_capture