Simplify ParseCWSChannelServiceResponse

Simplify ParseCWSChannelServiceResponse

CID=104393
BUG=
TEST=
TBR=mihaip@chromium.org


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

[Tyler Breisacher (Chromium), 2012-07-25 21:24:58]

Coverity was complaining that we didn't check the return value of GetAsString,
this seems like a good way to fix that and also simplify the code.

[Munjal (Google), 2012-07-25 23:06:09]

LGTM

[commit-bot: I haz the power, 2012-07-25 23:09:01]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/tbreisacher@chromium.org/10828021/1

[commit-bot: I haz the power, 2012-07-25 23:09:09]

Presubmit check for 10828021-1 failed and returned exit status 1.

Running presubmit commit checks ...

** Presubmit Messages **
If this change has an associated bug, add BUG=[bug number].

If this change requires manual test instructions to QA team, add
TEST=[instructions].

** Presubmit ERRORS **
Missing LGTM from an OWNER for files in these directories:
    chrome/browser/extensions

[commit-bot: I haz the power, 2012-07-25 23:11:54]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/tbreisacher@chromium.org/10828021/1

[Mihai Parparita -not on Chrome, 2012-07-25 23:41:36]

LGTM

[commit-bot: I haz the power, 2012-07-25 23:45:11]

Try job failure for 10828021-1 (retry) on linux_clang for step "compile"
(clobber build).
It's a second try, previously, step "compile" failed.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=linux_clan...

[Tyler Breisacher (Chromium), 2012-07-25 23:55:26]

Looks unrelated, trying again.

[commit-bot: I haz the power, 2012-07-25 23:55:41]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/tbreisacher@chromium.org/10828021/1

[commit-bot: I haz the power, 2012-07-26 02:13:29]

Change committed as 148475