Remove GICE (again).

Remove GICE (again).

R=guoweis@webrtc.org

Committed: https://crrev.com/7cbd188c5ed7df80bb737bd4ada94422730e2d89
Cr-Commit-Position: refs/heads/master@{#9979}

[pthatcher1, 2015-09-17 07:09:36]

pthatcher@webrtc.org changed reviewers:
+ deadbeef@webrtc.org, guoweis@webrtc.org

[pthatcher1, 2015-09-17 07:09:37]

https://codereview.webrtc.org/1353713002/diff/1/chromium/.gclient
File chromium/.gclient (right):

https://codereview.webrtc.org/1353713002/diff/1/chromium/.gclient#newcode13
chromium/.gclient:13: 'src/third_party/deqp/src': None,
My client won't sync without this.  But I think there's no risk in adding it,
since we don't use it.

https://codereview.webrtc.org/1353713002/diff/1/tools/valgrind-webrtc/memchec...
File tools/valgrind-webrtc/memcheck/suppressions.txt (right):

https://codereview.webrtc.org/1353713002/diff/1/tools/valgrind-webrtc/memchec...
tools/valgrind-webrtc/memcheck/suppressions.txt:129:
SignalsCloseAfterForcedCloseAll2
I think removing this while reverting the first GICE removal was a mistake.

[guoweis_webrtc, 2015-09-17 16:54:31]

On 2015/09/17 07:09:37, pthatcher1 wrote:
> https://codereview.webrtc.org/1353713002/diff/1/chromium/.gclient
> File chromium/.gclient (right):
> 
> https://codereview.webrtc.org/1353713002/diff/1/chromium/.gclient#newcode13
> chromium/.gclient:13: 'src/third_party/deqp/src': None,
> My client won't sync without this.  But I think there's no risk in adding it,
> since we don't use it.
> 
>
https://codereview.webrtc.org/1353713002/diff/1/tools/valgrind-webrtc/memchec...
> File tools/valgrind-webrtc/memcheck/suppressions.txt (right):
> 
>
https://codereview.webrtc.org/1353713002/diff/1/tools/valgrind-webrtc/memchec...
> tools/valgrind-webrtc/memcheck/suppressions.txt:129:
> SignalsCloseAfterForcedCloseAll2
> I think removing this while reverting the first GICE removal was a mistake.

lgtm. 

I don't understand the change in .gclient but if we're not using that, that is
probably fine.

[pthatcher1, 2015-09-18 01:14:28]

The CQ bit was checked by pthatcher@webrtc.org

[commit-bot: I haz the power, 2015-09-18 01:14:45]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1353713002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1353713002/20001

[commit-bot: I haz the power, 2015-09-18 01:16:49]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-09-18 01:16:50]

Try jobs failed on following builders:
  presubmit on tryserver.webrtc (JOB_FAILED,
http://build.chromium.org/p/tryserver.webrtc/builders/presubmit/builds/819)

[pthatcher1, 2015-09-18 01:55:10]

Committed patchset #3 (id:40001) manually as
7cbd188c5ed7df80bb737bd4ada94422730e2d89 (presubmit successful).

[commit-bot: I haz the power, 2015-09-18 01:55:12]

Patchset 3 (id:??) landed as
https://crrev.com/7cbd188c5ed7df80bb737bd4ada94422730e2d89
Cr-Commit-Position: refs/heads/master@{#9979}