1038997 - Backout newtab changes including layout and styling changes for beta

Status: VERIFIED FIXED

(Firefox :: New Tab Page, defect)

Description

[Ed Lee :Mardak]

For 31 currently in beta, we want to backout all newtab related changes to avoid bug 1000097

Comment 1

[Ed Lee :Mardak]

Attachment: v1

https://tbpl.mozilla.org/?tree=Try&rev=405f57f4722e

Comment 2

[Ed Lee :Mardak]

For the latest attempt, I reverted newtab related directories to just before this commit:

* a682dda - Bug 895359 - Switch about:newtab to Promise.jsm and remove remaining callback r=jaws (6 months ago) <Tim Taubert>

git di --binary a682dda^ browser/{base/content{,/test},themes/{linux,osx,windows}}/newtab | git apply -R

Then added back in various commits:
https://github.com/Mardak/tiles-dev/pull/40


Tests are failing because drag/drop events aren't being synthesized correctly:
https://tbpl.mozilla.org/?tree=Try&rev=ff4cda08a684
https://tbpl.mozilla.org/php/getParsedLog.php?id=43899728&tree=Try
uncaught exception - TypeError: win is null at chrome://mochitests/content/browser/browser/base/content/test/newtab/head.js:465
https://hg.mozilla.org/try/file/a557e47d7c26/browser/base/content/test/newtab/head.js#l465
   462 function synthesizeNativeMouseEvent(aElement, aMsg, aOffsetX = 0, aOffsetY = 0) {

   463   let rect = aElement.getBoundingClientRect();

   464   let win = aElement.ownerDocument.defaultView;

   465   let x = aOffsetX + win.mozInnerScreenX + rect.left + rect.width / 2;

Comment 3

[Ed Lee :Mardak]

So backing out newtab related changes didn't seem to work and backing out even more still didn't work, so I tried to revert just bug 991210 bug 980014 bug 978338 and patch up search by making the search-form flex 5 and padding it on both sides with <div class="newtab-side-margin"/>.

https://tbpl.mozilla.org/?tree=Try&rev=b5e6c1fce07a
https://github.com/Mardak/tiles-dev/pull/41

Comment 4

[Ed Lee :Mardak]

Attachment: 30 Release screenshot

Comment 5

[Ed Lee :Mardak]

Attachment: 31 Beta backout screenshot

http://ftp.mozilla.org/pub/mozilla.org/firefox/try-builds/[e-mail]-b5e6c1fce07a/try-macosx64/

Comment 6

[Ed Lee :Mardak]

Attachment: 30/31 scrollbar difference overlap

With the backedout beta keeping the search bar, the scrollbars start appearing at a bigger height. See attached overlapped view of 31 vs 30.

Comment 7

[Ed Lee :Mardak]

Attachment: v2

There were 4 steps involved in this patch:
- Revert Bug 980014
- Revert Bug 978338
- Revert Bug 991210
- Fix up search form 
https://github.com/Mardak/tiles-dev/pull/41/commits

Comment 8

[Ed Lee :Mardak]

Attachment: release-to-backout.patch

Not sure if this helps, but these are the changes to browser newtab code since Release.

Comment 9

[Ed Lee :Mardak]

https://hg.mozilla.org/releases/mozilla-release/rev/2552ea57100d

Comment 10

[u279076]

(In reply to Ed Lee :Mardak from comment #9)
> https://hg.mozilla.org/releases/mozilla-release/rev/2552ea57100d

Setting the status flag to fixed so it shows up in our RC2 verification query. 

Also marking ESR status flag since this needs to land for 31esr AFAIK.

Comment 11

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-esr31/rev/acedd56e2e08

Comment 12

[Ed Lee :Mardak]

Attachment: for aurora

I cherry-picked the patch to aurora, but bc1 is failing:
https://tbpl.mozilla.org/?tree=Try&rev=17ae17d2bcc7

TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/browser/base/content/test/general/browser_tabopen_reflows.js | Test timed out
TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/browser/base/content/test/general/browser_tabopen_reflows.js | application timed out after 330 seconds with no output
TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/browser/base/content/test/general/browser_tabopen_reflows.js | application terminated with exit code 5
PROCESS-CRASH | chrome://mochitests/content/browser/browser/base/content/test/general/browser_tabopen_reflows.js | application crashed [@ libSystem.B.dylib + 0xd7a]

Comment 13

[Ed Lee :Mardak]

Actually, the test failures in comment 12 seem to be caused by bug 1039881.

Comment 14

[Ed Lee :Mardak]

Comment on attachment 8458055 [details] [diff] [review]
for aurora

Approval Request Comment
[Feature/regressing bug #]: revert bug 980014, bug 978338
[User impact if declined]: users will see a new tiles layout in 32 then a different one in 33
[Describe test coverage new/current, TBPL]: https://tbpl.mozilla.org/?tree=Try&rev=0768f53e2850
[Risks and why]: low risk - already landed for 31rc2
[String/UUID change made/needed]: none

Comment 15

[Cornel Ionce [:noni] [Hubs QA]]

Verified that the mentioned newtab changes were successfully backed-out from:
1. Firefox 31 RC build 2 (build ID: 20140716183446)
2. Firefox 31 ESR (build ID: 20140717132905).

Changing the tracking flags accordingly. I will leave the "verifyme" keyword untouched until all branches will be fixed and verified.

Comment 16

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-aurora/rev/238174823b7c

Comment 17

[Ryan VanderMeulen [:RyanVM]]

Backed out from Aurora due to tart crashes that appear to have been triggered by this.
https://hg.mozilla.org/releases/mozilla-aurora/rev/5bb486b0f89a

https://tbpl.mozilla.org/php/getParsedLog.php?id=44124951&tree=Mozilla-Aurora

Comment 18

[Ed Lee :Mardak]

Attachment: for aurora checkin

This bug is clear of the tart crash: https://tbpl.mozilla.org/?tree=Try&rev=9a0ba6ff126f

Comment 19

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-aurora/rev/4e4056fa74af

Comment 20

[Ed Lee :Mardak]

Fixed for both 31 (beta -> release) and 32 (aurora -> beta)

Comment 21

[Cornel Ionce [:noni] [Hubs QA]]

Verified fixed on Firefox 32.0a2 Aurora, build ID: 20140721004001.
I'll keep an eye on this fix to make sure it landed properly in Firefox 31 release and Firefox 32 beta.

Comment 22

[Cornel Ionce [:noni] [Hubs QA]]

This landed properly in Firefox 31 release (build ID: 20140716183446) and Firefox 32 beta 1 (build ID: 20140722195627).