Coverity underground warfare: check return code

- Check return code consistently
- NOTREACHED() added in one case
- CID 4169

BUG = none
TEST = none

Review URL: http://codereview.chromium.org/118042

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@17360 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 6 insertions, 2 deletions

diff --git a/chrome/browser/debugger/inspectable_tab_proxy.cc b/chrome/browser/debugger/inspectable_tab_proxy.cc
index 064ee8f..7d584dc 100644
--- a/chrome/browser/debugger/inspectable_tab_proxy.cc
+++ b/chrome/browser/debugger/inspectable_tab_proxy.cc
@@ -52,12 +52,16 @@ void DevToolsClientHostImpl::OnRpcMessage(const std::string& class_name,
   ListValue* list_msg = static_cast<ListValue*>(message.get());
   if (class_name == kDebuggerAgentDelegate && message_name == kDebuggerOutput) {
     std::string str;
-    list_msg->GetString(0, &str);
+    if (!list_msg->GetString(0, &str))
+      return;
     DebuggerOutput(str);
   } else if (class_name == kToolsAgentDelegate &&
              message_name == kFrameNavigate) {
     std::string url;
-    list_msg->GetString(0, &url);
+    if (!list_msg->GetString(0, &url)) {
+      NOTREACHED();
+      return;
+    }
     FrameNavigate(url);
   }
 }