Kevin Grandon :kgrandon Reporter
	Description • 10 years ago

In order to sufficiently test our features we need to be able to emulate landscape support for b2g desktop while running MarionetteJS tests.

Since these will be running on real devices soon, I would hope that the same API might be able to be used to trigger landscape mode on a real device as well.

	Kevin Grandon :kgrandon Reporter
	Comment 1 • 10 years ago

Myk, Fabrice - Would you guys be able to weigh in on what's needed for this? I believe that the simulator can do something similar here, would we be able to reuse the code for b2g desktop?

	[:fabrice] Fabrice Desré
	Comment 2 • 10 years ago

(In reply to Kevin Grandon :kgrandon from comment #1)
> Myk, Fabrice - Would you guys be able to weigh in on what's needed for this?
> I believe that the simulator can do something similar here, would we be able
> to reuse the code for b2g desktop?

Likely, yes. It's just resizing the window, so I guess we won't trigger orientationchange events. Is that important?

	Kevin Grandon :kgrandon Reporter
	Comment 3 • 10 years ago

(In reply to Fabrice Desré [:fabrice] from comment #2)
> Likely, yes. It's just resizing the window, so I guess we won't trigger
> orientationchange events. Is that important?

Yes, I don't think a simple resize will do it. We need orientationchange events and CSS orientation psuedo-selectors to work.

	Kevin Grandon :kgrandon Reporter
	Comment 4 • 10 years ago

Code for simulator support is here: https://mxr.mozilla.org/mozilla-central/source/b2g/components/GlobalSimulatorScreen.jsm#55

	Kevin Grandon :kgrandon Reporter
	Comment 5 • 10 years ago

This is becoming important so I will do some initial investigation. Anyone is welcome to steal this from me in the meantime though.

	Myk Melez [:myk] [@mykmelez]
	Comment 6 • 10 years ago

Erm, sorry for the delay responding!  ochameau and jryans (cc:ed) are better positioned to respond now, as I haven't been hacking on Simulator code lately, but you're looking in the right place; also see desktop.js, where GlobalSimulatorScreen.flipScreen is called in response to UI events:

http://mxr.mozilla.org/mozilla-central/source/b2g/chrome/content/desktop.js#41
http://mxr.mozilla.org/mozilla-central/source/b2g/chrome/content/desktop.js#96

	J. Ryan Stinnett [:jryans] (Use needinfo, replies may be slow)
	Comment 7 • 10 years ago

Alex may have some suggestions on how to best add this, so flagging him for ni?.

	Kevin Grandon :kgrandon Reporter
	Comment 8 • 10 years ago

Thanks guys. It turns out that we can execute chrome JS from our tests, so this might not be too bad. It'll be easy for us to add a helper like: 

var chromeClient = client.scope({ context: 'chrome' });
chromeClient.executeScript(function() {
  GlobalSimulatorScreen.flipScreen();
});

This actually works today, but I think we are still missing some events for the system to resize things. I need to dig in to find out exactly what we're missing.

	Kevin Grandon :kgrandon Reporter
	Comment 10 • 10 years ago

That's a picture of B2G desktop after calling GlobalSimulatorScreen.flipScreen();. Seems like we're either not getting a resize event, or maybe the portrait/landscape CSS selectors are not yet working.

	Alexandre Poirot [:ochameau]
	Comment 11 • 10 years ago

Hum. Do you get different behavior between calling flipScreen and clicking manually on the rotate button?!!
Here it works fine if I use the buttons...

And given that clicking on the UI just call flipScreen()...

	Kevin Grandon :kgrandon Reporter
	Comment 14 • 9 years ago

I unfortunately haven't had time to look at this. Hope someone can take it soon though!

	Tim Guan-tin Chien [:timdream] (please needinfo)
	Comment 15 • 9 years ago

It looks like GlobalOrientationScreen and OrientationScreen are re-implementation of MozOrientation API in JS, which is deprecated with the landing of bug 1131470, the new API?

Is it possible to instead hook to the actual native code that in responsible of orientation management instead? I don't think we could rely on that since there were no tests landed w/ bug 944443.

(Just thinking aloud here -- maybe it's not doable at all)

	Tim Guan-tin Chien [:timdream] (please needinfo)
	Comment 16 • 9 years ago

Kind of related: bug 826913, bug 992131. It's better if we could share the code with some feature arguably with more users.

	Tim Guan-tin Chien [:timdream] (please needinfo)
	Comment 17 • 9 years ago

(In reply to Tim Guan-tin Chien [:timdream] (please needinfo) from comment #15)
> Is it possible to instead hook to the actual native code that in responsible
> of orientation management instead?

OK. That's a really bad idea. Both old and the new APIs calls Hal.cpp directly, and the actual backend is responsible of keeping and maintaining the orientation lock. For example, [1] is how Gonk do it.

[1] https://dxr.mozilla.org/mozilla-central/source/widget/gonk/OrientationObserver.cpp#262

So in order for B2G Desktop/Mulet to support a fake orientation, we would need to change all of the calls and insert a middle layer that could manage orientation independently; and it's worse for DevTools responsive mode because to work with that the middle layer would need to manage orientation *per window*.

I don't think this is attainable (or worth the effort) if we are only looking at solving the use case on running tests. Unless DevTools team is seriously thinking about working on the feature and help this bug along the way.

	Tim Guan-tin Chien [:timdream] (please needinfo)
	Comment 18 • 9 years ago

BTW I know we are not running Gij on Emulator, but with bug 975359 test scripts can already control the orientation of Emulator.

Maybe it's worthy to think about whether or not to run Gij on Emulator?

	Kan-Ru Chen [:kanru] (UTC+9)
	Comment 19 • 9 years ago

(In reply to Tim Guan-tin Chien [:timdream] (OOO Nov 16-26; please needinfo) from comment #17)
> (In reply to Tim Guan-tin Chien [:timdream] (please needinfo) from comment
> #15)
> > Is it possible to instead hook to the actual native code that in responsible
> > of orientation management instead?
> 
> OK. That's a really bad idea. Both old and the new APIs calls Hal.cpp
> directly, and the actual backend is responsible of keeping and maintaining
> the orientation lock. For example, [1] is how Gonk do it.

I think we could fake at the hal level. For example make hal::GetCurrentScreenConfiguration return fake data based on pref.

> [1]
> https://dxr.mozilla.org/mozilla-central/source/widget/gonk/
> OrientationObserver.cpp#262

	Tim Guan-tin Chien [:timdream] (please needinfo)
	Comment 20 • 9 years ago

Looks like a nice project to work on after my PTO!

	Tim Guan-tin Chien [:timdream] (please needinfo)
	Comment 21 • 9 years ago

Some update on what I've found in trying to achieve comment 19: Here are how content talk to the HAL implementation.

Parent process content <--> DOM APIs <----------------------------------> Hal.cpp <---> (hal_impl)
Content process content <-> DOM APIs <--> Hal.cpp <-> HalSandbox.cpp <-->

There are two implementations of DOM APIs, and more than four HAL implementation in the tree. So intercepting the call in the one and only Hal.cpp on the parent process is our only way to prevent repetition.

The next best place is within the DOM APIs but it would then have to create it's own testing IPC messages...

So far I am trying to insert an ObserverService#NotifyObservers() into hal::LockScreenOrientation, but I have trouble converting dom::ScreenOrientationInternal (an uint32_t) to char16_t that can be sent through observers. Going to ask around for C++ help.

	Tim Guan-tin Chien [:timdream] (please needinfo)
	Comment 23 • 9 years ago

Base on priority of my work and the project, I can no longer work on this. Feel free to take my patch and carry on...

	Mantaroh Yoshinaga[:mantaroh]
	Comment 24 • 9 years ago

Hi tim.

I faced similar problem(bug 1241385).

What is the lack of your patch?
I think that we should take task as follow.
 - remove Simulator.js from b2g.
 - remove the code of calculating orientation from gaia(orientation_manager.js).

Am I right?

	Tim Guan-tin Chien [:timdream] (please needinfo)
	Comment 25 • 9 years ago

(In reply to Mantaroh Yoshinaga[:mantaroh] from comment #24)
> Hi tim.
> 
> I faced similar problem(bug 1241385).
> 
> What is the lack of your patch?
> I think that we should take task as follow.
>  - remove Simulator.js from b2g.

I don't really know what this file do, but it should be done in this bug if we have something equivalent to that.

>  - remove the code of calculating orientation from
> gaia(orientation_manager.js).

Yes we should do this in another bug.

> 
> Am I right?

	BMO Automation
	Comment 26 • 6 years ago

Firefox OS is not being worked on