Skip to content
Snippets Groups Projects
  1. Oct 01, 2021
    • Mike Hommey's avatar
      Bug 1733545 - Remove REQUIRES_UNIFIED_BUILD from third-party directories that... · 5245f849
      Mike Hommey authored
      Bug 1733545 - Remove REQUIRES_UNIFIED_BUILD from third-party directories that don't need it. r=firefox-build-system-reviewers,andi
      
      Differential Revision: https://phabricator.services.mozilla.com/D127229
      5245f849
    • Christoph Kerschbaumer's avatar
      Bug 1732259: Update tests within devtools/client/webconsole/test to work with... · e7d8bd52
      Christoph Kerschbaumer authored
      Bug 1732259: Update tests within devtools/client/webconsole/test to work with https-first enabled r=jdescottes
      
      Differential Revision: https://phabricator.services.mozilla.com/D126485
      e7d8bd52
    • Julian Descottes's avatar
      Bug 1733464 - [devtools] Fix webconsole stubs and node tests for https-first r=nchevobbe · 316d273d
      Julian Descottes authored
      It is slightly more complicated than expected because we have an issue capturing 404 from httpd.js in DevTools when using HTTPS (see Bug 1733420).
      
      Therefore the stub for network events, which generates 404 requests, is not using https://example.com but http://mochi.test:8888
      
      Differential Revision: https://phabricator.services.mozilla.com/D127161
      316d273d
    • Jan de Mooij's avatar
      Bug 1733075 part 5 - Use NativeObject::hasEnumerableProperty in a few more places. r=anba · 8c4903ad
      Jan de Mooij authored
      Also change MaybeOneOf to Maybe + if-constexpr now that this works.
      
      Differential Revision: https://phabricator.services.mozilla.com/D127068
      8c4903ad
    • Jan de Mooij's avatar
      Bug 1733075 part 4 - Add tests for some for-in edge cases. r=iain · 12f87955
      Jan de Mooij authored
      At least some of these were not covered by other jit-tests.
      
      Depends on D127056
      
      Differential Revision: https://phabricator.services.mozilla.com/D127057
      12f87955
    • Jan de Mooij's avatar
      Bug 1733075 part 3 - Optimize property enumeration in EnumerateNativeProperties. r=iain · 51fb18ba
      Jan de Mooij authored
      If we only care about enumerable properties and the object doesn't have any, we don't need
      to iterate its properties.
      
      We can't use this optimization in the CheckForDuplicates case, but fortunately the
      optimization in the previous patch made that a lot less common.
      
      Depends on D127055
      
      Differential Revision: https://phabricator.services.mozilla.com/D127056
      51fb18ba
    • Jan de Mooij's avatar
      Bug 1733075 part 2 - Optimize enumeration with no enumerable properties on the... · 66114545
      Jan de Mooij authored
      Bug 1733075 part 2 - Optimize enumeration with no enumerable properties on the proto chain better. r=iain
      
      Add a fast path to better optimize the common case where the proto chain has no
      enumerable properties. The advantages of this are:
      
      * We no longer have to iterate over all the properties on the proto chain.
      * We don't need to check for duplicates (this involves a relatively slow HashSet and includes non-enumerable properties too).
      
      ProtoMayHaveEnumerableProperties returns false (meaning the fast path works) in
      97-100% of cases on typical web workloads.
      
      Differential Revision: https://phabricator.services.mozilla.com/D127055
      66114545
    • Jan de Mooij's avatar
      Bug 1733075 part 1 - Add HasEnumerable ObjectFlag. r=iain · 5c84d76a
      Jan de Mooij authored
      This gives us the ability to check whether an object has only non-enumerable
      properties in constant time. Prototype objects typically don't have enumerable
      properties.
      
      Differential Revision: https://phabricator.services.mozilla.com/D127054
      5c84d76a
    • criss's avatar
      Backed out 4 changesets (bug 1728934) for causing Mochitest failures. CLOSED TREE · 7897dc69
      criss authored
      Backed out changeset 5e64ae0a8b00 (bug 1728934)
      Backed out changeset 3538237090bf (bug 1728934)
      Backed out changeset a240e5a52df4 (bug 1728934)
      Backed out changeset a932fbd95f01 (bug 1728934)
      7897dc69
    • Nicolas Chevobbe's avatar
      Bug 1733039 - [devtools] Fix walkerFront#findNodeFront method for EFT/Fission. r=ochameau. · 48c08d86
      Nicolas Chevobbe authored
      The top-level target walker was used to find elements, where we should use the
      walker front of the nodeFront we have at hand.
      The function was also checking if the node we're searching was associated with
      this walker, which, with EFT, won't be the case as soon as we have to go through
      an iframe.
      
      Differential Revision: https://phabricator.services.mozilla.com/D126909
      48c08d86
    • Nicolas Chevobbe's avatar
      Bug 1732906 - [devtools] Refactor eyedropper show/hide methods. r=ochameau. · d70c24af
      Nicolas Chevobbe authored
      With EFT, tests using the eyedropper were failing because of the isXul check
      done when the eyedropper gets hidden: the document.documentElement property
      was null at this point and the isXul method was throwing.
      This patch replace the isXul check with a check on the presence of an abortController
      property, which we do now set in the `show` method.
      
      Differential Revision: https://phabricator.services.mozilla.com/D126808
      d70c24af
    • Nicolas Chevobbe's avatar
      Bug 1732690 - [devtools] Fix GeometryHighlighter in iframes. r=jdescottes. · 807a90b0
      Nicolas Chevobbe authored
      The highlighter wasn't working on remote iframe as we were trying to retrieve it
      from the top-level inspector front.
      
      Differential Revision: https://phabricator.services.mozilla.com/D126807
      807a90b0
    • Nicolas Chevobbe's avatar
    • Nicolas Chevobbe's avatar
      Bug 1730170 - [devtools] Fix inspector search when EFT is enabled. r=bomsy. · 0aef0dd1
      Nicolas Chevobbe authored
      We were getting results of nodes in iframes twice: from the iframe target
      as well as from the top-level target.
      
      Differential Revision: https://phabricator.services.mozilla.com/D126067
      0aef0dd1
    • Nicolas Chevobbe's avatar
      Bug 1730162 - [devtools] Don't get grids from frames in LayoutActor#getGrids... · b4dad3bd
      Nicolas Chevobbe authored
      Bug 1730162 - [devtools] Don't get grids from frames in LayoutActor#getGrids when EFT is enabled. r=ochameau.
      
      Differential Revision: https://phabricator.services.mozilla.com/D126065
      b4dad3bd
    • Nicolas Chevobbe's avatar
      Bug 1730167 - [devtools] Replace usage of isRemoteFrame with isFrameWithChildTarget. r=ochameau. · bc4eb07f
      Nicolas Chevobbe authored
      `isRemoteFrame` was used as a way to know if an iframe was tracked by its own target.
      Since we can now have dedicated targets for every iframe, we need to check if a given
      iframe is tracked by its dedicated target.
      
      Differential Revision: https://phabricator.services.mozilla.com/D126805
      bc4eb07f
    • Nicolas Chevobbe's avatar
      Bug 1730167 - [devtools] Use appropriate walker front to retrieve frame... · 3c196757
      Nicolas Chevobbe authored
      Bug 1730167 - [devtools] Use appropriate walker front to retrieve frame children when EFT is enabled in a11y panel. r=ochameau.
      
      This is a similar change to what was done to node actors in the previous patch
      of this queue.
      
      Differential Revision: https://phabricator.services.mozilla.com/D126804
      3c196757
    • Nicolas Chevobbe's avatar
      Bug 1730167 - [devtools] Use appropriate walker front to retrieve frame... · 58313c2a
      Nicolas Chevobbe authored
      Bug 1730167 - [devtools] Use appropriate walker front to retrieve frame children when EFT is enabled. r=ochameau
      
      Differential Revision: https://phabricator.services.mozilla.com/D126064
      58313c2a
    • Mark Banner's avatar
      Bug 1733485 - Lazy load modules in GMPUtils.jsm, UpdateUtils.jsm,... · aec46f6b
      Mark Banner authored
      Bug 1733485 - Lazy load modules in GMPUtils.jsm, UpdateUtils.jsm, GMPProvider.jsm to avoid loading earlier than necessary. r=bytesized
      
      Differential Revision: https://phabricator.services.mozilla.com/D127171
      aec46f6b
    • Petr Sumbera's avatar
    • Cristian Tuns's avatar
    • shravanrn@gmail.com's avatar
    • shravanrn@gmail.com's avatar
    • shravanrn@gmail.com's avatar
    • shravanrn@gmail.com's avatar
    • Emilio Cobos Álvarez's avatar
      Bug 1733339 - Remove NS_AUTHOR_SPECIFIED_PADDING. r=mstange · b53a78f4
      Emilio Cobos Álvarez authored
      There's only one meaningful usage of it, which is to disable native
      appearance of the <input type=range> (the windows native theme is no
      longer exposed to content).
      
      <input type=range> is inconsistent with every other native widget, which
      only disables native appearance if the author specifies backgrounds or
      borders. So make it match literally all other widgets and simplify a bit
      the code.
      
      We had no tests for this special behavior, let me know if you think it's
      worth adding one (but I don't feel very strongly about it).
      
      Differential Revision: https://phabricator.services.mozilla.com/D127082
      b53a78f4
    • Nils Ohlmeier's avatar
      e58ba15e
    • Nils Ohlmeier's avatar
    • Makoto Kato's avatar
      Bug 1582606 - Allow data: url on GVE. r=geckoview-reviewers,agi · c3d7964c
      Makoto Kato authored
      If data: url has space, it doesn't be recognized as data: url on GVE.
      
      Differential Revision: https://phabricator.services.mozilla.com/D127124
      c3d7964c
    • Mike Hommey's avatar
      Bug 1733076 - Add missing wasi sysroot toolchain to searchfox arm builds.... · cf3b35f6
      Mike Hommey authored
      Bug 1733076 - Add missing wasi sysroot toolchain to searchfox arm builds. r=firefox-build-system-reviewers,andi
      
      Differential Revision: https://phabricator.services.mozilla.com/D127034
      cf3b35f6
    • Mike Hommey's avatar
      Bug 1733308 - Allow to disable unified builds in xpcom/tests/windows.... · 845953db
      Mike Hommey authored
      Bug 1733308 - Allow to disable unified builds in xpcom/tests/windows. r=firefox-build-system-reviewers,andi
      
      There's nothing wrong there.
      
      Differential Revision: https://phabricator.services.mozilla.com/D127043
      845953db
    • Mike Hommey's avatar
      Bug 1733308 - Fix non-unified builds errors in xpcom/tests/gtest. r=xpcom-reviewers,mccr8 · 2d21eb6e
      Mike Hommey authored
      objdir/dist/include/mozilla/Queue.h:251:31: error: use of undeclared identifier 'moz_xcalloc'
      xpcom/base/nsCOMPtr.h:436:5: error: static_assert failed due to requirement '1 < sizeof (TestForIID<nsIEventTarget>(nullptr))' "nsCOMPtr only works for types with IIDs.  Either use RefPtr; add an IID to your type with NS_DECLARE_STATIC_IID_ACCESSOR/NS_DEFINE_STATIC_IID_ACCESSOR; or make the nsCOMPtr point to a base class with an IID."
      xpcom/tests/gtest/TestDelayedRunnable.cpp:16:28: error: field has incomplete type '(anonymous namespace)::ReleaseDetector'
      xpcom/tests/gtest/TestDelayedRunnable.cpp:16:3: error: 'explicit' can only appear on non-static member functions
      xpcom/tests/gtest/TestDelayedRunnable.cpp:16:40: error: expected ')'
      xpcom/tests/gtest/TestDelayedRunnable.cpp:16:61: error: use of undeclared identifier 'aActive'
      xpcom/tests/gtest/TestDelayedRunnable.cpp:28:3: error: no template named 'Atomic'; did you mean 'mozilla::Atomic'?
      xpcom/tests/gtest/TestDelayedRunnable.cpp:34:3: error: no template named 'Atomic'; did you mean 'mozilla::Atomic'?
      xpcom/tests/gtest/TestDelayedRunnable.cpp:36:18: error: use of undeclared identifier 'TaskQueue'; did you mean 'taskQueue'?
      xpcom/tests/gtest/TestDelayedRunnable.cpp:36:18: error: variable 'taskQueue' declared with deduced type 'auto' cannot appear in its own initializer
      xpcom/tests/gtest/TestDelayedRunnable.cpp:36:48: error: use of undeclared identifier 'MediaThreadType'
      xpcom/tests/gtest/TestDelayedRunnable.cpp:36:48: error: use of undeclared identifier 'MediaThreadType'; did you mean 'mozilla::MediaThreadType'?
      xpcom/tests/gtest/TestDelayedRunnable.cpp:38:51: error: no matching conversion for functional-style cast from 'Atomic<bool> *' to '(anonymous namespace)::ReleaseDetector'
      xpcom/tests/gtest/TestDelayedRunnable.cpp:54:3: error: no template named 'Atomic'; did you mean 'mozilla::Atomic'?
      xpcom/tests/gtest/TestDelayedRunnable.cpp:58:51: error: no matching conversion for functional-style cast from 'Atomic<bool> *' to '(anonymous namespace)::ReleaseDetector'
      xpcom/tests/gtest/TestDelayedRunnable.cpp:88:3: error: use of undeclared identifier 'Unused'; did you mean 'mozilla::Unused'?
      xpcom/tests/gtest/TestDelayedRunnable.cpp:104:3: error: use of undeclared identifier 'Unused'; did you mean 'mozilla::Unused'?
      xpcom/tests/gtest/TestDelayedRunnable.cpp:109:10: error: unknown type name 'SharedThreadPool'; did you mean 'mozilla::SharedThreadPool'?
      xpcom/tests/gtest/TestDelayedRunnable.cpp:109:35: error: use of undeclared identifier 'SharedThreadPool'
      xpcom/tests/gtest/TestDelayedRunnable.cpp:109:35: error: use of undeclared identifier 'SharedThreadPool'; did you mean 'mozilla::SharedThreadPool'?
      xpcom/tests/gtest/TestDelayedRunnable.cpp:110:25: error: use of function template name with no prior declaration in function call with explicit template arguments is a C++20 extension [-Werror,-Wc++20-extensions]
      xpcom/tests/gtest/TestDelayedRunnable.cpp:110:36: error: unknown type name 'TaskQueue'; did you mean 'mozilla::TaskQueue'?
      xpcom/tests/gtest/TestDelayedRunnable.cpp:112:36: error: unknown type name 'TaskQueue'; did you mean 'mozilla::TaskQueue'?
      xpcom/tests/gtest/TestFileNTFSSpecialPaths.cpp:18:28: error: use of undeclared identifier 'do_CreateInstance'
      xpcom/tests/gtest/TestFileNTFSSpecialPaths.cpp:26:28: error: use of undeclared identifier 'do_CreateInstance'
      xpcom/tests/gtest/TestFileNTFSSpecialPaths.cpp:39:28: error: use of undeclared identifier 'do_CreateInstance'
      xpcom/tests/gtest/TestFileNTFSSpecialPaths.cpp:279:28: error: use of undeclared identifier 'do_CreateInstance'
      xpcom/tests/gtest/TestFilePreferencesWin.cpp:113:26: error: use of undeclared identifier 'do_CreateInstance'
      xpcom/tests/gtest/TestFilePreferencesWin.cpp:140:26: error: use of undeclared identifier 'do_CreateInstance'
      xpcom/tests/gtest/TestFilePreferencesWin.cpp:156:31: error: use of undeclared identifier 'NS_OS_TEMP_DIR'
      xpcom/tests/gtest/TestJSHolderMap.cpp:50:28: error: unknown type name 'JSHolderMap'
      xpcom/tests/gtest/TestJSHolderMap.cpp:52:35: error: use of undeclared identifier 'i'
      xpcom/tests/gtest/TestJSHolderMap.cpp:52:45: error: use of undeclared identifier 'i'
      xpcom/tests/gtest/TestJSHolderMap.cpp:52:8: error: use of undeclared identifier 'JSHolderMap'
      xpcom/tests/gtest/TestJSHolderMap.cpp:52:8: error: use of undeclared identifier 'JSHolderMap'; did you mean 'mozilla::JSHolderMap'?
      xpcom/tests/gtest/TestJSHolderMap.cpp:53:24: error: use of undeclared identifier 'i'
      xpcom/tests/gtest/TestJSHolderMap.cpp:54:24: error: use of undeclared identifier 'i'
      xpcom/tests/gtest/TestJSHolderMap.cpp:68:3: error: unknown type name 'JSHolderMap'; did you mean 'mozilla::JSHolderMap'?
      xpcom/tests/gtest/TestJSHolderMap.cpp:73:3: error: unknown type name 'JSHolderMap'; did you mean 'mozilla::JSHolderMap'?
      xpcom/tests/gtest/TestJSHolderMap.cpp:99:3: error: unknown type name 'JSHolderMap'; did you mean 'mozilla::JSHolderMap'?
      xpcom/tests/gtest/TestJSHolderMap.cpp:104:9: error: use of undeclared identifier 'JSHolderMap'
      xpcom/tests/gtest/TestJSHolderMap.cpp:104:9: error: use of undeclared identifier 'JSHolderMap'; did you mean 'mozilla::JSHolderMap'?
      xpcom/tests/gtest/TestJSHolderMap.cpp:125:41: error: use of undeclared identifier 'MakeUnique'
      xpcom/tests/gtest/TestJSHolderMap.cpp:147:3: error: unknown type name 'JSHolderMap'; did you mean 'mozilla::JSHolderMap'?
      xpcom/tests/gtest/TestJSHolderMap.cpp:151:41: error: use of undeclared identifier 'MakeUnique'
      xpcom/tests/gtest/TestRWLock.cpp:92:3: error: no template named 'Maybe'; did you mean 'mozilla::Maybe'?
      xpcom/tests/gtest/TestRWLock.cpp:95:23: error: use of undeclared identifier 'SyncRunnable'; did you mean 'mozilla::SyncRunnable'?
      xpcom/tests/gtest/TestRWLock.cpp:103:5: error: unknown type name 'AutoTryReadLock'; did you mean 'mozilla::AutoTryReadLock'?
      xpcom/tests/gtest/TestRWLock.cpp:107:5: error: unknown type name 'AutoTryReadLock'; did you mean 'mozilla::AutoTryReadLock'?
      xpcom/tests/gtest/TestRWLock.cpp:111:7: error: unknown type name 'AutoTryReadLock'; did you mean 'mozilla::AutoTryReadLock'?
      xpcom/tests/gtest/TestRWLock.cpp:120:7: error: unknown type name 'AutoTryReadLock'; did you mean 'mozilla::AutoTryReadLock'?
      xpcom/tests/gtest/TestRWLock.cpp:130:5: error: unknown type name 'AutoWriteLock'; did you mean 'mozilla::AutoWriteLock'?
      xpcom/tests/gtest/TestRWLock.cpp:133:5: error: unknown type name 'AutoTryReadLock'; did you mean 'mozilla::AutoTryReadLock'?
      xpcom/tests/gtest/TestRWLock.cpp:137:7: error: unknown type name 'AutoTryReadLock'; did you mean 'mozilla::AutoTryReadLock'?
      xpcom/tests/gtest/TestRWLock.cpp:142:3: error: unknown type name 'AutoTryReadLock'; did you mean 'mozilla::AutoTryReadLock'?
      xpcom/tests/gtest/TestRWLock.cpp:146:5: error: unknown type name 'AutoTryReadLock'; did you mean 'mozilla::AutoTryReadLock'?
      xpcom/tests/gtest/TestRWLock.cpp:155:5: error: unknown type name 'AutoTryWriteLock'; did you mean 'mozilla::AutoTryWriteLock'?
      xpcom/tests/gtest/TestRWLock.cpp:159:5: error: unknown type name 'AutoTryReadLock'; did you mean 'mozilla::AutoTryReadLock'?
      xpcom/tests/gtest/TestRWLock.cpp:163:7: error: unknown type name 'AutoTryWriteLock'; did you mean 'mozilla::AutoTryWriteLock'?
      xpcom/tests/gtest/TestRWLock.cpp:172:7: error: unknown type name 'AutoTryWriteLock'; did you mean 'mozilla::AutoTryWriteLock'?
      xpcom/tests/gtest/TestRWLock.cpp:182:5: error: unknown type name 'AutoReadLock'; did you mean 'mozilla::AutoReadLock'?
      xpcom/tests/gtest/TestRWLock.cpp:184:5: error: unknown type name 'AutoTryWriteLock'; did you mean 'mozilla::AutoTryWriteLock'?
      xpcom/tests/gtest/TestRWLock.cpp:188:7: error: unknown type name 'AutoTryWriteLock'; did you mean 'mozilla::AutoTryWriteLock'?
      xpcom/tests/gtest/TestRWLock.cpp:194:5: error: unknown type name 'AutoWriteLock'; did you mean 'mozilla::AutoWriteLock'?
      xpcom/tests/gtest/TestRWLock.cpp:197:5: error: unknown type name 'AutoTryWriteLock'; did you mean 'mozilla::AutoTryWriteLock'?
      xpcom/tests/gtest/TestRWLock.cpp:201:7: error: unknown type name 'AutoTryWriteLock'; did you mean 'mozilla::AutoTryWriteLock'?
      
      Differential Revision: https://phabricator.services.mozilla.com/D127042
      2d21eb6e
    • Mike Hommey's avatar
      Bug 1733308 - Fix non-unified builds errors in xpcom/threads. r=xpcom-reviewers,mccr8 · 25134bc8
      Mike Hommey authored
      xpcom/threads/CPUUsageWatcher.cpp:43:23: error: unused variable 'kCPUCheckInterval' [-Werror,-Wunused-const-variable]
      xpcom/threads/CPUUsageWatcher.cpp:109:23: error: unused variable 'kCPUCheckInterval' [-Werror,-Wunused-const-variable]
      xpcom/threads/InputEventStatistics.cpp:18:5: error: use of undeclared identifier 'ClearOnShutdown'
      xpcom/threads/InputTaskManager.cpp:62:9: error: use of undeclared identifier 'InputEventStatistics'
      xpcom/threads/InputTaskManager.cpp:89:3: error: use of undeclared identifier 'InputEventStatistics'
      xpcom/threads/InputTaskManager.cpp:105:7: error: use of undeclared identifier 'nsRefreshDriver'
      xpcom/threads/InputTaskManager.cpp:124:9: error: use of undeclared identifier 'InputEventStatistics'
      xpcom/threads/MainThreadIdlePeriod.cpp:29:23: error: unused variable 'kMaxTimerThreadBoundClamp' [-Werror,-Wunused-const-variable]
      xpcom/threads/VsyncTaskManager.h:15:67: error: implicit instantiation of undefined template 'mozilla::StaticRefPtr<mozilla::VsyncTaskManager>'
      xpcom/threads/VsyncTaskManager.h:16:52: error: no viable overloaded '='
      xpcom/threads/nsMemoryPressure.cpp:61:12: error: use of undeclared identifier 'nsIObserverService'
      xpcom/threads/nsMemoryPressure.cpp:61:37: error: use of undeclared identifier 'services'
      xpcom/threads/nsThreadManager.cpp:422:28: error: member access into incomplete type 'mozilla::ThreadEventTarget'
      xpcom/threads/nsThreadManager.cpp:661:5: error: use of undeclared identifier 'CrashReporter'
      xpcom/threads/nsThreadManager.cpp:662:59: error: expected ')'
      xpcom/threads/nsThreadManager.cpp:662:9: error: use of undeclared identifier 'CrashReporter'
      xpcom/threads/nsThreadManager.cpp:669:7: error: use of undeclared identifier 'CrashReporter'
      xpcom/threads/nsThreadManager.cpp:670:11: error: use of undeclared identifier 'CrashReporter'
      xpcom/threads/nsThreadManager.cpp:670:61: error: expected ')'
      xpcom/threads/nsThreadManager.cpp:672:7: error: use of undeclared identifier 'CrashReporter'
      xpcom/threads/nsThreadManager.cpp:673:11: error: use of undeclared identifier 'CrashReporter'
      xpcom/threads/nsThreadUtils.cpp:636:13: error: member access into incomplete type 'mozilla::Task'
      
      Differential Revision: https://phabricator.services.mozilla.com/D127041
      25134bc8
    • Mike Hommey's avatar
      Bug 1733308 - Fix non-unified builds errors in xpcom/io. r=xpcom-reviewers,mccr8 · c8c38574
      Mike Hommey authored
      xpcom/io/InputStreamLengthHelper.cpp:14:22: error: unused variable 'kStreamTransportServiceCID' [-Werror,-Wunused-const-variable]
      xpcom/io/SeekableStreamWrapper.cpp:422:17: error: no member named 'InputStreamHelper' in namespace 'mozilla::ipc'
      xpcom/io/nsSegmentedBuffer.cpp:117:11: error: no viable conversion from '(lambda at /tmp/central/xpcom/io/nsSegmentedBuffer.cpp:117:11)' to 'void *'
      xpcom/io/nsSegmentedBuffer.cpp:128:11: error: no viable conversion from '(lambda at /tmp/central/xpcom/io/nsSegmentedBuffer.cpp:128:11)' to 'void *'
      xpcom/io/nsSegmentedBuffer.cpp:131:25: error: out-of-line definition of 'FreeOMT' does not match any declaration in 'nsSegmentedBuffer'
      xpcom/io/nsSegmentedBuffer.cpp:143:16: error: use of function template name with no prior declaration in function call with explicit template arguments is a C++20 extension [-Werror,-Wc++20-extensions]
      xpcom/io/nsSegmentedBuffer.cpp:143:16: error: use of undeclared identifier 'MakeRefPtr'; did you mean 'mozilla::MakeRefPtr'?
      xpcom/io/nsSegmentedBuffer.cpp:164:35: error: implicit instantiation of undefined template 'nsTArray<std::function<void ()>>'
      xpcom/io/nsSegmentedBuffer.cpp:164:44: error: lambda capture 'this' is not used [-Werror,-Wunused-lambda-capture]
      xpcom/io/nsSegmentedBuffer.cpp:165:14: error: use of undeclared identifier 'mTasks'
      xpcom/io/nsSegmentedBuffer.h:90:25: error: member initializer 'mTasks' does not name a non-static data member or base class
      xpcom/io/nsSegmentedBuffer.h:95:25: error: no template named 'function' in namespace 'std'
      xpcom/io/nsSegmentedBuffer.h:96:20: error: use of undeclared identifier 'mTasks'
      xpcom/io/nsSegmentedBuffer.h:104:24: error: use of undeclared identifier 'nsTArray'
      xpcom/io/nsSegmentedBuffer.h:104:38: error: no member named 'function' in namespace 'std'
      xpcom/io/nsSegmentedBuffer.h:104:55: error: expected member name or ';' after declaration specifiers
      xpcom/io/nsSegmentedBuffer.h:104:57: error: private field 'mTasks' is not used [-Werror,-Wunused-private-field]
      xpcom/io/nsSegmentedBuffer.h:108:21: error: no template named 'function' in namespace 'std'
      xpcom/io/nsStorageStream.cpp:576:3: error: no template named 'DebugOnly'
      xpcom/io/nsStorageStream.cpp:576:3: error: no template named 'DebugOnly'; did you mean 'mozilla::DebugOnly'?
      
      Differential Revision: https://phabricator.services.mozilla.com/D127040
      c8c38574
    • Mike Hommey's avatar
      Bug 1733308 - Fix non-unified builds errors in xpcom/ds. r=xpcom-reviewers,mccr8 · 26a98990
      Mike Hommey authored
      xpcom/ds/nsProperties.cpp:56:3: error: use of undeclared identifier 'AppendToArray'; did you mean 'mozilla::AppendToArray'?
      xpcom/ds/PLDHashTable.cpp:324:3: error: use of undeclared identifier 'NS_ASSERTION'
      xpcom/ds/PLDHashTable.cpp:395:3: error: use of undeclared identifier 'NS_ASSERTION'
      xpcom/ds/PLDHashTable.cpp:679:7: error: use of undeclared identifier 'NS_ABORT_OOM'
      xpcom/ds/PLDHashTable.cpp:685:7: error: use of undeclared identifier 'NS_ABORT_OOM'
      
      Differential Revision: https://phabricator.services.mozilla.com/D127039
      26a98990
    • Mike Hommey's avatar
      Bug 1733308 - Allow to disable unified builds in xpcom/components.... · 1eb69b0b
      Mike Hommey authored
      Bug 1733308 - Allow to disable unified builds in xpcom/components. r=firefox-build-system-reviewers,andi
      
      There's nothing wrong there.
      
      Differential Revision: https://phabricator.services.mozilla.com/D127038
      1eb69b0b
    • Mike Hommey's avatar
      Bug 1733308 - Fix non-unified builds errors in xpcom/build. r=xpcom-reviewers,mccr8 · 2432a64b
      Mike Hommey authored
      xpcom/glue/FileUtils.cpp:130:10: error: 'return' will never be executed [-Werror,-Wunreachable-code-return]
      
      Differential Revision: https://phabricator.services.mozilla.com/D127037
      2432a64b
    • Mike Hommey's avatar
      Bug 1733308 - Fix non-unified builds errors in xpcom/base. r=xpcom-reviewers,mccr8 · 0679fea9
      Mike Hommey authored
      objdir/dist/include/mozilla/DeferredFinalize.h:30:1: error: unknown type name 'MOZ_NEVER_INLINE'
      objdir/dist/include/mozilla/dom/BindingUtils.h:2776:5: error: no matching function for call to 'DeferredFinalize'
      xpcom/base/AvailableMemoryTracker.cpp:55,40:: error: expected class name
      xpcom/base/AvailableMemoryTracker.cpp:59,3:: error: 'AddRef' marked 'override' but does not override any member functions
      xpcom/base/AvailableMemoryTracker.cpp:59,3:: error: 'QueryInterface' marked 'override' but does not override any member functions
      xpcom/base/AvailableMemoryTracker.cpp:59,3:: error: 'Release' marked 'override' but does not override any member functions
      xpcom/base/AvailableMemoryTracker.cpp:61,29:: error: unknown type name 'nsIHandleReportCallback'
      xpcom/base/AvailableMemoryTracker.cpp:65,37:: error: use of undeclared identifier 'KIND_OTHER'
      xpcom/base/AvailableMemoryTracker.cpp:65,49:: error: use of undeclared identifier 'UNITS_COUNT_CUMULATIVE'
      xpcom/base/AvailableMemoryTracker.cpp:77,1:: error: invalid application of 'sizeof' to an incomplete type 'const QITableEntry []'
      xpcom/base/AvailableMemoryTracker.cpp:77,1:: error: static_cast from '(anonymous namespace)::LowEventsReporter *' to 'nsISupports *', which are not related by inheritance, is not allowed
      xpcom/base/AvailableMemoryTracker.cpp:77,38:: error: unknown type name 'nsIMemoryReporter'
      xpcom/base/AvailableMemoryTracker.cpp:77,38:: error: use of undeclared identifier 'nsIMemoryReporter'
      xpcom/base/AvailableMemoryTracker.cpp:187,3:: error: use of undeclared identifier 'RegisterLowMemoryEventsPhysicalDistinguishedAmount'
      xpcom/base/AvailableMemoryWatcherWin.cpp:31,47:: error: base class has incomplete type
      xpcom/base/AvailableMemoryWatcherWin.cpp:37,3:: error: unknown type name 'NS_DECL_NSIOBSERVER'
      xpcom/base/AvailableMemoryWatcherWin.cpp:38,3:: error: 'virtual' can only appear on non-static member functions
      xpcom/base/AvailableMemoryWatcherWin.cpp:38,3:: error: expected ';' at end of declaration list
      xpcom/base/AvailableMemoryWatcherWin.cpp:38,3:: error: static_assert failed due to requirement 'std::is_convertible<mozilla::nsAvailableMemoryWatcher *, nsINamed *>::value' "nsITimerCallback implementations must also implement nsINamed"
      xpcom/base/AvailableMemoryWatcherWin.cpp:39,3:: error: must use 'enum' tag to refer to type 'nsresult' in this scope
      xpcom/base/AvailableMemoryWatcherWin.cpp:42,3:: error: must use 'enum' tag to refer to type 'nsresult' in this scope
      xpcom/base/AvailableMemoryWatcherWin.cpp:102,1:: error: cannot initialize object parameter of type 'mozilla::nsAvailableMemoryWatcherBase' with an expression of type 'mozilla::nsAvailableMemoryWatcher'
      xpcom/base/AvailableMemoryWatcherWin.cpp:102,1:: error: invalid application of 'sizeof' to an incomplete type 'const QITableEntry []'
      xpcom/base/AvailableMemoryWatcherWin.cpp:102,1:: error: must use 'enum' tag to refer to type 'nsresult' in this scope
      xpcom/base/AvailableMemoryWatcherWin.cpp:102,1:: error: static_cast from 'mozilla::nsAvailableMemoryWatcher *' to 'nsINamed *', which are not related by inheritance, is not allowed
      xpcom/base/AvailableMemoryWatcherWin.cpp:102,1:: error: static_cast from 'mozilla::nsAvailableMemoryWatcher *' to 'nsIObserver *', which are not related by inheritance, is not allowed
      xpcom/base/AvailableMemoryWatcherWin.cpp:102,1:: error: static_cast from 'mozilla::nsAvailableMemoryWatcher *' to 'nsITimerCallback *', which are not related by inheritance, is not allowed
      xpcom/base/AvailableMemoryWatcherWin.cpp:103,59:: error: 'nsIObserver' does not refer to a value
      xpcom/base/AvailableMemoryWatcherWin.cpp:103,59:: error: incomplete type 'nsIObserver' named in nested name specifier
      xpcom/base/AvailableMemoryWatcherWin.cpp:140,45:: error: cannot initialize a parameter of type 'nsIObserver *' with an rvalue of type 'mozilla::nsAvailableMemoryWatcher *'
      xpcom/base/AvailableMemoryWatcherWin.cpp:140,5:: error: must use 'enum' tag to refer to type 'nsresult' in this scope
      xpcom/base/CycleCollectedJSRuntime.cpp:1605,7:: error: use of undeclared identifier 'PR_GetEnv'
      xpcom/base/CycleCollectedJSRuntime.cpp:1605:7: error: use of undeclared identifier 'PR_GetEnv'
      xpcom/base/DeferredFinalize.cpp:12,15:: error: out-of-line definition of 'DeferredFinalize' does not match any declaration in namespace 'mozilla'
      xpcom/base/DeferredFinalize.cpp:12:15: error: out-of-line definition of 'DeferredFinalize' does not match any declaration in namespace 'mozilla'
      xpcom/base/MemoryTelemetry.cpp:48,27:: error: unused variable 'kTotalMemoryCollectorTimeout' [-Werror,-Wunused-const-variable]
      xpcom/base/MemoryTelemetry.cpp:48:27: error: unused variable 'kTotalMemoryCollectorTimeout' [-Werror,-Wunused-const-variable]
      xpcom/base/nsErrorService.cpp:42,21:: error: use of function template name with no prior declaration in function call with explicit template arguments is a C++20 extension [-Werror,-Wc++20-extensions]
      xpcom/base/nsErrorService.cpp:42,21:: error: use of undeclared identifier 'MakeUnique'; did you mean 'mozilla::MakeUnique'?
      xpcom/base/nsErrorService.cpp:42:21: error: use of function template name with no prior declaration in function call with explicit template arguments is a C++20 extension [-Werror,-Wc++20-extensions]
      xpcom/base/nsErrorService.cpp:42:21: error: use of undeclared identifier 'MakeUnique'; did you mean 'mozilla::MakeUnique'?
      xpcom/base/nsMacPreferencesReader.mm:20:12: error: implicit instantiation of undefined template 'nsTSubstring<char16_t>'
      xpcom/base/nsMacPreferencesReader.mm:24:60: error: use of undeclared identifier 'NSString'
      xpcom/base/nsMacPreferencesReader.mm:24:69: error: expected expression
      xpcom/base/nsMacPreferencesReader.mm:26:48: error: unknown type name 'NSArray'
      xpcom/base/nsMacPreferencesReader.mm:28:30: error: use of undeclared identifier 'NSString'
      xpcom/base/nsMacPreferencesReader.mm:29:30: error: no matching function for call to 'MakeStringSpan'
      xpcom/base/nsMacPreferencesReader.mm:29:51: error: instance method '-UTF8String' not found (return type defaults to 'id') [-Werror,-Wobjc-method-access]
      xpcom/base/nsMacPreferencesReader.mm:30:22: error: instance method '-isKindOfClass:' not found (return type defaults to 'id') [-Werror,-Wobjc-method-access]
      xpcom/base/nsMacPreferencesReader.mm:30:37: error: receiver type 'int (*)(int)' is not 'id' or interface pointer, consider casting it to 'id' [-Werror,-Wreceiver-expr]
      xpcom/base/nsMacPreferencesReader.mm:30:37: error: use of undeclared identifier 'NSNumber'; did you mean 'isnumber'?
      xpcom/base/nsMacPreferencesReader.mm:30:46: error: instance method '-class' not found (return type defaults to 'id') [-Werror,-Wobjc-method-access]
      xpcom/base/nsMacPreferencesReader.mm:31:27: error: cannot initialize a parameter of type 'int64_t' (aka 'long long') with an rvalue of type 'id'
      xpcom/base/nsMacPreferencesReader.mm:31:33: error: instance method '-longLongValue' not found (return type defaults to 'id') [-Werror,-Wobjc-method-access]
      xpcom/base/nsMacPreferencesReader.mm:32:37: error: use of undeclared identifier 'NSArray'
      xpcom/base/nsMacPreferencesReader.mm:34:7: error: no matching function for call to 'EvaluateArray'
      xpcom/base/nsMacPreferencesReader.mm:36:37: error: use of undeclared identifier 'NSDictionary'
      xpcom/base/nsMacPreferencesReader.mm:38:7: error: no matching function for call to 'EvaluateDict'
      xpcom/base/nsMacPreferencesReader.mm:44:60: error: use of undeclared identifier 'NSString'
      xpcom/base/nsMacPreferencesReader.mm:44:69: error: expected expression
      xpcom/base/nsSystemInfo.cpp:415,35:: error: no template named 'NotNull'; did you mean 'NonNull'?
      xpcom/base/nsSystemInfo.cpp:419,25:: error: member reference type 'NonNull<IWSCProductList *>' is not a pointer; did you mean to use '.'?
      xpcom/base/nsSystemInfo.cpp:419,27:: error: no member named 'get_Count' in 'mozilla::dom::NonNull<IWSCProductList *>'
      xpcom/base/nsSystemInfo.cpp:426,19:: error: member reference type 'NonNull<IWSCProductList *>' is not a pointer; did you mean to use '.'?
      xpcom/base/nsSystemInfo.cpp:426,21:: error: no member named 'get_Item' in 'mozilla::dom::NonNull<IWSCProductList *>'
      xpcom/base/nsSystemInfo.cpp:484,17:: error: use of undeclared identifier 'ArrayLength'; did you mean 'mozilla::ArrayLength'?
      xpcom/base/nsSystemInfo.cpp:484,47:: error: use of undeclared identifier 'ArrayLength'; did you mean 'mozilla::ArrayLength'?
      xpcom/base/nsSystemInfo.cpp:487,36:: error: use of undeclared identifier 'ArrayLength'; did you mean 'mozilla::ArrayLength'?
      xpcom/base/nsSystemInfo.cpp:500,10:: error: no matching function for call to 'EnumWSCProductList'
      xpcom/base/nsSystemInfo.cpp:500,46:: error: use of undeclared identifier 'WrapNotNull'; did you mean 'mozilla::WrapNotNull'?
      xpcom/base/nsTraceRefcnt.cpp:313,32:: error: use of undeclared identifier 'MakeUnique'
      xpcom/base/nsTraceRefcnt.cpp:313:32: error: use of undeclared identifier 'MakeUnique'
      xpcom/base/nsTraceRefcnt.cpp:461,19:: error: no matching member function for call to 'WithEntryHandle'
      xpcom/base/nsTraceRefcnt.cpp:461:19: error: no matching member function for call to 'WithEntryHandle'
      xpcom/base/nsTraceRefcnt.cpp:468,33:: error: use of function template name with no prior declaration in function call with explicit template arguments is a C++20 extension [-Werror,-Wc++20-extensions]
      xpcom/base/nsTraceRefcnt.cpp:468,33:: error: use of undeclared identifier 'MakeUnique'
      xpcom/base/nsTraceRefcnt.cpp:468:33: error: use of undeclared identifier 'MakeUnique'
      xpcom/base/nsTraceRefcnt.cpp:736,20:: error: use of undeclared identifier '_write'
      
      Differential Revision: https://phabricator.services.mozilla.com/D127036
      0679fea9
    • Mike Hommey's avatar
      Bug 1733308 - Forbid sources files with the same base name. r=firefox-build-system-reviewers,andi · ae874d3d
      Mike Hommey authored
      While the build system could be improved to actually support those
      cases, because of all the legacy things involved, it would be a lot more
      work for a low reward.
      
      Instead, we forbid them, which allows to remove the non-unified builds
      exception in xpcom/rust/gtest.
      
      Differential Revision: https://phabricator.services.mozilla.com/D127035
      ae874d3d
    • Doug Thayer's avatar
Loading