Closed Bug 935178 Opened 11 years ago Closed 11 years ago

Story - Support broadcast message for switching between Metro and Desktop

Categories

(Firefox for Metro Graveyard :: General, defect, P1)

x86_64
Windows 8.1
defect

Tracking

(Not tracked)

RESOLVED FIXED
Firefox 28

People

(Reporter: bbondy, Assigned: bbondy)

References

Details

(Whiteboard: [block28][completed-oak] feature=story c=tbd u=tbd p=3)

Attachments

(1 file)

We should add support for auto-switch to Metro and auto-switch to Desktop browsers.

To do this we should follow Intels' guidelines here:
http://software.intel.com/en-us/articles/detecting-slateclamshell-mode-screen-orientation-in-convertible-pc

Basically we'll need to add to our WinRT and Win32 widget layer, message handling for WM_SETTINGS with an lPram of “ConvertibleSlateMode”.

To detect the mode we'd use GetSystemMetrics:
BOOL bSlateMode = (GetSystemMetrics(SM_CONVERTIBLESLATEMODE) == 0);

We'd use the code from bug 924911 and bug 924914 to do a relaunch in response to these messages if a mode switch is detected and the wrong browser is running.

p=3

Note for Marco: I'm going to delay subtracting this from the contingency fund until we get hardware that supports this.  That way we can verify a good experience.
Whiteboard: [release28] feature=story c=tbd u=tbd p=0
Removed 3 points from the contingency bug for this.
Whiteboard: [release28] feature=story c=tbd u=tbd p=0 → [release28] feature=story c=tbd u=tbd p=3
Blocks: metrov1it19
Dell device from Intel that supports this just arrived this morning. Going to aim to get this work in during this 2-week iteration sprint.
Status: NEW → ASSIGNED
Priority: -- → P1
QA Contact: jbecerra
Whiteboard: [release28] feature=story c=tbd u=tbd p=3 → [block28] feature=story c=tbd u=tbd p=3
Attached patch rev1Splinter Review
A bit of background, mfinkle, krudnitski, and I had a meeting with Intel about supporting this functionality. They sent myself and Juan an XPS 12 to develop and test the functionality. 

Basically when you transform the laptop into a tablet, and you have the desktop browser open, it will close that and open the Metro browser.  Likewise, when you are in tablet mode, and you transform to a laptop, if you have the Metro browser open it will close and open the Desktop browser.

See this link for more information:
http://software.intel.com/en-us/articles/detecting-slateclamshell-mode-screen-orientation-in-convertible-pc

I tried it out and it feels awesome to me :)
Attachment #8333594 - Flags: review?(jmathies)
We should tie this to a pref so people can turn it off.
Do these tablets automatically switch between the touch interface and desktop? I'm curious what else happens on the device when you switch modes.
Just to keep this bug sync'ed up to IRC:
[13:27:57] <bbondy> jimm: ping, check out this video http://dot.kde.org/2013/09/11/krita-demonstrated-
[13:28:13] <bbondy> witch-> switch
[13:29:53] <jimm> bbondy: thx
[13:30:19] <bbondy> that video shows a paint program doing the same thing
[13:30:25] <bbondy> they generally switch via file activation
[13:30:30] <bbondy> we do via url activation
Comment on attachment 8333594 [details] [diff] [review]
rev1

bbondy's going to add a pref in a follow up.
Attachment #8333594 - Flags: review?(jmathies) → review+
https://hg.mozilla.org/projects/oak/rev/95f653bf41d9

Bug 935099 will track the landing on m-c.
When it lands on m-c a new comment will be added here as well with the m-c changeset.
This is being done so we can still use scrumbugs efficiently. 
See bug 935099 for further details.
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Depends on: 940677
Blocks: 935099
Whiteboard: [block28] feature=story c=tbd u=tbd p=3 → [block28][completed-oak] feature=story c=tbd u=tbd p=3
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Status: REOPENED → NEW
https://hg.mozilla.org/integration/fx-team/rev/594ae98c4fdd
Target Milestone: --- → Firefox 28
https://hg.mozilla.org/mozilla-central/rev/594ae98c4fdd
Status: NEW → RESOLVED
Closed: 11 years ago11 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: