The bulk of this commit was generated by running:
run-clang-tidy.py \
-checks='-*,llvm-namespace-comment' \
-header-filter=^/.../mozilla-central/.* \
-fix
We'll need this later so ImageHost can select the correct image to use.
Adding a TimeStamp parameter to BeginFrame is a bit annoying since BeginFrame
is overridden by every subclass. It's a bit more convenient to just call a
separate non-virtual method just before we call BeginFrame.
--HG--
extra : commitid : 3QUviDozXL
extra : rebase_source : ba6bc3d846606f3f7598f4a3b687b68746067ce3
This conversion was done with the script:
find . -name '*.cpp' -o -name '*.h' -o -name '*.mm' -o -name '*.idl' | \
egrep -v 'cairo-win32-refptr.h|RefPtr.h|TestRefPtr.cpp' | \
xargs sed -i -e 's/mozilla::TemporaryRef</already_AddRefed</g' \
-e 's/TemporaryRef</already_AddRefed</g'
Manual fixups were performed in the following instances:
- We handled mfbt/RefPtr.h manually so as to not convert TemporaryRef itself
into already_AddRefed.
- The following files had explicit Move() calls added to make up for the lack
of a copy constructor on already_AddRefed:
dom/base/ImageEncoder.cpp
dom/media/MediaTaskQueue.{h,cpp}
dom/media/webaudio/PannerNode.cpp
- A redundant overload for MediaTaskQueue::Dispatch was deleted.
- A few manual fixups were required in mfbt/tests/TestRefPtr.cpp.
- Comments, using declarations, and forward declarations relating to
TemporaryRef in dom/canvas/ and gfx/layers/ were changed to refer to
already_AddRefed.
We can run these now that we don't split out gkmedias.
This also changes a test to avoid MSVC codegening a divide by 0
--HG--
extra : rebase_source : 06af10f1d6779237d2cc6c1dabfd9378889893ee
Part 5 adds an assert that we always return an APZC from GetAPZCAtPoint. The test for bug 1119497 fails this assumption because it does not give it's layer tree a root AZPC. In all non-test code we ensure that there is always a root APZC and many things would break without such. So we modify the test to always have a root APZC more like actual code.
The test is testing that we don't return the layer below (with an APZC) the topmost child layer (without an APZC). So checking if we hit the root APZC still makes the test test the same thing.
Having this implicit conversion means that we can silently do extra
refcounting when it's completely unnecessary. It's also an obstacle to
making RefPtr more nsRefPtr-like, so let's get rid of it.
Change interface of getter/setter for mClipRect,
also necessary modification for codes that use these resources.
* * *
Bundle mUseClipRect and mClipRect as Maybe<ParentLayerIntRect> mClipRect
--HG--
extra : rebase_source : ca0c60404a28b7418df88104b863760033289910
Matt Woodrow
Ryan VanderMeulen
Wes Kocher
Bobby Holley
Birunthan Mohanathas
Robert O'Callahan
Lynn Tran
Benoit Girard
Nicolas Silva
Nathan Froyd
Kartikaya Gupta
Jeff Muizelaar
Botond Ballo
Amanda Sambath
Timothy Nikkel
Tom Klein
Mike Hommey
Milan Sreckovic
Thibaud Backenstrass
Ying-Ruei Liang (KK)
Jonathan Kew