Backed out changeset af0fd51dc41a (bug 1550757) for vrhost related mingw bustages CLOSED TREE
authorBogdan Tara <btara@mozilla.com>
Wed, 15 May 2019 08:28:00 +0300
changeset 473889 76bbedc1ec1ae367906390c01a8ca008d7944cac
parent 473888 af0fd51dc41a2c7e0271a932079ac4652d5f5b03
child 473890 2200929c82ea851c14d57515d50cd9e543b3cfb2
child 474113 946206b6d05bf1c42683f57249043e1eec87ec4f
push id36017
push userrgurzau@mozilla.com
push dateWed, 15 May 2019 09:25:56 +0000
treeherdermozilla-central@76bbedc1ec1a [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
bugs1550757
milestone68.0a1
backs outaf0fd51dc41a2c7e0271a932079ac4652d5f5b03
first release with
nightly linux32
76bbedc1ec1a / 68.0a1 / 20190515092556 / files
nightly linux64
76bbedc1ec1a / 68.0a1 / 20190515092556 / files
nightly mac
76bbedc1ec1a / 68.0a1 / 20190515092556 / files
nightly win32
76bbedc1ec1a / 68.0a1 / 20190515092556 / files
nightly win64
76bbedc1ec1a / 68.0a1 / 20190515092556 / files
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
releases
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Backed out changeset af0fd51dc41a (bug 1550757) for vrhost related mingw bustages CLOSED TREE
gfx/vr/moz.build
gfx/vr/vrhost/moz.build
gfx/vr/vrhost/testhost/moz.build
gfx/vr/vrhost/testhost/testhost.cpp
gfx/vr/vrhost/vrhost.cpp
gfx/vr/vrhost/vrhost.def
gfx/vr/vrhost/vrhostex.h
js/src/vm/Debugger.cpp
js/src/vm/Debugger.h
--- a/gfx/vr/moz.build
+++ b/gfx/vr/moz.build
@@ -64,23 +64,16 @@ SOURCES += [
 
 if CONFIG['OS_TARGET'] == 'Android':
     LOCAL_INCLUDES += ['/widget/android']
 else:
     DIRS += [
         'service',
     ]
 
-# Only target x64 for vrhost
-if CONFIG['OS_ARCH'] == 'WINNT':
-    if CONFIG['HAVE_64BIT_BUILD']:
-        DIRS += [
-            'vrhost'
-        ]
-
 IPDL_SOURCES = [
     'ipc/PVR.ipdl',
     'ipc/PVRGPU.ipdl',
     'ipc/PVRLayer.ipdl',
     'ipc/PVRManager.ipdl',
 ]
 
 CXXFLAGS += CONFIG['MOZ_CAIRO_CFLAGS']
deleted file mode 100644
--- a/gfx/vr/vrhost/moz.build
+++ /dev/null
@@ -1,22 +0,0 @@
-# -*- Mode: python; indent-tabs-mode: nil; tab-width: 40 -*-
-# vim: set filetype=python:
-# This Source Code Form is subject to the terms of the Mozilla Public
-# License, v. 2.0. If a copy of the MPL was not distributed with this
-# file, You can obtain one at http://mozilla.org/MPL/2.0/.
-
-SOURCES += [
-  'vrhost.cpp'
-]
-
-EXPORTS.vrhost = [
-  'vrhostex.h'
-]
-
-DEFFILE = 'vrhost.def'
-
-DIRS += [
-  'testhost'
-]
-
-# Use SharedLibrary to generate the dll
-SharedLibrary('vrhost')
\ No newline at end of file
deleted file mode 100644
--- a/gfx/vr/vrhost/testhost/moz.build
+++ /dev/null
@@ -1,16 +0,0 @@
-# -*- Mode: python; indent-tabs-mode: nil; tab-width: 40 -*-
-# vim: set filetype=python:
-# This Source Code Form is subject to the terms of the Mozilla Public
-# License, v. 2.0. If a copy of the MPL was not distributed with this
-# file, You can obtain one at http://mozilla.org/MPL/2.0/.
-
-SOURCES += [
-  'testhost.cpp'
-]
-
-USE_LIBS += [
-  'vrhost'
-]
-
-# Use Progam to generate the executable
-Program('vrtesthost')
\ No newline at end of file
deleted file mode 100644
--- a/gfx/vr/vrhost/testhost/testhost.cpp
+++ /dev/null
@@ -1,23 +0,0 @@
-/* -*- Mode: C++; tab-width: 8; indent-tabs-mode: nil; c-basic-offset: 2 -*- */
-/* vim: set ts=8 sts=2 et sw=2 tw=80: */
-/* This Source Code Form is subject to the terms of the Mozilla Public
- * License, v. 2.0. If a copy of the MPL was not distributed with this
- * file, You can obtain one at http://mozilla.org/MPL/2.0/. */
-
-#include <windows.h>
-#include "vrhost/vrhostex.h"
-
-int main() {
-  HINSTANCE hVR = ::LoadLibrary("vrhost.dll");
-  if (hVR != nullptr) {
-    PFN_SAMPLE lpfnSample = (PFN_SAMPLE)GetProcAddress(hVR, "SampleExport");
-    if (lpfnSample != nullptr) {
-      lpfnSample();
-    }
-
-    ::FreeLibrary(hVR);
-    hVR = nullptr;
-  }
-
-  return 0;
-}
\ No newline at end of file
deleted file mode 100644
--- a/gfx/vr/vrhost/vrhost.cpp
+++ /dev/null
@@ -1,13 +0,0 @@
-/* -*- Mode: C++; tab-width: 8; indent-tabs-mode: nil; c-basic-offset: 2 -*- */
-/* vim: set ts=8 sts=2 et sw=2 tw=80: */
-/* This Source Code Form is subject to the terms of the Mozilla Public
- * License, v. 2.0. If a copy of the MPL was not distributed with this
- * file, You can obtain one at http://mozilla.org/MPL/2.0/. */
-
-// vrhost.cpp
-// Definition of functions that are exported from this dll
-
-#include "vrhostex.h"
-#include <stdio.h>
-
-void SampleExport() { printf("vrhost.cpp hello world"); }
\ No newline at end of file
deleted file mode 100644
--- a/gfx/vr/vrhost/vrhost.def
+++ /dev/null
@@ -1,7 +0,0 @@
-;+# This Source Code Form is subject to the terms of the Mozilla Public
-;+# License, v. 2.0. If a copy of the MPL was not distributed with this
-;+# file, You can obtain one at http://mozilla.org/MPL/2.0/.
-
-LIBRARY        vrhost.dll
-
-EXPORTS        SampleExport      PRIVATE
deleted file mode 100644
--- a/gfx/vr/vrhost/vrhostex.h
+++ /dev/null
@@ -1,14 +0,0 @@
-/* -*- Mode: C++; tab-width: 8; indent-tabs-mode: nil; c-basic-offset: 2 -*- */
-/* vim: set ts=8 sts=2 et sw=2 tw=80: */
-/* This Source Code Form is subject to the terms of the Mozilla Public
- * License, v. 2.0. If a copy of the MPL was not distributed with this
- * file, You can obtain one at http://mozilla.org/MPL/2.0/. */
-
-// vrhostex.h
-// This file declares the functions and their typedefs for functions exported
-// by vrhost.dll
-
-#pragma once
-
-// void SampleExport();
-typedef void (*PFN_SAMPLE)();
\ No newline at end of file
--- a/js/src/vm/Debugger.cpp
+++ b/js/src/vm/Debugger.cpp
@@ -3619,19 +3619,20 @@ bool Debugger::findSweepGroupEdges(JSRun
     }
 
     for (auto e = dbg->debuggeeZones.all(); !e.empty(); e.popFront()) {
       Zone* debuggeeZone = e.front();
       if (!debuggeeZone->isGCMarking()) {
         continue;
       }
 
+
       if (!debuggerZone->addSweepGroupEdgeTo(debuggeeZone) ||
           !debuggeeZone->addSweepGroupEdgeTo(debuggerZone)) {
-        return false;
+          return false;
       }
     }
   }
 
   return true;
 }
 
 const ClassOps Debugger::classOps_ = {nullptr, /* addProperty */
@@ -8914,17 +8915,18 @@ void ScriptedOnPopHandler::drop() {
   js_free(this);
 }
 
 void ScriptedOnPopHandler::trace(JSTracer* tracer) {
   TraceEdge(tracer, &object_, "OnStepHandlerFunction.object");
 }
 
 bool ScriptedOnPopHandler::onPop(JSContext* cx, HandleDebuggerFrame frame,
-                                 ResumeMode& resumeMode, MutableHandleValue vp,
+                                 ResumeMode& resumeMode,
+                                 MutableHandleValue vp,
                                  HandleSavedFrame exnStack) {
   Debugger* dbg = frame->owner();
 
   // Make it possible to distinguish 'return' from 'await' completions.
   // Bug 1470558 will investigate a more robust solution.
   bool isAfterAwait = false;
   AbstractFramePtr referent = DebuggerFrame::getReferent(frame);
   if (resumeMode == ResumeMode::Return && referent &&
@@ -9385,21 +9387,24 @@ static bool EvaluateInEnv(JSContext* cx,
     if (!script) {
       return false;
     }
   }
 
   return ExecuteKernel(cx, script, *env, NullValue(), frame, rval.address());
 }
 
-static bool DebuggerGenericEval(
-    JSContext* cx, const mozilla::Range<const char16_t> chars,
-    HandleObject bindings, const EvalOptions& options, ResumeMode& resumeMode,
-    MutableHandleValue value, MutableHandleSavedFrame exnStack, Debugger* dbg,
-    HandleObject envArg, FrameIter* iter) {
+static bool DebuggerGenericEval(JSContext* cx,
+                                const mozilla::Range<const char16_t> chars,
+                                HandleObject bindings,
+                                const EvalOptions& options,
+                                ResumeMode& resumeMode,
+                                MutableHandleValue value,
+                                MutableHandleSavedFrame exnStack, Debugger* dbg,
+                                HandleObject envArg, FrameIter* iter) {
   // Either we're specifying the frame, or a global.
   MOZ_ASSERT_IF(iter, !envArg);
   MOZ_ASSERT_IF(!iter, envArg && IsGlobalLexicalEnvironment(envArg));
 
   // Gather keys and values of bindings, if any. This must be done in the
   // debugger compartment, since that is where any exceptions must be thrown.
   RootedIdVector keys(cx);
   RootedValueVector values(cx);
--- a/js/src/vm/Debugger.h
+++ b/js/src/vm/Debugger.h
@@ -1659,21 +1659,24 @@ class DebuggerObject : public NativeObje
   static MOZ_MUST_USE bool deleteProperty(JSContext* cx,
                                           HandleDebuggerObject object,
                                           HandleId id, ObjectOpResult& result);
   static MOZ_MUST_USE bool call(JSContext* cx, HandleDebuggerObject object,
                                 HandleValue thisv, Handle<ValueVector> args,
                                 MutableHandleValue result);
   static MOZ_MUST_USE bool forceLexicalInitializationByName(
       JSContext* cx, HandleDebuggerObject object, HandleId id, bool& result);
-  static MOZ_MUST_USE bool executeInGlobal(
-      JSContext* cx, HandleDebuggerObject object,
-      mozilla::Range<const char16_t> chars, HandleObject bindings,
-      const EvalOptions& options, ResumeMode& resumeMode,
-      MutableHandleValue value, MutableHandleSavedFrame exnStack);
+  static MOZ_MUST_USE bool executeInGlobal(JSContext* cx,
+                                           HandleDebuggerObject object,
+                                           mozilla::Range<const char16_t> chars,
+                                           HandleObject bindings,
+                                           const EvalOptions& options,
+                                           ResumeMode& resumeMode,
+                                           MutableHandleValue value,
+                                           MutableHandleSavedFrame exnStack);
   static MOZ_MUST_USE bool makeDebuggeeValue(JSContext* cx,
                                              HandleDebuggerObject object,
                                              HandleValue value,
                                              MutableHandleValue result);
   static MOZ_MUST_USE bool unsafeDereference(JSContext* cx,
                                              HandleDebuggerObject object,
                                              MutableHandleObject result);
   static MOZ_MUST_USE bool unwrap(JSContext* cx, HandleDebuggerObject object,