Add-on errors, missing navbar buttons, start page doesn't work after switching from desktop

VERIFIED FIXED in Firefox 28

Status

defect
P1
normal
VERIFIED FIXED
6 years ago
6 years ago

People

(Reporter: guigs, Assigned: mbrubeck)

Tracking

28 Branch
Firefox 30
x86
Windows 8.1
Dependency tree / graph

Firefox Tracking Flags

(firefox27 unaffected, firefox28+ verified, firefox29 verified, firefox30 verified, b2g-v1.3 fixed)

Details

(Whiteboard: [release28] p=2 s=it-30c-29a-28b.3 r=ff28 [qa+])

Attachments

(5 attachments, 1 obsolete attachment)

Steps taken: 
Windows 8 Touch version on my Surface Pro 2. 

1. Relaunched via menu link in desktop version.
2.  Opened in Metro on my home page ok, but no menu bar/ icon.
3.  Closed and reopened using Metro icon.
4.  Opened to the start page? 
where it shows links, history, etc. None of the links worked and there is no menu bar visible. Forced to restart in desktop mode. 

Other observations: 
" In the app itself there is no menu. When it opens form the Metro side of windows it show a start page with tiles/links for bookmarks recent history top sites but the links don't open and there is no menu bar. Not sure if it a Surface issue or the beta. "

    Shockwave Flash 12.0 r0
    Next Generation Java Plug-in 10.51.2 for Mozilla browsers
    NPRuntime Script Plug-in Library for Java(TM) Deploy 




add ons
    App Button Color 1.0 (appbuttoncolor@mozilla.org)
    Fasterfox Lite 3.9.9Lite (FasterFox_Lite@BigRedBrent)
    FastestFox 5.2.1 (smarterwiki@wikiatic.com)
    FAYT 2.0.5 ({c2d0e930-64de-11db-bd13-0800200c9a66})
    FlashGot 1.5.5.97 ({19503e42-ca3c-4c27-b1e2-9cdb2170ee34})
    Free Download Manager plugin 1.6.0.1 (fdm_ffext@freedownloadmanager.org)
    Google Toolbar for Firefox 7.1.20110512W ({3112ca9c-de6d-4884-a869-9855de68056c})
    Greasemonkey 1.15 ({e4a8a97b-f2ed-450b-b12d-ee082ba24781})
    IE Tab 2 (FF 3.6+) 5.12.12.1 ({1BC9BA34-1EED-42ca-A505-6D2F1A935BBB})
    IE View 1.5.6 ({6e84150a-d526-41f1-a480-a67d3fed910d})
    LastPass 3.0.12 (support@lastpass.com)
    Movable Firefox Button 1.4 (movableAppButton@Merci.chao)
    NewTabURL 2.2.3 (newtaburl@sogame.cat)
    Password Exporter 1.2.1 ({B17C1C5A-04B1-11DB-9804-B622A1EF5492})
    Places Maintenance 1.3 (places-maintenance@bonardo.net)
    Saved Password Editor 2.7.2 (savedpasswordeditor@daniel.dawson)
    Status-4-Evar 2013.10.31.22 (status4evar@caligonstudios.com)
    Stylish 1.4.1 ({46551EC9-40F0-4e47-8E18-8E5CF550CFB8})
    Troubleshooter 1.1a (troubleshooter@mozilla.org)
    Utopia FFSE White Options 5.0.4 (utopia_ffse_white_options@www.theme-oasis.org)
    Vacuum Places Improved 1.2 (VacuumPlacesImproved@lultimouomo-gmail.com)
    WebSlingPlayer 1.5.12.732 ({9EB34849-81D3-4841-939D-666D522B889A})
    Xmarks 4.2.5 (foxmarks@kei.com)
    Microsoft .NET Framework Assistant 1.2.1 ({20a82645-c095-46ed-80e3-08825760534b}) (Inactive)


Please see: https://support.mozilla.org/en-US/questions/987521 for full story
Posted image Screenshot (1).png
Updated to the new beta 02/26/14.  Exhibited the same behavior as above.  Restarted Firefox in Safe mode with addons disabled, then restarted in Metro touch mode. No change. When started clean in touch mode the "Start page" opens with the tile links. When selected/touched the link animates/depresses? but does not open.
Posted image Screenshot (2).png
Posted image Screenshot (3).png
Posted image Screenshot (4).png
Blocks: metrobacklog
Whiteboard: [triage]
This looks pretty serious, probably has to do with session restore.

Tom when you get this, are there any errors in the console? (ctrl-shift-j)
Component: Components → Browser
Flags: needinfo?(rmcguigan)
OS: Mac OS X → Windows 8.1
Priority: -- → P1
Summary: Touch not working on Surface[user story]No address bar The tiles in the start page depress/move → Nav bar is missing buttons, start page doesn't work after switching from desktop
Selecting "errors" only in the console produces a list of 15-20.  A number are related to addons from desktop mode.  Restarting in safe mode and then restarting in touch mode produce the same results.  I assume no way start touch mode in safe mode.  But there are errors that look specific to touch mode?  I'd have to copy each individually as there does not seem to be a way of selecting all.
(In reply to Tom from comment #7)
> Selecting "errors" only in the console produces a list of 15-20.  A number
> are related to addons from desktop mode.  Restarting in safe mode and then
> restarting in touch mode produce the same results.  I assume no way start
> touch mode in safe mode.  But there are errors that look specific to touch
> mode?  I'd have to copy each individually as there does not seem to be a way
> of selecting all.

Hmm, desktop addons should be disabled in metrofx. Have you changed any addon related prefs through the metro browsers about:config?
Jim which about config is that? dom.w3c_touch_events.expose: False  ?
Flags: needinfo?(rmcguigan)
(In reply to rmcguigan from comment #9)
> Jim which about config is that? dom.w3c_touch_events.expose: False  ?

just addon prefs in general. Would you mind visiting about:support in metrofx and copy pasting the info into this bug?
Posted patch patch (obsolete) — Splinter Review
I'm assuming this is caused by one or more of the errors from the desktop add-ons.  We meant to disable loading of add-ons completely in bug 946296, but we didn't do it correctly.  This patch should prevent loading of add-ons that were already installed into the profile via desktop Firefox.
Assignee: nobody → mbrubeck
Status: NEW → ASSIGNED
Attachment #8383818 - Flags: review?(robert.strong.bugs)
Blocks: 946296
Summary: Nav bar is missing buttons, start page doesn't work after switching from desktop → Add-on errors, missing navbar buttons, start page doesn't work after switching from desktop
Whiteboard: [triage] → [release28]
Posted patch patchSplinter Review
forgot to qref
Attachment #8383818 - Attachment is obsolete: true
Attachment #8383818 - Flags: review?(robert.strong.bugs)
Attachment #8383821 - Flags: review?(robert.strong.bugs)
(In reply to Jim Mathies [:jimm] from comment #8)
> (In reply to Tom from comment #7)
> > Selecting "errors" only in the console produces a list of 15-20.  A number
> > are related to addons from desktop mode.  Restarting in safe mode and then
> > restarting in touch mode produce the same results.  I assume no way start
> > touch mode in safe mode.  But there are errors that look specific to touch
> > mode?  I'd have to copy each individually as there does not seem to be a way
> > of selecting all.
> 
> Hmm, desktop addons should be disabled in metrofx. Have you changed any
> addon related prefs through the metro browsers about:config?

No.  Couldn't if I wanted to as the address bar has never been diplayed.
Whatever fix we come up with here is a must uplift to beta. Desktop addons are currently leaking through into metrofx.
Comment on attachment 8383821 [details] [diff] [review]
patch

Given that this is in sync with the method to disable add-ons that was implemented in bug 946296 I think this is the right approach. I'd like Mossop or Unfocused to take a look as well.
Attachment #8383821 - Flags: review?(robert.strong.bugs)
Attachment #8383821 - Flags: review?(dtownsend+bugmail)
Attachment #8383821 - Flags: review+
Attachment #8383821 - Flags: review?(dtownsend+bugmail) → review+
Blocks: metrov1backlog
No longer blocks: metrobacklog
Whiteboard: [release28] → [release28] p=0
QA Contact: kamiljoz
Whiteboard: [release28] p=0 → [release28] p=0 s=it-30c-29a-28b.3 r=ff28
Whiteboard: [release28] p=0 s=it-30c-29a-28b.3 r=ff28 → [release28] p=2 s=it-30c-29a-28b.3 r=ff28
https://hg.mozilla.org/mozilla-central/rev/8aad3de473ee
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 30
For testing and verification.  Reopen if any defects found.
Flags: needinfo?(kamiljoz)
Today's the last FF28 beta - can you nom with a risk/reward assessment here?  If this is Metro only and low risk to land we can consider it but it must be in the next hour or two.
Flags: needinfo?(mbrubeck)
Comment on attachment 8383821 [details] [diff] [review]
patch

[Approval Request Comment]
Bug caused by (feature/regressing bug #): long standing bug
User impact if declined: desktop addons leak into metrofx breaking it.
Testing completed (on m-c, etc.): yes.
Risk to taking this patch (and alternatives if risky): low.
String or IDL/UUID changes made by this patch: none.
Attachment #8383821 - Flags: approval-mozilla-beta?
Attachment #8383821 - Flags: approval-mozilla-aurora?
Flags: needinfo?(mbrubeck)
Attachment #8383821 - Flags: approval-mozilla-beta?
Attachment #8383821 - Flags: approval-mozilla-beta+
Attachment #8383821 - Flags: approval-mozilla-aurora?
Attachment #8383821 - Flags: approval-mozilla-aurora+
Went through the verification process using the following build:
- http://ftp.mozilla.org/pub/mozilla.org/firefox/nightly/2014-03-06-03-02-01-mozilla-central/

Before going through the verification process, I installed several of the addons from comment #0 and reproduced the issue using the following build:
- http://ftp.mozilla.org/pub/mozilla.org/firefox/nightly/2014-02-25-03-02-01-mozilla-central/

- Ensured that the URL text field is appearing correctly in the Navigation App Bar under about:start
- Ensured that all the buttons in the Navigation App Bar are appearing in the correct locations under about:start
- Ensured that all the tiles are selectable via the about:start window
- Ensure that you can delete, unselect and undelete selected tiles from Top Sites, Bookmarks, Recent History
- Ensured that all the blank tiles are not selectable under about:start window
- Ensured that the overlays are working correctly (Back and New Tab)
- Ensured that you can switch between fxdesktop and fxmetro while addons are installed under fxdesktop (tried this many times with different variations of tabs opened)
- Ensured that you can scroll through websites, click on links and open context menu's
- Ensured that all the buttons under the Navigation App Bar are working (Bookmark, Pin and the Settings Menu)
- Enusred you can use the Navigation App Bar settings (Find in page, Relaunch in Desktop, View page source)
- Ensured that you can refresh the website being viewed
- Ensured that you can snap fxmetro into different configurations
- Ensured that all the flyouts under the Settings Charm are sliding into view
- Ensured that the auto search feature is working correctly without any issues
- Ensured that you can type in URL's into the URL field and visit the desired website
- Ensured that you can swipe through the Tap App Bar
- Ensured that you can select and remove all the tabs under the Tap App Bar
- Reproduced the original issue using the build mentioned above, updated fxmetro using the "About" flyout to the latest Nightly went through all of the above test cases
Flags: needinfo?(kamiljoz)
Whiteboard: [release28] p=2 s=it-30c-29a-28b.3 r=ff28 → [release28] p=2 s=it-30c-29a-28b.3 r=ff28 [qa+]
Flags: needinfo?(kamiljoz)
I couldn't reproduce this issue with the Nightly from 2014-02-25 on a Dell XPS 12 ultrabook with Windows 8 64-bit. I've installed several add-ons while in Desktop mode (App Button Color 1.0, FlashGot 1.5.5.97, Greasemonkey 1.15, LastPASS 2.0.20) and then switched to Metro mode, but everything worked fine for me.

Kamil, could you please also check this with the latest beta, since you could reproduce the issue? Thanks!
Went through the verification process using the following builds:
- http://ftp.mozilla.org/pub/mozilla.org/firefox/nightly/2014-03-06-00-40-01-mozilla-aurora/

- Went through the original test case from comment #0 and ensured I couldn't reproduce the issue anymore
- Went through the test cases from comment #22

I reproduced the issue and than updated and made sure the problem was resolved once the update was complete. Once that was done, also ran through all the test cases.
I can confirm as an end user, that the latest v28 beta (BuildID=20140306171728) has fixed the issues on my Surface 2 Pro.
Awesome! Thanks for checking Tom, much appreciated!

Went through the verification process using the following builds:
- http://ftp.mozilla.org/pub/mozilla.org/firefox/releases/28.0b9/win32/en-US/

- Went through the original test case from comment #0 and ensured I couldn't reproduce the original issue
- Went through all the test cases from comment #22

I reproduced the original issue and than updated and ensured the problem was resolved once the update was complete. Once that was done, also ran through all the test cases.
Status: RESOLVED → VERIFIED
Flags: needinfo?(kamiljoz)
Keywords: verifyme
You need to log in before you can comment on or make changes to this bug.