1173703 - [Search][Rocketbar] Get lots of 'Browser' errors when having input at search app

Status: RESOLVED FIXED

(Firefox OS Graveyard :: Gaia::Search, defect)

Description

[Hermes Cheng (inactive after July 27, 2015)]

After below STR, get below errors
* STR:
1. tap rocketbar to enter search app
2. input something

* Errors:
W/Browser ( 1419): [JavaScript Warning: "Property contained reference to invalid variable.  Error in parsing value for 'transition-delay'.  Falling back to 'initial'." {file: "app://search.gaiamobile.org/shared/elements/gaia_grid/style.css" line: 143 column: 3124 source: " transform var(--transition-duration),"}]
W/Browser ( 1419): [JavaScript Warning: "Property contained reference to invalid variable.  Error in parsing value for 'transition-duration'.  Falling back to 'initial'." {file: "app://search.gaiamobile.org/shared/elements/gaia_grid/style.css" line: 143 column: 3124 source: " transform var(--transition-duration),"}]
W/Browser ( 1419): [JavaScript Warning: "Property contained reference to invalid variable.  Error in parsing value for 'transition-property'.  Falling back to 'initial'." {file: "app://search.gaiamobile.org/shared/elements/gaia_grid/style.css" line: 143 column: 3124 source: " transform var(--transition-duration),"}]
W/Browser ( 1419): [JavaScript Warning: "Property contained reference to invalid variable.  Error in parsing value for 'transition-timing-function'.  Falling back to 'initial'." {file: "app://search.gaiamobile.org/shared/elements/gaia_grid/style.css" line: 143 column: 3124 source: " transform var(--transition-duration),"}]

Comment 1

[Jayme Mercado [:JMercado] (Inactive 3/4/2016)]

The changes for Bug 1143226 seem to have caused this issue.

B2g-inbound Regression Window

Last Working 
Environmental Variables:
Device: Flame 3.0
BuildID: 20150501103336
Gaia: fd40a1f7911ac989dbca8e89de679e974de4ff41
Gecko: f7dcddd57503
Version: 40.0a1 (3.0) 
Firmware Version: v18D-1
User Agent: Mozilla/5.0 (Mobile; rv:40.0) Gecko/40.0 Firefox/40.0

First Broken 
Environmental Variables:
Device: Flame 3.0
BuildID: 20150501105332
Gaia: 07a1a20b86931ee9911b16df94df60a178825bb2
Gecko: e99d30fb8fdc
Version: 40.0a1 (3.0) 
Firmware Version: v18D-1
User Agent: Mozilla/5.0 (Mobile; rv:40.0) Gecko/40.0 Firefox/40.0

Last Working gaia / First Broken gecko - Issue does NOT occur
Gaia: fd40a1f7911ac989dbca8e89de679e974de4ff41
Gecko: e99d30fb8fdc

First Broken gaia / Last Working gecko - Issue DOES occur
Gaia: 07a1a20b86931ee9911b16df94df60a178825bb2
Gecko: f7dcddd57503

Gaia Pushlog: https://github.com/mozilla-b2g/gaia/compare/fd40a1f7911ac989dbca8e89de679e974de4ff41...07a1a20b86931ee9911b16df94df60a178825bb2

Comment 2

[KTucker [:KTucker][Inactive 3/4/2016]]

Dietrich, can you take a look at this please? This might have been caused by the landing for bug 1143226.

Comment 3

[GitHub Autolander Bot]

Attachment: [gaia] autonome:1173703-rocketbar-errors > mozilla-b2g:master

Comment 4

[Dietrich Ayala (:dietrich)]

Sorry for delay, was OOO for the last month.

Comment 5

[Kevin Grandon :kgrandon]

Comment on attachment 8635736 [details] [review]
[gaia] autonome:1173703-rocketbar-errors > mozilla-b2g:master

Looks good. Do we need to include the stylesheet in search/index.html like so?

 <link rel="stylesheet" type="text/css" href="app://theme.gaiamobile.org/shared/elements/gaia-theme/gaia-theme.css">

Comment 6

[Dietrich Ayala (:dietrich)]

Why do we need both that and the import? Is that how the new architecture is doing it?

Comment 7

[Dietrich Ayala (:dietrich)]

You gave the r+ and the patch fixed the problem as-is, so I merged and am closing this. If there's a follow-up change, let me know and i'll fix that separately.

Comment 8

[Dietrich Ayala (:dietrich)]

Oops, never marked fixed!