Defect - Find in page throws

VERIFIED FIXED in Firefox 24

Status

Firefox for Metro
App Bar
P2
normal
VERIFIED FIXED
5 years ago
4 years ago

People

(Reporter: rsilveira, Assigned: rsilveira)

Tracking

Trunk
Firefox 24
x86_64
Windows 8.1
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [shovel-ready] feature=defect c=Find_in_page_app_bar u=metro_firefox_user p=2)

Attachments

(1 attachment)

Mostly harmless. I'm seeing this 2 exceptions:
 
Error: this._fastFind.currentWindow is null
Source File: chrome://browser/content/contenthandlers/FindHandler.js
Line: 43

Error: ReferenceError: kBrowserFormZoomLevelMin is not defined
Source File: chrome://browser/content/helperui/FindHelperUI.js
Line: 186

kBrowserFormZoomLevelMin is actually defined in *Form*HelperUI.js but it's not used there. Moving it to FindHelperUI makes find highlighting and page repositioning stop working. Didn't investigate further. We need to make sure the zoom logic is solid at
https://mxr.mozilla.org/mozilla-central/source/browser/metro/base/content/helperui/FindHelperUI.js#166

Updated

5 years ago
Blocks: 859003
Summary: Find in page throws → Defect - Find in page throws
Whiteboard: [shovel-ready] → [shovel-ready] feature=defect c=Find_in_page_app_bar u=metro_firefox_user p=0

Updated

5 years ago
Priority: -- → P2
(Assignee)

Updated

5 years ago
Assignee: nobody → rsilveira
Status: NEW → ASSIGNED
Whiteboard: [shovel-ready] feature=defect c=Find_in_page_app_bar u=metro_firefox_user p=0 → [shovel-ready] feature=defect c=Find_in_page_app_bar u=metro_firefox_user p=2

Updated

5 years ago
Blocks: 875024
No longer blocks: 859003
QA Contact: jbecerra
Created attachment 759457 [details] [diff] [review]
Patch v1

this._fastFind.currentWindow was null when opening the findbar for the first time with an empty string.

The zoom code in FindHelperUI doesn't work because we don't have apzc yet, so it ends up messing with the scroll to the found text. I've disabled it by changing the constants so it's easy to re-enable once we get zoom working.

I'll create a bug to track the re-enabling of the find zoom before landing this.
Attachment #759457 - Flags: review?(mbrubeck)
Attachment #759457 - Flags: review?(mbrubeck) → review+
https://hg.mozilla.org/mozilla-central/rev/20c1ebbc31cf
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 24
Hi Rodrigo,

Is there any way for me to test this? Is there a specific action or website that spawned those error messages? Thanks!
Flags: needinfo?(rsilveira)
Hi Kamil,

You just need to use find in any page and you'd see those exception on the js console (CTRL + SHIFT + J). If you don't see those exceptions, it's fixed.
Flags: needinfo?(rsilveira)
Thanks Rodrigo!

Went through the following "Defect" for iteration #9 testing without any issues. Used the following build:

http://ftp.mozilla.org/pub/mozilla.org/firefox/nightly/2013-07-11-03-02-04-mozilla-central/

- Went through the steps provided in comment 5 and looked for any errors listed in comment 0
- Visited several websites and tapped "Up" & "Down" under the "Find" bar, checked and ensured none of the errors in comment 0 appeared after a few minutes
- Visited several websites and pressed "Enter" under the "Find" bar, checked and ensured none of the errors in comment 0 appeared after a few minutes
- Went through all of the above test cases in "Filled" view without any issues
Went through the following "Defect" for iteration #10 testing without any issues. Used the following build:

http://ftp.mozilla.org/pub/mozilla.org/firefox/nightly/2013-07-19-03-02-04-mozilla-central/

- Went through the steps provided in comment 5 and looked for any errors listed in comment 0
- Went through the test cases added in comment 6 without any issues
User Agent: Mozilla/5.0 (Windows NT 6.2; WOW64; rv:26.0) Gecko/20100101 Firefox/26.0
Build ID: 20130816030205
Built from http://hg.mozilla.org/mozilla-central/rev/1ed5a88cd4d0

WFM
Tested on windows 8 using latest nightly for iteration-12. Followed steps provided in comment5 and got expected result.
Status: RESOLVED → VERIFIED
Went through the following defect for iteration #20 testing without any issues. Used the following build:
http://ftp.mozilla.org/pub/mozilla.org/firefox/nightly/2013-12-22-03-02-04-mozilla-central/

- Went through the steps provided in comment 5 and looked for any errors listed in comment #0
- Used the up/down arrows under the Find App Bar and ensured the errors listed in comment #0 didn't appear
- Used "Enter" under the Find App Bar and ensured that the errors listed in comment #0 didn't appear
- Used "Shift + Enter" under the Find App Bar and ensured that the errors listed in comment #0 didn't appear
- Used the "Search" on the OSK while under the Find App Bar and ensured that the errors listed in comment #0 didn't apppear
- Went through the above test cases in filled view without any issues
OS: Windows 8 Metro → Windows 8.1
You need to log in before you can comment on or make changes to this bug.