Remove NOTREACHED verifying no previous exceptions are set in PepperTryCatch

When setting an exception we check that no other exceptions have been set.
However, sometimes it is preferable not to bail out of the
codepath immediately because the code becomes messier. In particular, if
the context is destroyed, an exception will be set which can happen upon
construction of the try/catch. Keeping this NOTREACHED means that we have to check
for an exception after the construction of every try/catch. I can't see a
problem with not checking the exception immediately - other operations will
fail and the result of the try-catch will eventually be set.

BUG=303491

Review URL: https://codereview.chromium.org/554173002

Cr-Commit-Position: refs/heads/master@{#294046}

content/renderer/pepper/pepper_try_catch.cc

@@ -109,10 +109,9 @@ void PepperTryCatchV8::ThrowException(const char* message) {
 }
 
 void PepperTryCatchV8::SetException(const char* message) {
-  if (HasException()) {
-    NOTREACHED();
+  if (HasException())
     return;
-  }
+
   exception_ = ppapi::StringVar::StringToPPVar(message);
 }
 
@@ -156,10 +155,9 @@ bool PepperTryCatchVar::HasException() {
 }
 
 void PepperTryCatchVar::SetException(const char* message) {
-  if (exception_is_set_) {
-    NOTREACHED();
+  if (exception_is_set_)
     return;
-  }
+
   if (exception_)
     *exception_ = ppapi::StringVar::StringToPPVar(message, strlen(message));
   exception_is_set_ = true;