Closed Bug 739219 Opened 13 years ago Closed 7 years ago

Remove nsIBrowserHistory

Categories

(Toolkit :: Places, defect, P2)

Product:
Toolkit
Component:
Places
Type:
defect

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla62
Tracking Flags:
Tracking Status
firefox62 --- fixed

People

(Reporter: mak, Assigned: mak)

References

Details

(Keywords: perf, Whiteboard: [snappy:P1] )

Attachments

(1 file)

Bug 739219 - Remove nsIBrowserHistory.
(deleted), text/x-review-board-request
standard8
: review+
Details
Once we have removed most synchronous stuff from here, the only remaning bits will be RemovePageXXX methods and MarkAsXXX methods, we should just move the MarkAsXXX stuff to nsINavHistoryService and provide fully-async remove methods in mozIAsyncHistory, and remove this quite pointless interface.
Depends on: 739221
Whiteboard: [snappy]
Whiteboard: [snappy] → [snappy:P1]
Depends on: 751608
Depends on: 834545
Depends on: 839034
Depends on: 854886
Blocks: fxdesktopbacklog
Whiteboard: [snappy:P1] → [snappy:P1] p=0
Removed from Backlog based on team feedback from the point estimation meeting.
No longer blocks: fxdesktopbacklog
Flags: firefox-backlog-
Whiteboard: [snappy:P1] p=0 → [snappy:P1]
Blocks: 854886
No longer depends on: 854886
Depends on: 1076775
No longer blocks: 854886
Depends on: 854886
Depends on: 1087255
Depends on: 1113175
Depends on: 1124185
Depends on: 871908, 1089691, 1089695
Priority: -- → P1
Keywords: perf
Depends on: 1220654
Depends on: 1249259
Depends on: 1086549
Depends on: 1249263
Depends on: 1254923
No longer depends on: 871908
Depends on: 1261313
Depends on: 1306276
Depends on: 1364337
Priority: P1 → P5
Summary: Kill nsIBrowserHistory → [meta] Kill nsIBrowserHistory
Priority: P5 → P3
Summary: [meta] Kill nsIBrowserHistory → Remove nsIBrowserHistory
Assignee: nobody → mak77
Status: NEW → ASSIGNED
Priority: P3 → P2
Comment on attachment 8973790 [details] Bug 739219 - Remove nsIBrowserHistory. https://reviewboard.mozilla.org/r/242164/#review248498
Attachment #8973790 - Flags: review?(standard8) → review+
Pushed by mak77@bonardo.net: https://hg.mozilla.org/integration/autoland/rev/6c4bad08fa9a Remove nsIBrowserHistory. r=standard8
https://hg.mozilla.org/mozilla-central/rev/6c4bad08fa9a
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
status-firefox62: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla62
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: