Revert "[dart2js,rti] Add direct methods for as JSObject"

This reverts commit f35c922.

Reason for revert: _isJSObject requires a full Rti.

Bug: #60746
Original change's description:
> [dart2js,rti] Add direct methods for `as JSObject`
>
> The specialized methods are generally smaller.
>
> Bug: #60746
> Change-Id: I1824bcded9312bb1ba1c3ecd11fe904de1eb32cc
> Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/434142
> Reviewed-by: Mayank Patke <[email protected]>
> Commit-Queue: Stephen Adams <[email protected]>
> Reviewed-by: Nicholas Shahan <[email protected]>

Bug: #60746
Change-Id: I26b8e25d50fe46f840ea4b3f6eda18524c41b84d
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/434901
Reviewed-by: Nicholas Shahan <[email protected]>
Bot-Commit: Rubber Stamper <[email protected]>
Commit-Queue: Stephen Adams <[email protected]>

Author: rakudrama

pkg/compiler/lib/src/common/elements.dart

@@ -700,9 +700,6 @@ abstract class CommonElements {
 
   late final ClassEntity jsUInt31Class = _findInterceptorsClass('JSUInt31');
 
-  // Static interop JavaScript object interface class.
-  late final ClassEntity jsObjectClass = _findInterceptorsClass('JSObject');
-
   /// Returns `true` member is the 'findIndexForNativeSubclassType' method
   /// declared in `dart:_interceptors`.
   bool isFindIndexForNativeSubclassType(MemberEntity member) {
@@ -1103,11 +1100,6 @@ abstract class CommonElements {
   FunctionEntity get specializedAsStringNullable =>
       _findRtiFunction('_asStringQ');
 
-  FunctionEntity get specializedAsJSObject => _findRtiFunction('_asJSObject');
-
-  FunctionEntity get specializedAsJSObjectNullable =>
-      _findRtiFunction('_asJSObjectQ');
-
   FunctionEntity get instantiatedGenericFunctionType =>
       _findRtiFunction('instantiatedGenericFunctionType');
 

pkg/compiler/lib/src/js_backend/specialized_checks.dart

@@ -192,15 +192,6 @@ class SpecializedChecks {
       return commonElements.specializedAsInt;
     }
 
-    if (element == commonElements.objectClass) {
-      if (!nullable) return commonElements.specializedAsObject;
-    }
-
-    if (element == commonElements.jsObjectClass) {
-      if (nullable) return commonElements.specializedAsJSObjectNullable;
-      return commonElements.specializedAsJSObject;
-    }
-
     return null;
   }
 }

pkg/compiler/lib/src/js_emitter/startup_emitter/fragment_emitter.dart

@@ -1973,7 +1973,6 @@ class FragmentEmitter {
       commonElements.listClass,
       commonElements.objectClass,
       commonElements.mapClass,
-      commonElements.jsObjectClass,
     ];
     // TODO(floitsch): this should probably be on a per-fragment basis.
 

sdk/lib/_internal/js_shared/lib/rti.dart

@@ -1337,8 +1337,6 @@ Object? _specializedAsCheck(Rti testRti) {
       asFn = RAW_DART_FUNCTION_REF(_asNumQ);
     } else if (_Utils.isIdentical(testRti, TYPE_REF<double?>())) {
       asFn = RAW_DART_FUNCTION_REF(_asDoubleQ);
-    } else if (_Utils.isIdentical(testRti, TYPE_REF<JSObject?>())) {
-      asFn = RAW_DART_FUNCTION_REF(_asJSObjectQ);
     }
   } else {
     if (_Utils.isIdentical(testRti, TYPE_REF<int>())) {
@@ -1351,8 +1349,6 @@ Object? _specializedAsCheck(Rti testRti) {
       asFn = RAW_DART_FUNCTION_REF(_asNum);
     } else if (_Utils.isIdentical(testRti, TYPE_REF<double>())) {
       asFn = RAW_DART_FUNCTION_REF(_asDouble);
-    } else if (_Utils.isIdentical(testRti, TYPE_REF<JSObject>())) {
-      asFn = RAW_DART_FUNCTION_REF(_asJSObject);
     }
   }
   return asFn;
@@ -1705,23 +1701,6 @@ String? _asStringQ(dynamic object) {
   throw _TypeError.forType(object, 'String?');
 }
 
-/// Specialization for 'as JSObject'.
-/// Called from generated code.
-@pragma('dart2js:stack-starts-at-throw')
-JSObject _asJSObject(Object? object) {
-  if (_isJSObject(object)) return _Utils.asJSObject(object);
-  throw _TypeError.forType(object, 'JSObject');
-}
-
-/// Specialization for 'as JSObject?'.
-/// Called from generated code.
-@pragma('dart2js:stack-starts-at-throw')
-JSObject? _asJSObjectQ(Object? object) {
-  if (_isJSObject(object)) return _Utils.asJSObject(object);
-  if (object == null) return _Utils.asNull(object);
-  (throw _TypeError.forType(object, 'JSObject?'));
-}
-
 String _rtiArrayToString(Object? array, List<String>? genericContext) {
   String s = '', sep = '';
   for (int i = 0; i < _Utils.arrayLength(array); i++) {
@@ -3941,7 +3920,6 @@ class _Utils {
   static Rti asRti(Object? s) => JS('Rti', '#', s);
   static Rti? asRtiOrNull(Object? s) => JS('Rti|Null', '#', s);
   static _Type as_Type(Object? o) => JS('_Type', '#', o);
-  static JSObject asJSObject(Object? o) => JS('', '#', o);
 
   static bool isString(Object? o) => JS('bool', 'typeof # == "string"', o);
   static bool isNum(Object? o) => JS('bool', 'typeof # == "number"', o);