Remove CHECKs used for bug investigation.

Removed CHECKs used for bug investigation.

CHECKs was added when investigating issue 103957 and issue 95732. Since these two bugs have been fixed, remove CHECKs or replace with normal error handling.

BUG=95732, 103957
TEST=none


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

[xhwang, 2012-02-01 02:18:45]

Hi ddorwin,

Removed some CHECKs as issue 103957 and issue 95732 are fixed. Could you do a
review?

xhwang

[ddorwin, 2012-02-01 02:25:23]

LGTM. You'll need OWNERs approval and it would be good to have a GPU person look
at the new condition in the last file.

[xhwang, 2012-02-02 00:42:41]

Hello, jam, apatrick and piman,

Could you please do an ONWERS review.

jam: content/browser/renderer_host/render_message_filter.cc
apatrick: content/common/gpu/gpu_channel_manager.cc
piman: the rest

Thanks,
Xiaohan

[jam, 2012-02-02 21:48:11]

lgtm for my part

[piman, 2012-02-03 18:52:22]

LGTM. Please consider leaving DCHECKs to help catching regressions if the root
cause of those bugs isn't known.

http://codereview.chromium.org/9307009/diff/1/content/renderer/pepper_plugin_...
File content/renderer/pepper_plugin_delegate_impl.cc (left):

http://codereview.chromium.org/9307009/diff/1/content/renderer/pepper_plugin_...
content/renderer/pepper_plugin_delegate_impl.cc:535: CHECK_NE(-1,
channel_handle.socket.fd);
Should we leave this as a DCHECK to catch regressions? The bug suggests the root
cause isn't entirely known even though it's fixed.

[apatrick_chromium, 2012-02-06 18:24:29]

Sorry missed this. LGTM

[commit-bot: I haz the power, 2012-02-06 21:02:28]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/xhwang@chromium.org/9307009/7001

[commit-bot: I haz the power, 2012-02-06 23:06:27]

Try job failure for 9307009-7001 (retry) on mac_rel for step "browser_tests".
It's a second try, previously, step "browser_tests" failed.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=mac_rel&nu...

[commit-bot: I haz the power, 2012-02-07 00:38:58]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/xhwang@chromium.org/9307009/7001

[commit-bot: I haz the power, 2012-02-07 11:05:59]

Change committed as 120763