Closed
Bug 1280601
Opened 8 years ago
Closed 8 years ago
Add lastKnownTitle to onVisit
Categories
(Toolkit :: Places, defect, P3)
Toolkit
Places
Tracking
()
RESOLVED
FIXED
People
(Reporter: bsilverberg, Unassigned)
References
Details
As discussed in bug 1265845, in order to mimic Chrome's onVisited API event, we should deliver the last known title for a page in onVisited. We can do this if the title is provided to us via the onVisit event.
Perhaps we should call it `lastKnownTitle` in onVisit to be clear that it is not the title from the current visit, but rather the last known title of the page. If there is no known title it would return an empty string.
Reporter | ||
Updated•8 years ago
|
Assignee: nobody → kmaglione+bmo
Status: NEW → ASSIGNED
Comment 1•8 years ago
|
||
+1 on the API naming.
Reporter | ||
Comment 2•8 years ago
|
||
Thanks for taking this, Kris. I'd really like to land bug 1280357 before the end of the month. Do you think you'll be able to land this change before then?
Reporter | ||
Updated•8 years ago
|
Flags: needinfo?(kmaglione+bmo)
Comment 3•8 years ago
|
||
Is there a particular reason this needs to be done before the end of the month? The next merge isn't until August, and Andy has told me that I shouldn't treat this API as a particularly high priority for the moment. I was planning to take care of it when I'm back in the states at the end of next week.
Flags: needinfo?(kmaglione+bmo)
Reporter | ||
Comment 4•8 years ago
|
||
The only reason is that this is part of my Q2 deliverable, which was to complete the implementation of the history API, but I think we can consider it basically complete with only a couple of minor enhancement bugs still open, so don't worry about it. I didn't realize you were still traveling and it's fine to leave it until you get back.
Updated•8 years ago
|
Priority: -- → P3
Comment 5•8 years ago
|
||
Fixed by bug 1341097
Assignee: kmaglione+bmo → nobody
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•