Correct confusing error message from OfflineAudioContext.suspend()

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

 /*
  * Copyright (C) 2012, Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1.  Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2.  Redistributions in binary form must reproduce the above copyright
  *    notice, this list of conditions and the following disclaimer in the
@@ skipping 239 matching lines @@
         "cannot schedule a suspend at frame " + String::number(frame) + " (" +
             String::number(when) + " seconds) " + "because it is greater than "
                                                   "or equal to the total "
                                                   "render duration of " +
             String::number(m_totalRenderFrames) + " frames"));
     return promise;
   }
 
   // The specified suspend time is in the past; reject the promise.
   if (frame < currentSampleFrame()) {
+    size_t currentFrameClamped = std::min(currentSampleFrame(), length());
+    double currentTimeClamped =
+        std::min(currentTime(), length() / static_cast<double>(sampleRate()));
     resolver->reject(DOMException::create(
         InvalidStateError,
-        "cannot schedule a suspend at frame " + String::number(frame) + " (" +
-            String::number(when) +
-            " seconds) because it is earlier than the current frame of " +
-            String::number(currentSampleFrame()) + " (" +
-            String::number(currentTime()) + " seconds)"));
+        "suspend(" + String::number(when) + ") failed to suspend at frame " +
+            String::number(frame) + " because it is earlier than the current " +
+            "frame of " + String::number(currentFrameClamped) + " (" +
+            String::number(currentTimeClamped) + " seconds)"));
     return promise;
   }
 
   // Wait until the suspend map is available for the insertion. Here we should
   // use AutoLocker because it locks the graph from the main thread.
   AutoLocker locker(this);
 
   // If there is a duplicate suspension at the same quantized frame,
   // reject the promise.
   if (m_scheduledSuspends.contains(frame)) {
@@ skipping 164 matching lines @@
 
   // Note that the GraphLock is required before this check. Since this needs
   // to run on the audio thread, OfflineGraphAutoLocker must be used.
   if (m_scheduledSuspends.contains(currentSampleFrame()))
     return true;
 
   return false;
 }
 
 }  // namespace blink