Bug 1510717 - Part 1: Delete unused js::Class for abstract base class ReadableStreamController. r=jwalden
authorJason Orendorff <jorendorff@mozilla.com>
Tue, 04 Dec 2018 19:54:33 +0000
changeset 508537 e31ab8b54aa37a2f8f6adf0f242c8c2b62e1fa55
parent 508536 da4789ed7ec046470b3662ec5c26973398f042cd
child 508538 a9aecbc2d273351da812e8a365485a0ea67e6bea
push id1905
push userffxbld-merge
push dateMon, 21 Jan 2019 12:33:13 +0000
treeherdermozilla-release@c2fca1944d8c [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersjwalden
bugs1510717
milestone65.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1510717 - Part 1: Delete unused js::Class for abstract base class ReadableStreamController. r=jwalden Differential Revision: https://phabricator.services.mozilla.com/D13293
js/src/builtin/Stream.cpp
js/src/builtin/Stream.h
--- a/js/src/builtin/Stream.cpp
+++ b/js/src/builtin/Stream.cpp
@@ -2430,18 +2430,16 @@ static const JSPropertySpec ReadableStre
     JS_PSG("desiredSize", ReadableStreamDefaultController_desiredSize, 0),
     JS_PS_END};
 
 static const JSFunctionSpec ReadableStreamDefaultController_methods[] = {
     JS_FN("close", ReadableStreamDefaultController_close, 0, 0),
     JS_FN("enqueue", ReadableStreamDefaultController_enqueue, 1, 0),
     JS_FN("error", ReadableStreamDefaultController_error, 1, 0), JS_FS_END};
 
-const Class ReadableStreamController::class_ = {"ReadableStreamController"};
-
 CLASS_SPEC(ReadableStreamDefaultController, 0, SlotCount,
            ClassSpec::DontDefineConstructor, 0, JS_NULL_CLASS_OPS);
 
 /**
  * Unified implementation of ReadableStream controllers' [[CancelSteps]]
  * internal methods.
  * Streams spec, 3.8.5.1. [[CancelSteps]] ( reason )
  * and
--- a/js/src/builtin/Stream.h
+++ b/js/src/builtin/Stream.h
@@ -317,18 +317,16 @@ class ReadableStreamController : public 
   bool isTeeBranch1() const { return flags() & Flag_TeeBranch1; }
   void setTeeBranch1() { addFlags(Flag_TeeBranch | Flag_TeeBranch1); }
   bool isTeeBranch2() const { return flags() & Flag_TeeBranch2; }
   void setTeeBranch2() { addFlags(Flag_TeeBranch | Flag_TeeBranch2); }
   bool hasExternalSource() const { return flags() & Flag_ExternalSource; }
   bool sourceLocked() const { return flags() & Flag_SourceLocked; }
   void setSourceLocked() { addFlags(Flag_SourceLocked); }
   void clearSourceLocked() { removeFlags(Flag_SourceLocked); }
-
-  static const Class class_;
 };
 
 class ReadableStreamDefaultController : public ReadableStreamController {
  private:
   /**
    * Memory layout for ReadableStreamDefaultControllers, starting after the
    * slots shared among all types of controllers.
    *