Skip to content

Commit

Permalink
Remove an useless ifndef NDEBUG in video_capture_types.cc
Browse files Browse the repository at this point in the history
Let's let the compiler do its job :). As mentionned in https://codereview.chromium.org/532533002/ this preprocessor precondition is useless as the linker will removed the unreferenced code anyway, and this break the PGO build.

TBR=magjed, tommi
NOTRY=true

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

Cr-Commit-Position: refs/heads/master@{#293667}
  • Loading branch information
sebmarchand authored and Commit bot committed Sep 7, 2014
1 parent cb9416e commit a008610
Showing 1 changed file with 0 additions and 2 deletions.
2 changes: 0 additions & 2 deletions media/video/capture/video_capture_types.cc
Original file line number Diff line number Diff line change
Expand Up @@ -31,14 +31,12 @@ bool VideoCaptureFormat::IsValid() const {
(pixel_format < PIXEL_FORMAT_MAX);
}

#if !defined(NDEBUG)
std::string VideoCaptureFormat::ToString() const {
return base::StringPrintf("resolution: %s, fps: %f, pixel format: %s",
frame_size.ToString().c_str(),
frame_rate,
PixelFormatToString(pixel_format).c_str());
}
#endif

std::string VideoCaptureFormat::PixelFormatToString(VideoPixelFormat format) {
switch (format) {
Expand Down

0 comments on commit a008610

Please sign in to comment.