Throw correct errors for invalid OfflineAudioContext values

third_party/WebKit/Source/modules/webaudio/OfflineAudioContext.cpp

Index: third_party/WebKit/Source/modules/webaudio/OfflineAudioContext.cpp
diff --git a/third_party/WebKit/Source/modules/webaudio/OfflineAudioContext.cpp b/third_party/WebKit/Source/modules/webaudio/OfflineAudioContext.cpp
index 37bb3022d4eb77deec92f6c0f3e487bedb692b24..7e4aec0812b8c8d6a637ae450ca68b878326ed1e 100644
--- a/third_party/WebKit/Source/modules/webaudio/OfflineAudioContext.cpp
+++ b/third_party/WebKit/Source/modules/webaudio/OfflineAudioContext.cpp
@@ -59,29 +59,32 @@ OfflineAudioContext* OfflineAudioContext::create(
   Document* document = toDocument(context);
 
   if (!numberOfFrames) {
-    exceptionState.throwDOMException(SyntaxError,
-                                     "number of frames cannot be zero.");
+    exceptionState.throwDOMException(
+        NotSupportedError,
+        ExceptionMessages::indexExceedsMinimumBound<unsigned>(
+            "number of frames", numberOfFrames, 1));
     return nullptr;
   }
 
-  if (numberOfChannels > BaseAudioContext::maxNumberOfChannels()) {
+  if (numberOfChannels == 0 ||
+      numberOfChannels > BaseAudioContext::maxNumberOfChannels()) {
     exceptionState.throwDOMException(
-        IndexSizeError, ExceptionMessages::indexOutsideRange<unsigned>(
-                            "number of channels", numberOfChannels, 0,
-                            ExceptionMessages::InclusiveBound,
-                            BaseAudioContext::maxNumberOfChannels(),
-                            ExceptionMessages::InclusiveBound));
+        NotSupportedError, ExceptionMessages::indexOutsideRange<unsigned>(
+                               "number of channels", numberOfChannels, 1,
+                               ExceptionMessages::InclusiveBound,
+                               BaseAudioContext::maxNumberOfChannels(),
+                               ExceptionMessages::InclusiveBound));
     return nullptr;
   }
 
   if (!AudioUtilities::isValidAudioBufferSampleRate(sampleRate)) {
     exceptionState.throwDOMException(
-        IndexSizeError, ExceptionMessages::indexOutsideRange(
-                            "sampleRate", sampleRate,
-                            AudioUtilities::minAudioBufferSampleRate(),
-                            ExceptionMessages::InclusiveBound,
-                            AudioUtilities::maxAudioBufferSampleRate(),
-                            ExceptionMessages::InclusiveBound));
+        NotSupportedError, ExceptionMessages::indexOutsideRange(
+                               "sampleRate", sampleRate,
+                               AudioUtilities::minAudioBufferSampleRate(),
+                               ExceptionMessages::InclusiveBound,
+                               AudioUtilities::maxAudioBufferSampleRate(),
+                               ExceptionMessages::InclusiveBound));
     return nullptr;
   }
 
@@ -94,6 +97,7 @@ OfflineAudioContext* OfflineAudioContext::create(
                                String::number(numberOfChannels) + ", " +
                                String::number(numberOfFrames) + ", " +
                                String::number(sampleRate) + ")");
+    return nullptr;
   }
 
 #if DEBUG_AUDIONODE_REFERENCES