Commit aa34af42 authored by Yuki Shiino's avatar Yuki Shiino Committed by Commit Bot

v8bindings: Refactor ExceptionState::AddExceptionContext

Bug: 839389
Change-Id: I868fc7bc0294a4cbc6d6202293dc5a891a37f2b2
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2159169Reviewed-by: default avatarHitoshi Yoshida <peria@chromium.org>
Commit-Queue: Yuki Shiino <yukishiino@chromium.org>
Cr-Commit-Position: refs/heads/master@{#761014}
parent 8591b65b
...@@ -184,38 +184,65 @@ String ExceptionState::AddExceptionContext(const String& message) const { ...@@ -184,38 +184,65 @@ String ExceptionState::AddExceptionContext(const String& message) const {
if (message.IsEmpty()) if (message.IsEmpty())
return message; return message;
String processed_message = message; const char* i = InterfaceName();
if (PropertyName() && InterfaceName() && context_ != kUnknownContext) { const char* p = PropertyName();
if (context_ == kDeletionContext) const auto& m = message;
processed_message = ExceptionMessages::FailedToDelete(
PropertyName(), InterfaceName(), message); if (i && p) {
else if (context_ == kExecutionContext) switch (context_) {
processed_message = ExceptionMessages::FailedToExecute( case kConstructionContext:
PropertyName(), InterfaceName(), message); return ExceptionMessages::FailedToConstruct(i, m);
else if (context_ == kGetterContext) case kExecutionContext:
processed_message = ExceptionMessages::FailedToGet( return ExceptionMessages::FailedToExecute(p, i, m);
PropertyName(), InterfaceName(), message); case kDeletionContext:
else if (context_ == kSetterContext) return ExceptionMessages::FailedToDelete(p, i, m);
processed_message = ExceptionMessages::FailedToSet( case kGetterContext:
PropertyName(), InterfaceName(), message); return ExceptionMessages::FailedToGet(p, i, m);
} else if (!PropertyName() && InterfaceName()) { case kSetterContext:
if (context_ == kConstructionContext) return ExceptionMessages::FailedToSet(p, i, m);
processed_message = case kEnumerationContext:
ExceptionMessages::FailedToConstruct(InterfaceName(), message); return ExceptionMessages::FailedToEnumerate(i, m);
else if (context_ == kEnumerationContext) case kQueryContext:
processed_message = break;
ExceptionMessages::FailedToEnumerate(InterfaceName(), message); case kIndexedGetterContext:
else if (context_ == kIndexedDeletionContext) return ExceptionMessages::FailedToGetIndexed(i, m);
processed_message = case kIndexedSetterContext:
ExceptionMessages::FailedToDeleteIndexed(InterfaceName(), message); return ExceptionMessages::FailedToSetIndexed(i, m);
else if (context_ == kIndexedGetterContext) case kIndexedDeletionContext:
processed_message = return ExceptionMessages::FailedToDeleteIndexed(i, m);
ExceptionMessages::FailedToGetIndexed(InterfaceName(), message); case kUnknownContext:
else if (context_ == kIndexedSetterContext) break;
processed_message = }
ExceptionMessages::FailedToSetIndexed(InterfaceName(), message);
} }
return processed_message;
if (i) {
switch (context_) {
case kConstructionContext:
return ExceptionMessages::FailedToConstruct(i, m);
case kExecutionContext:
break;
case kDeletionContext:
break;
case kGetterContext:
break;
case kSetterContext:
break;
case kEnumerationContext:
return ExceptionMessages::FailedToEnumerate(i, m);
case kQueryContext:
break;
case kIndexedGetterContext:
return ExceptionMessages::FailedToGetIndexed(i, m);
case kIndexedSetterContext:
return ExceptionMessages::FailedToSetIndexed(i, m);
case kIndexedDeletionContext:
return ExceptionMessages::FailedToDeleteIndexed(i, m);
case kUnknownContext:
break;
}
}
return message;
} }
NonThrowableExceptionState::NonThrowableExceptionState() NonThrowableExceptionState::NonThrowableExceptionState()
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment