|
|
Created:
5 years, 4 months ago by jbroman Modified:
5 years, 4 months ago CC:
blink-reviews Base URL:
svn://svn.chromium.org/blink/trunk Target Ref:
refs/heads/master Project:
blink Visibility:
Public. |
DescriptionRemove unused include of RetainPtr.h from PlatformSpeechSynthesizer.cpp.
This file does not use RetainPtr.
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=201195
Patch Set 1 #
Messages
Total messages: 21 (9 generated)
jbroman@chromium.org changed reviewers: + dmazzoni@chromium.org
trivial cleanup mac_chromium_rel_ng failures seem unrelated
The CQ bit was checked by dmazzoni@chromium.org
lgtm
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1313733002/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1313733002/1
The CQ bit was unchecked by commit-bot@chromium.org
Try jobs failed on following builders: chromium_presubmit on tryserver.chromium.linux (JOB_FAILED, http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)
jbroman@chromium.org changed reviewers: + pdr@chromium.org
On 2015/08/25 at 18:18:58, commit-bot wrote: > Try jobs failed on following builders: > chromium_presubmit on tryserver.chromium.linux (JOB_FAILED, http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...) Hmm, I thought you were an OWNER of platform/speech/ as well, but apparently not. I'll grab a platform/ OWNER. pdr, mind stamping this trivial cleanup?
On 2015/08/25 at 18:20:42, jbroman wrote: > On 2015/08/25 at 18:18:58, commit-bot wrote: > > Try jobs failed on following builders: > > chromium_presubmit on tryserver.chromium.linux (JOB_FAILED, http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...) > > Hmm, I thought you were an OWNER of platform/speech/ as well, but apparently not. I'll grab a platform/ OWNER. > > pdr, mind stamping this trivial cleanup? LGTM
The CQ bit was checked by pdr@chromium.org
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1313733002/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1313733002/1
The CQ bit was unchecked by commit-bot@chromium.org
Try jobs failed on following builders: mac_chromium_rel_ng on tryserver.chromium.mac (JOB_FAILED, http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)
The CQ bit was checked by jbroman@chromium.org
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1313733002/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1313733002/1
The CQ bit was unchecked by commit-bot@chromium.org
Try jobs failed on following builders: mac_chromium_rel_ng on tryserver.chromium.mac (JOB_FAILED, http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)
The CQ bit was checked by jbroman@chromium.org
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1313733002/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1313733002/1
Message was sent while issue was closed.
Committed patchset #1 (id:1) as https://src.chromium.org/viewvc/blink?view=rev&revision=201195 |