Make V4L2VEA GetSupportedProfiles static.

content/common/gpu/media/v4l2_video_encode_accelerator.cc

 // Copyright 2014 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 <fcntl.h>
 #include <linux/videodev2.h>
 #include <poll.h>
 #include <sys/eventfd.h>
 #include <sys/ioctl.h>
 #include <sys/mman.h>
@@ skipping 264 matching lines @@
   }
 
   // Set to kError state just in case.
   SetEncoderState(kError);
 
   delete this;
 }
 
 std::vector<media::VideoEncodeAccelerator::SupportedProfile>
 V4L2VideoEncodeAccelerator::GetSupportedProfiles() {
+  return GetSupportedProfilesStatic();

[Pawel Osciak, 2014/10/13 20:17:48]

Are their any users of this? I think we should have NOTIMPLEMENTED() here
instead?

[wuchengli, 2014/10/14 02:07:43]

There are no uesrs. But this is "not used", not NOTIMPLEMENTED.

[Pawel Osciak, 2014/10/14 07:04:19]

Right, I was just looking for something that would be visible in case someone
used it later by accident or we missed something, iirc NOTREACHED is only a
debug check while NOTIMPLEMENTED is a LOG(ERROR).
In any case, I don't have issues with this, so ok.

+}
+
+std::vector<media::VideoEncodeAccelerator::SupportedProfile>
+V4L2VideoEncodeAccelerator::GetSupportedProfilesStatic() {
   std::vector<SupportedProfile> profiles;
   SupportedProfile profile;
 
   const base::CommandLine* cmd_line = base::CommandLine::ForCurrentProcess();
   if (cmd_line->HasSwitch(switches::kEnableWebRtcHWVp8Encoding)) {
     profile.profile = media::VP8PROFILE_ANY;
     profile.max_resolution.SetSize(1920, 1088);
     profile.max_framerate_numerator = 30;
     profile.max_framerate_denominator = 1;
     profiles.push_back(profile);
@@ skipping 794 matching lines @@
   reqbufs.count = 0;
   reqbufs.type = V4L2_BUF_TYPE_VIDEO_CAPTURE_MPLANE;
   reqbufs.memory = V4L2_MEMORY_MMAP;
   IOCTL_OR_LOG_ERROR(VIDIOC_REQBUFS, &reqbufs);
 
   output_buffer_map_.clear();
   free_output_buffers_.clear();
 }
 
 }  // namespace content