Chromium Code Reviews
Help | Chromium Project | Sign in
(288)

Issue 540003: Revert 35837 - Failed media tests - linux: grab device name before closing it... (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
4 years, 3 months ago by rsesek
Modified:
2 years, 11 months ago
Reviewers:
Evan Martin
CC:
chromium-reviews_googlegroups.com, scherkus, fbarchard, awong, Alpha
Visibility:
Public.

Description

Revert 35837 - Failed media tests - linux: grab device name before closing it

PcmClose frees the handle regardless of whether there's an error
while shutting down, so grab the PcmName result first in case we
want to use it in an error message.

BUG=20006
Review URL: http://codereview.chromium.org/538005

TBR=evan@chromium.org

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=35848

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+2 lines, -3 lines) Lint Patch
M media/audio/linux/alsa_output.cc View 1 chunk +2 lines, -3 lines 0 comments 0 errors Download
Trybot results:
Commit:

Messages

Total messages: 5
rsesek
4 years, 3 months ago #1
rsesek
Broke media tests on Linux and ChromeOS: http://build.chromium.org/buildbot/waterfall/builders/Modules%20Linux/builds/15024/steps/media_unittests/logs/stdio unknown file: Failure Uninteresting mock function call ...
4 years, 3 months ago #2
Evan Martin
Hate computers. Nobody knows how this mock crap works either so now I get to ...
4 years, 3 months ago #3
Evan Martin
(Thank you for reverting quickly and sorry for the trouble.) On Fri, Jan 8, 2010 ...
4 years, 3 months ago #4
awong
4 years, 3 months ago #5
Evan, I can fix it!!!!

On Fri, Jan 8, 2010 at 3:51 PM, Evan Martin <evan@chromium.org> wrote:

> (Thank you for reverting quickly and sorry for the trouble.)
>
> On Fri, Jan 8, 2010 at 3:50 PM, Evan Martin <evan@chromium.org> wrote:
> > Hate computers.  Nobody knows how this mock crap works either so now I
> > get to dig through a new API.
> >
> > On Fri, Jan 8, 2010 at 3:42 PM,  <rsesek@chromium.org> wrote:
> >> Reviewers: Evan Martin,
> >>
> >> Description:
> >> Revert 35837 - Failed media tests - linux: grab device name before
> closing
> >> it
> >>
> >> PcmClose frees the handle regardless of whether there's an error
> >> while shutting down, so grab the PcmName result first in case we
> >> want to use it in an error message.
> >>
> >> BUG=20006
> >>
> >> Review URL: http://codereview.chromium.org/538005
> >>
> >> TBR=evan@chromium.org
> >>
> >> Please review this at http://codereview.chromium.org/540003
> >>
> >> SVN Base: svn://svn.chromium.org/chrome/trunk/src/
> >>
> >> Affected files:
> >>  M     media/audio/linux/alsa_output.cc
> >>
> >>
> >> Index: media/audio/linux/alsa_output.cc
> >> ===================================================================
> >> --- media/audio/linux/alsa_output.cc    (revision 35846)
> >> +++ media/audio/linux/alsa_output.cc    (working copy)
> >> @@ -756,11 +756,10 @@
> >>  }
> >>
> >>  bool AlsaPcmOutputStream::CloseDevice(snd_pcm_t* handle) {
> >> -  std::string name = wrapper_->PcmName(handle);
> >>   int error = wrapper_->PcmClose(handle);
> >>   if (error < 0) {
> >> -    LOG(ERROR) << "Error closing audio device (" << name << "): "
> >> -               << wrapper_->StrError(error);
> >> +    LOG(ERROR) << "Cannot close audio device (" <<
> >> wrapper_->PcmName(handle)
> >> +               << "): " << wrapper_->StrError(error);
> >>     return false;
> >>   }
> >>
> >>
> >>
> >>
> >
>
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld 1275:d14800f88434