[Presentation API] PresentationRequest::getAvailability() should return the same promise

third_party/WebKit/Source/modules/presentation/PresentationError.cpp

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "modules/presentation/PresentationError.h"
 
 #include "core/dom/DOMException.h"
 #include "core/dom/ExceptionCode.h"
 #include "public/platform/modules/presentation/WebPresentationError.h"
 
 namespace blink {
 
 // static
-DOMException* PresentationError::take(ScriptPromiseResolver*,
-                                      const WebPresentationError& error) {
+DOMException* PresentationError::take(const WebPresentationError& error) {

[mlamouri (slow - plz ping), 2016/12/13 13:56:12]

For next time, in the spirit of keeping the CLs atomic, you could have done this
change in a follow-up :)

[zhaobin, 2016/12/14 02:25:09]

Sure :)

   ExceptionCode code = UnknownError;
   switch (error.errorType) {
     case WebPresentationError::ErrorTypeNoAvailableScreens:
     case WebPresentationError::ErrorTypeNoPresentationFound:
       code = NotFoundError;
       break;
     case WebPresentationError::ErrorTypeSessionRequestCancelled:
       code = NotAllowedError;
       break;
     case WebPresentationError::ErrorTypeAvailabilityNotSupported:
       code = NotSupportedError;
       break;
     case WebPresentationError::ErrorTypeUnknown:
       code = UnknownError;
       break;
   }
 
   return DOMException::create(code, error.message);
 }
 
 }  // namespace blink