There is some ambiguity about whether ScheduleComposite will necessarily
trigger a composite all the way to nsWindow::DrawWindowUnderlay. Android
robocop tests assume it will, because they rely on DrawWindowOverlay
being called so they can take a screenshot and make progress,
but this is a very fragile assumption. They also rely on the entire
window being painted, which is also a fragile assumption.
This patch improves the situation by explicitly invalidating the current
window area when Android Java code needs to trigger a composite. This avoids
regressions from future patches in this series which make composition bail
out when there is nothing invalid.
The resulting setup is still a bit fragile for my taste but I'm not sure
what the ideal solution would be.
--HG--
extra : commitid : 3t3xqRdZs24
extra : rebase_source : b23749613663ca805484776ccf5e36b4ff00e3fe
Otherwise we can get a crash with the following stack:
Program received signal SIGSEGV, Segmentation fault.
[Switching to Thread 14711]
0x5d99974e in mozilla::gl::GLContext::BeforeGLCall (this=0x6dbf0800,
funcName=0x60f251a4 <mozilla::gl::GLContext::raw_fDeleteProgram(unsigned int)::__PRETTY_FUNCTION__> "void mozilla::gl::GLContext::raw_fDeleteProgram(GLuint)") at /home/roc/mozilla-inbound/gfx/gl/GLContext.h:683
683 MOZ_ASSERT(IsCurrent());
(gdb) where
#0 0x5d99974e in mozilla::gl::GLContext::BeforeGLCall (this=0x6dbf0800,
funcName=0x60f251a4 <mozilla::gl::GLContext::raw_fDeleteProgram(unsigned int)::__PRETTY_FUNCTION__> "void mozilla::gl::GLContext::raw_fDeleteProgram(GLuint)") at /home/roc/mozilla-inbound/gfx/gl/GLContext.h:683
#1 0x5d99bed6 in mozilla::gl::GLContext::raw_fDeleteProgram (this=0x6dbf0800, program=210003)
at /home/roc/mozilla-inbound/gfx/gl/GLContext.h:2232
#2 0x5d99c10a in mozilla::gl::GLContext::fDeleteProgram (this=0x6dbf0800, program=210003)
at /home/roc/mozilla-inbound/gfx/gl/GLContext.h:2270
#3 0x5daa0ae6 in mozilla::layers::ShaderProgramOGL::~ShaderProgramOGL (this=0x6d7df000, __in_chrg=<optimized out>)
at /home/roc/mozilla-inbound/gfx/layers/opengl/OGLShaderProgram.cpp:491
#4 0x5da86bdc in mozilla::layers::CompositorOGL::CleanupResources (this=0x67ae4d70)
at /home/roc/mozilla-inbound/gfx/layers/opengl/CompositorOGL.cpp:177
--HG--
extra : commitid : LPnSogXNNio
extra : rebase_source : 0564dd5688916271c4a709ae6f15ba7ad493a761
On some Android versions, GraphicBuffer.h ends up including libui's
hardware.h, which #defines the symbols version_minor and version_major, which
are used as field names in Ogg Theora's th_info struct. Later patches will
require some files to include both Theora headers and LayerTypes.h.
--HG--
extra : commitid : HM7e5zfyoAz
extra : rebase_source : de8dd7e5bc6bfcfeb729cfab86fe44b8bb12ab80
========
https://hg.mozilla.org/integration/gaia-central/rev/b126eb6c8cb4
Author: Martijn <martijn.martijn@gmail.com>
Desc: Merge pull request #30846 from mozilla-b2g/revert-30841-1179993
Revert "Bug 1179993 - debug.peformancedata.shared should be debug.per...", because it caused a test failure in make_test.js
========
https://hg.mozilla.org/integration/gaia-central/rev/5b6d780a0f81
Author: Martijn <martijn.martijn@gmail.com>
Desc: Revert "Bug 1179993 - debug.peformancedata.shared should be debug.performance_data.shared in common-settings.json"
========
https://hg.mozilla.org/integration/gaia-central/rev/d5367c89aeb2
Author: Martijn <martijn.martijn@gmail.com>
Desc: Merge pull request #30845 from mwargers/1176725_enable
Bug 1176725 - Intermittent failures in test_rocketbar_add_collection_save_bookmark.py is fixed, so re-enabling this test
========
https://hg.mozilla.org/integration/gaia-central/rev/87b92553b803
Author: Martijn Wargers <mwargers@mozilla.com>
Desc: Bug 1176725 - Intermittent failures in test_rocketbar_add_collection_save_bookmark.py is fixed, so re-enabling this test
We should never create DocAccessibles for documents that are going away so it
seems like this shouldn't be necessary, but without a test case its hard to
know why we are creating DocAccessibles for documents without docshells. So
for now work around the issue and hope it doesn't matter in practice.
========
https://hg.mozilla.org/integration/gaia-central/rev/0c9d26b2d715
Author: Fernando Jiménez Moreno <ferjmoreno@gmail.com>
Desc: Merge pull request #30840 from ferjm/bug1180727.browser.ui.test
Bug 1180727 - Use manifest URL instead of position to select activity handler on browser functional tests. r=jlorenzo, mwargers
========
https://hg.mozilla.org/integration/gaia-central/rev/c3a9a8492353
Author: Fernando Jiménez Moreno <ferjmoreno@gmail.com>
Desc: Bug 1180727 - Use manifest URL instead of position to select activity handler on browser functional tests. r=jlorenzo