Minor cleanup of the videocapture code. No "real" code change :)
Remove declaration of a function that doesn't exist. Remove call to GetProductId from video_capture_factory_windows that's not necessary. This was the only dependency on device_info_windows.h outside of video_capture_windows.cc. TEST=There's no code change, so no manual testing is required. BUG=none Review URL: https://webrtc-codereview.appspot.com/936025 git-svn-id: http://webrtc.googlecode.com/svn/trunk@3114 4adac7df-926f-26a2-2b94-8c16560cd09d
This commit is contained in:
parent
d14ff54de2
commit
12773eaa8c
@ -712,7 +712,7 @@ void DeviceInfoWindows::GetProductId(const char* devicePath,
|
|||||||
"Failed to get the product Id");
|
"Failed to get the product Id");
|
||||||
return;
|
return;
|
||||||
}
|
}
|
||||||
// Find the second occurence
|
// Find the second occurrence.
|
||||||
pos = strchr(pos + 1, '&');
|
pos = strchr(pos + 1, '&');
|
||||||
WebRtc_UWord32 bytesToCopy = (WebRtc_UWord32)(pos - startPos);
|
WebRtc_UWord32 bytesToCopy = (WebRtc_UWord32)(pos - startPos);
|
||||||
if (pos && (bytesToCopy <= productUniqueIdUTF8Length) && bytesToCopy
|
if (pos && (bytesToCopy <= productUniqueIdUTF8Length) && bytesToCopy
|
||||||
|
@ -10,7 +10,6 @@
|
|||||||
|
|
||||||
#include "ref_count.h"
|
#include "ref_count.h"
|
||||||
#include "video_capture_windows.h"
|
#include "video_capture_windows.h"
|
||||||
#include "trace.h"
|
|
||||||
|
|
||||||
namespace webrtc
|
namespace webrtc
|
||||||
{
|
{
|
||||||
@ -20,17 +19,9 @@ VideoCaptureModule* VideoCaptureImpl::Create(
|
|||||||
const WebRtc_Word32 id,
|
const WebRtc_Word32 id,
|
||||||
const char* deviceUniqueIdUTF8)
|
const char* deviceUniqueIdUTF8)
|
||||||
{
|
{
|
||||||
|
|
||||||
if (deviceUniqueIdUTF8 == NULL)
|
if (deviceUniqueIdUTF8 == NULL)
|
||||||
{
|
|
||||||
return NULL;
|
return NULL;
|
||||||
}
|
|
||||||
|
|
||||||
char productId[kVideoCaptureProductIdLength];
|
|
||||||
videocapturemodule::DeviceInfoWindows::GetProductId(deviceUniqueIdUTF8,
|
|
||||||
productId,
|
|
||||||
sizeof(productId));
|
|
||||||
|
|
||||||
RefCountImpl<videocapturemodule::VideoCaptureDS>* newCaptureModule =
|
RefCountImpl<videocapturemodule::VideoCaptureDS>* newCaptureModule =
|
||||||
new RefCountImpl<videocapturemodule::VideoCaptureDS>(id);
|
new RefCountImpl<videocapturemodule::VideoCaptureDS>(id);
|
||||||
|
|
||||||
|
@ -29,10 +29,6 @@ class CaptureSinkFilter;
|
|||||||
class VideoCaptureDS: public VideoCaptureImpl
|
class VideoCaptureDS: public VideoCaptureImpl
|
||||||
{
|
{
|
||||||
public:
|
public:
|
||||||
|
|
||||||
static VideoCaptureModule* Create(const WebRtc_Word32 id,
|
|
||||||
const WebRtc_UWord8* deviceUniqueIdUTF8);
|
|
||||||
|
|
||||||
VideoCaptureDS(const WebRtc_Word32 id);
|
VideoCaptureDS(const WebRtc_Word32 id);
|
||||||
|
|
||||||
virtual WebRtc_Word32 Init(const WebRtc_Word32 id,
|
virtual WebRtc_Word32 Init(const WebRtc_Word32 id,
|
||||||
|
Loading…
x
Reference in New Issue
Block a user