Closed Bug 1464414 Opened 3 years ago Closed 3 years ago

Remove gURLBar null-check from XULBrowserWindow.setOverLink

Categories

(Firefox :: Address Bar, enhancement, P5)

Product:
Firefox
Component:
Address Bar
Type:
enhancement

Tracking

()

Status:
RESOLVED FIXED
Milestone:
Firefox 62
Tracking Flags:
Tracking Status
firefox62 --- fixed

People

(Reporter: dao, Assigned: lucas05, Mentored)

References

Details

(Keywords: good-first-bug, Whiteboard: [lang=js])

Attachments

(1 file, 1 obsolete file)

"gURLBar &&" only removed from If statement
525 bytes, patch
dao
: review+
Details | Diff | Splinter Review 
+++ This bug was initially created as a clone of Bug #1464384 +++

This goes back to when the address bar could be removed from the toolbar. The address bar cannot be removed anymore, so gURLBar is always non-null in browsers windows and this null-check is now pointless:

https://searchfox.org/mozilla-central/rev/bf4def01bf8f6ff0d18f02f2d7e9efc73e12c63f/browser/base/content/browser.js#4628
Priority: -- → P5
Can I work on this? :)
(In reply to lucas05 from comment #1)
> Can I work on this? :)

Sure!
Assignee: nobody → lucas05
Are you still working on this?
If need any help, write here or ask in IRC.

(In reply to lucas05 from comment #1)
> Can I work on this? :)
Flags: needinfo?(lucas05)
(In reply to Manish Kumar [:manishkk] from comment #3)
> Are you still working on this?
> If need any help, write here or ask in IRC.
> 
> (In reply to lucas05 from comment #1)
> > Can I work on this? :)

Yes. I know is an easy one :) I had little time, sorry. Either today or tomorrow I'll patch it!
Flags: needinfo?(lucas05)
Comment on attachment 8985800 [details] [diff] [review]
Bug 1464414 - Remove gURLBar null-check from XULBrowserWindow.setOverLink

>-      if (gURLBar && gURLBar._mayTrimURLs /* corresponds to browser.urlbar.trimURLs */)
>-        url = trimURL(url);
>+      url = trimURL(url);

We still need gURLBar._mayTrimURLs /* corresponds to browser.urlbar.trimURLs */. Only the "gURLBar &&" part can go.
Attachment #8985800 - Flags: review-
Here it is! I've fixed the patch :)
Comment on attachment 8986687 [details] [diff] [review]
"gURLBar &&" only removed from If statement

Looks good, thanks!
Attachment #8986687 - Flags: review+
Attachment #8985800 - Attachment is obsolete: true
Pushed by dgottwald@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/4b18075d2be2
Remove gURLBar null-check from XULBrowserWindow.setOverLink. r=dao
https://hg.mozilla.org/mozilla-central/rev/4b18075d2be2
Status: NEW → RESOLVED
Closed: 3 years ago
status-firefox62: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 62
You need to log in before you can comment on or make changes to this bug.