This table lists various inline task markers (TODO, FIXME, and XXX). These were generated from sb-trunk-oldxul on 2014-05-14. See also http://developer.getnightingale.com/dd/da0/todo.html ^ Description ^ Resource ^ Path ^ Location ^ Type ^ | XXXeps has a few differences that should be accounted for. | wizard.xml | /sb-trunk-oldxul/app/content/bindings | line 66 | XML Task | | XXXMook: need to be smarter and only rebuild things that have changed | propertiesMenuList.xml | /sb-trunk-oldxul/app/content/bindings | line 352 | XML Task | | XXXeps have time to do the wizard specific work this late into Fugazi. | wizard.xml | /sb-trunk-oldxul/app/content/bindings | line 64 | XML Task | | TODO: Change the use of these functions so we can remove the | servicePane.xml | /sb-trunk-oldxul/app/content/bindings | line 1473 | XML Task | | XXXeps wizards have a header instead of a titlebar and the wizard header | wizard.xml | /sb-trunk-oldxul/app/content/bindings | line 65 | XML Task | | TODO | deviceControl.xml | /sb-trunk-oldxul/app/content/bindings/device | line 98 | XML Task | | XXXeps Should use sb-sys-outer-frame and sb-sys-wizard-titlebar but don't | wizard.xml | /sb-trunk-oldxul/app/content/bindings | line 63 | XML Task | | XXXeps TODO | deviceControl.xml | /sb-trunk-oldxul/app/content/bindings/device | line 98 | XML Task | | TODO: allow external menu? | sysControls.xml | /sb-trunk-oldxul/app/content/bindings | line 397 | XML Task | | XXXeps this can be removed since click events now use | playlist.xml | /sb-trunk-oldxul/app/content/bindings | line 384 | XML Task | | XXXben This is here to fix a bug where 'this.tree.view' will return | playlist.xml | /sb-trunk-oldxul/app/content/bindings | line 272 | XML Task | | XXXsspitzer hack | globalBindings.xml | /sb-trunk-oldxul/app/skin/global | line 66 | XML Task | | TODO: Redundant, but needed for the remote API. | sbTabBrowserTab.xml | /sb-trunk-oldxul/app/content/bindings/browser | line 325 | XML Task | | XXXsspitzer hack | globalBindings.xml | /sb-trunk-oldxul/app/skin/global | line 57 | XML Task | | XXXsspitzer hack | globalBindings.xml | /sb-trunk-oldxul/app/skin/global | line 48 | XML Task | | TODO removed buggy/deadlocky code. This is used by the code for | sbDeviceLibrarySyncSettings.cpp | /sb-trunk-oldxul/components/devices/base/src | line 242 | C/C++ Task | | TODO a bit of a hack, we're assuming all lists are from the same | sbLocalDatabaseDiffingService.cpp | /sb-trunk-oldxul/components/library/localdatabase/src | line 1476 | C/C++ Task | | XXX a bit of a hack, we're assuming all lists are from the same | sbLocalDatabaseDiffingService.cpp | /sb-trunk-oldxul/components/library/localdatabase/src | line 1476 | C/C++ Task | | XXX Mook: USE_JSSTUB_FOR_ADDPROPERTY is needed to define things on the | sbScriptableFilterResult.cpp | /sb-trunk-oldxul/components/remoteapi/src | line 172 | C/C++ Task | | XXX Mook: this will fail at times near midnight. Watch me not care ;) | debugtest.cpp | /sb-trunk-oldxul/tools/disthelper/tests | line 81 | C/C++ Task | | TODO This and the other methods should use a reference | sbTArrayStringEnumerator.h | /sb-trunk-oldxul/components/moz/strings/src | line 51 | C/C++ Task | | XXX This and the other methods should use a reference | sbTArrayStringEnumerator.h | /sb-trunk-oldxul/components/moz/strings/src | line 51 | C/C++ Task | | TODO Set accurately? | sbGStreamerRTPStreamer.cpp | /sb-trunk-oldxul/components/mediacore/gstreamer/src | line 206 | C/C++ Task | | TODO Report the error more usefully using the GError | sbGStreamerRTPStreamer.cpp | /sb-trunk-oldxul/components/mediacore/gstreamer/src | line 147 | C/C++ Task | | TODO Add a 'range' attribute with the correct info about the media | sbGStreamerRTPStreamer.cpp | /sb-trunk-oldxul/components/mediacore/gstreamer/src | line 212 | C/C++ Task | | XXX Not sure how this is different from SetStatus | sbiTunesImporterJob.h | /sb-trunk-oldxul/components/mediaimport/library/itunes/src | line 75 | C/C++ Task | | XXX Mook: force no recursion for now; preed thinks it's likely to be dangerous | commands.cpp | /sb-trunk-oldxul/tools/disthelper/windows | line 151 | C/C++ Task | | TODO Figure out what format we want to use, or make configurable. | sbGStreamerRTPStreamer.cpp | /sb-trunk-oldxul/components/mediacore/gstreamer/src | line 127 | C/C++ Task | | TODO Not sure how this is different from SetStatus | sbiTunesImporterJob.h | /sb-trunk-oldxul/components/mediaimport/library/itunes/src | line 75 | C/C++ Task | | XXX Mook temp hack | sbTestHarnessConsoleListener.cpp | /sb-trunk-oldxul/components/testharness/src | line 51 | C/C++ Task | | TODO Add a writable temporaryFileFactory attribute to use to create the | sbIMediaItemDownloadJob.h | /sb-trunk-oldxul/compiled/components/library/download/public | line 47 | C/C++ Task | | TODO See Bug 24191 to see what's to come and feel free to comment. | sbLibraryListenerHelpers.cpp | /sb-trunk-oldxul/components/devices/device/src | line 461 | C/C++ Task | | XXX See Bug 24191 to see what's to come and feel free to comment. | sbLibraryListenerHelpers.cpp | /sb-trunk-oldxul/components/devices/device/src | line 461 | C/C++ Task | | TODO can i make this faster? | sbLocalDatabaseLibrary.cpp | /sb-trunk-oldxul/components/library/localdatabase/src | line 1815 | C/C++ Task | | TODO Add a comment here please. | sbiTunesAgentProcessor.h | /sb-trunk-oldxul/tools/itunesagent | line 207 | C/C++ Task | | FIXME This is wrong for several reasons. First, the mQueryCount we get is not meaningful anymore | sbLocalDatabaseLibrary.cpp | /sb-trunk-oldxul/components/library/localdatabase/src | line 4762 | C/C++ Task | | TODO if the user moves the cursor, it should become visible. | sbGStreamerPlatformGDK.cpp | /sb-trunk-oldxul/components/mediacore/gstreamer/src | line 145 | C/C++ Task | | TODO This is a temporary bit of "UI" to get out of fullscreen mode. | sbGStreamerPlatformGDK.cpp | /sb-trunk-oldxul/components/mediacore/gstreamer/src | line 62 | C/C++ Task | | TODO Write this documentation | sbIRemoteWebPlaylist.h | /sb-trunk-oldxul/compiled/components/remoteapi/public | line 35 | C/C++ Task | | TODO Currently this only handles the video windows, we'll probably need | sbIGstPlatformInterface.h | /sb-trunk-oldxul/components/mediacore/gstreamer/src | line 39 | C/C++ Task | | TODO The tooltip functionality is disabled because mozilla will crash if | sbIPlaylistCommands.h | /sb-trunk-oldxul/compiled/components/playlistcommands/public | line 278 | C/C++ Task | | XXX Songbird: enable F4 on mac too | search.xml | /sb-trunk-oldxul/compiled/dist/chrome/browser/content/browser/search | line 722 | XML Task | | XXX HACK UGH | sbLocalDatabaseQuery.cpp | /sb-trunk-oldxul/components/library/localdatabase/src | line 664 | C/C++ Task | | XXX This is very bad according to edward! But we need it until | sbGStreamerService.cpp | /sb-trunk-oldxul/components/mediacore/gstreamer/src | line 394 | C/C++ Task | | TODO Detail>. | sbIDownloadDevice.h | /sb-trunk-oldxul/compiled/components/devicesobsolete/download/public | line 42 | C/C++ Task | | XXX Now that we are using sqlite FTS, we need to live with its | sbLocalDatabaseQuery.cpp | /sb-trunk-oldxul/components/library/localdatabase/src | line 773 | C/C++ Task | | XXX Alfred: Update the function whenever any content type other than | sbLocalDatabaseSmartMediaList.cpp | /sb-trunk-oldxul/components/library/localdatabase/src | line 608 | C/C++ Task | | XXX HACK | sbLocalDatabaseQuery.cpp | /sb-trunk-oldxul/components/library/localdatabase/src | line 664 | C/C++ Task | | XXX HACK | sbLocalDatabaseQuery.cpp | /sb-trunk-oldxul/components/library/localdatabase/src | line 664 | C/C++ Task | | XXX: for 1.9, change min from 5 to 59, to select the quality Acme | mp4-mjpeg-adpcm.xml | /sb-trunk-oldxul/compiled/dist/gstreamer/encode-profiles | line 33 | XML Task | | XXX Mook: hack until I figure out what to do with missing images | nsSystemTrayServiceGTK.cpp | /sb-trunk-oldxul/extensions/systray/components/src | line 73 | C/C++ Task | | XXX Mook: hack until I figure out what to do with missing images | nsSystemTrayServiceGTK.cpp | /sb-trunk-oldxul/extensions/systray/components/src | line 106 | C/C++ Task | | TODO: split play/pause into two | mainwin.dtd | /sb-trunk-oldxul/extensions/systray/chrome/locale/en-US | line 1 | DTD Task | | TODO Can we add any more checks in here to be sure it's ACTUALLY raw | sbGStreamerMediaInspector.cpp | /sb-trunk-oldxul/components/mediacore/gstreamer/src | line 1282 | C/C++ Task | | TODO There's no need to check the codec type before extracting | sbGStreamerMediaInspector.cpp | /sb-trunk-oldxul/components/mediacore/gstreamer/src | line 1208 | C/C++ Task | | TODO Additional properties?? | sbGStreamerMediaInspector.cpp | /sb-trunk-oldxul/components/mediacore/gstreamer/src | line 1197 | C/C++ Task | | TODO Additional property: profile. | sbGStreamerMediaInspector.cpp | /sb-trunk-oldxul/components/mediacore/gstreamer/src | line 1178 | C/C++ Task | | TODO Add additional properties for raw video? | sbGStreamerMediaInspector.cpp | /sb-trunk-oldxul/components/mediacore/gstreamer/src | line 1131 | C/C++ Task | | TODO Can we add any more checks in here to be sure it's ACTUALLY raw | sbGStreamerMediaInspector.cpp | /sb-trunk-oldxul/components/mediacore/gstreamer/src | line 1126 | C/C++ Task | | TODO Additional property: profile, level. | sbGStreamerMediaInspector.cpp | /sb-trunk-oldxul/components/mediacore/gstreamer/src | line 1065 | C/C++ Task | | TODO Additional property: profile ID? | sbGStreamerMediaInspector.cpp | /sb-trunk-oldxul/components/mediacore/gstreamer/src | line 1048 | C/C++ Task | | TODO Additional properties for other container formats. | sbGStreamerMediaInspector.cpp | /sb-trunk-oldxul/components/mediacore/gstreamer/src | line 1001 | C/C++ Task | | TODO Connect up autoplug-sort signal to handle some special cases | sbGStreamerMediaInspector.cpp | /sb-trunk-oldxul/components/mediacore/gstreamer/src | line 627 | C/C++ Task | | TODO Signal failure somehow with more info? | sbGStreamerMediaInspector.cpp | /sb-trunk-oldxul/components/mediacore/gstreamer/src | line 618 | C/C++ Task | | XXX Mook: OOM | nsSystemTrayServiceGTK.cpp | /sb-trunk-oldxul/extensions/systray/components/src | line 133 | C/C++ Task | | TODO Report the error more usefully using the GError, and perhaps | sbGStreamerTranscode.cpp | /sb-trunk-oldxul/components/mediacore/gstreamer/src | line 261 | C/C++ Task | | XXX Mook: USE_JSSTUB_FOR_ADDPROPERTY is needed to define things on the | sbScriptableFilter.cpp | /sb-trunk-oldxul/components/remoteapi/src | line 194 | C/C++ Task | | XXX Extra copy, can be removed if we're sure tempCString can | sbVariant.cpp | /sb-trunk-oldxul/components/moz/variant/src | line 1105 | C/C++ Task | | TODO This can cause problems if this fires when the user is dragging | sbWatchFolder.cpp | /sb-trunk-oldxul/components/watchfolder/src | line 372 | C/C++ Task | | XXX This is doing 1 extra copy. Need to fix this | sbVariant.cpp | /sb-trunk-oldxul/components/moz/variant/src | line 1150 | C/C++ Task | | XXX This can cause problems if this fires when the user is dragging | sbWatchFolder.cpp | /sb-trunk-oldxul/components/watchfolder/src | line 372 | C/C++ Task | | XXX perhaps we'd like to add support for converting each of the various | sbVariant.cpp | /sb-trunk-oldxul/components/moz/variant/src | line 1334 | C/C++ Task | | XXX This iid handling is ugly! | sbVariant.cpp | /sb-trunk-oldxul/components/moz/variant/src | line 1478 | C/C++ Task | | XXX This is an extra copy that should be avoided | sbVariant.cpp | /sb-trunk-oldxul/components/moz/variant/src | line 1022 | C/C++ Task | | XXX Extra copy, can be removed if we're sure CSTRING can | sbVariant.cpp | /sb-trunk-oldxul/components/moz/variant/src | line 1064 | C/C++ Task | | XXX Extra copy, can be removed if we're sure CHAR_STR can | sbVariant.cpp | /sb-trunk-oldxul/components/moz/variant/src | line 1073 | C/C++ Task | | XXX Extra copy, can be removed if we're sure CHAR_STR can | sbVariant.cpp | /sb-trunk-oldxul/components/moz/variant/src | line 1082 | C/C++ Task | | TODO There probably is some size associated with storing a list at | sbBaseDevice.cpp | /sb-trunk-oldxul/components/devices/device/src | line 4585 | C/C++ Task | | XXX Is this really what we want to do? | sbVariant.cpp | /sb-trunk-oldxul/components/moz/variant/src | line 736 | C/C++ Task | | XXX MOOK this needs to be fixed to be not gstreamer specific | sbBaseDevice.cpp | /sb-trunk-oldxul/components/devices/device/src | line 5802 | C/C++ Task | | XXX should check for data loss here! | sbVariant.cpp | /sb-trunk-oldxul/components/moz/variant/src | line 773 | C/C++ Task | | XXX We might want stringified versions of these... ??? | sbVariant.cpp | /sb-trunk-oldxul/components/moz/variant/src | line 881 | C/C++ Task | | TODO Add a writable temporaryFileFactory attribute to | sbBaseDevice.cpp | /sb-trunk-oldxul/components/devices/device/src | line 807 | C/C++ Task | | TODO Some sort of API like this will be needed once we're using this | sbGStreamerVideoTranscode.cpp | /sb-trunk-oldxul/components/mediacore/gstreamer/src | line 985 | C/C++ Task | | XXX Would we rather print "true" / "false" ? | sbVariant.cpp | /sb-trunk-oldxul/components/moz/variant/src | line 948 | C/C++ Task | | XXX We ASSUME that "array of nsID" means "array of pointers to nsID". | sbVariant.cpp | /sb-trunk-oldxul/components/moz/variant/src | line 478 | C/C++ Task | | TODO Check the glib return values. | sbLinuxFileSystemWatcher.cpp | /sb-trunk-oldxul/components/filesystemevents/linux | line 269 | C/C++ Task | | XXX toFloat convertions need to be fixed! | sbVariant.cpp | /sb-trunk-oldxul/components/moz/variant/src | line 694 | C/C++ Task | | XXX toChar convertions need to be fixed! | sbVariant.cpp | /sb-trunk-oldxul/components/moz/variant/src | line 707 | C/C++ Task | | XXX toWChar convertions need to be fixed! | sbVariant.cpp | /sb-trunk-oldxul/components/moz/variant/src | line 714 | C/C++ Task | | TODO should we describe the sub-type of raw audio (i.e. integer, float, | sbGStreamerVideoTranscode.cpp | /sb-trunk-oldxul/components/mediacore/gstreamer/src | line 1891 | C/C++ Task | | XXX Should we check !isnan(fval) as well? Is it portable? We | sbVariant.cpp | /sb-trunk-oldxul/components/moz/variant/src | line 77 | C/C++ Task | | XXX Need boundary checking here. | sbVariant.cpp | /sb-trunk-oldxul/components/moz/variant/src | line 208 | C/C++ Task | | XXX We ASSUME that "array of nsID" means "array of pointers to nsID". | sbVariant.cpp | /sb-trunk-oldxul/components/moz/variant/src | line 318 | C/C++ Task | | XXX We ASSUME that "array of nsID" means "array of pointers to nsID". | sbVariant.cpp | /sb-trunk-oldxul/components/moz/variant/src | line 410 | C/C++ Task | | TODO Some sort of API like this will be needed once we're using this | sbGStreamerVideoTranscode.cpp | /sb-trunk-oldxul/components/mediacore/gstreamer/src | line 1003 | C/C++ Task | | TODO check if any of these variables might plausibly get large enough | sbGStreamerVideoTranscode.cpp | /sb-trunk-oldxul/components/mediacore/gstreamer/src | line 1064 | C/C++ Task | | TODO Should we also permit video/x-raw-rgb? It doesn't matter for now, | sbGStreamerVideoTranscode.cpp | /sb-trunk-oldxul/components/mediacore/gstreamer/src | line 1226 | C/C++ Task | | TODO should we describe the sub-type of raw video (i.e. YUV 4:2:0 or | sbGStreamerVideoTranscode.cpp | /sb-trunk-oldxul/components/mediacore/gstreamer/src | line 1859 | C/C++ Task | | XXX (darin): if this is the right thing, then why wasn't it fixed in NSPR?!? | sbVariant.cpp | /sb-trunk-oldxul/components/moz/variant/src | line 50 | C/C++ Task | | TODO Is this appropriate for a non-fatal failure to seek? Should we | sbGStreamerMediacore.cpp | /sb-trunk-oldxul/components/mediacore/gstreamer/src | line 1987 | C/C++ Task | | XXX not implemented by nsTreeSelection either | sbPlaylistTreeSelection.cpp | /sb-trunk-oldxul/components/library/localdatabase/src | line 180 | C/C++ Task | | TODO This is currently not localised (but we're probably not setting | sbGStreamerMediacore.cpp | /sb-trunk-oldxul/components/mediacore/gstreamer/src | line 1399 | C/C++ Task | | TODO This is currently not localised (but we're probably not setting | sbGStreamerMediacore.cpp | /sb-trunk-oldxul/components/mediacore/gstreamer/src | line 1433 | C/C++ Task | | TODO This is currently not localised (but we're probably not setting | sbGStreamerMediacore.cpp | /sb-trunk-oldxul/components/mediacore/gstreamer/src | line 1164 | C/C++ Task | | TODO clean up | sbMetadataCrashTracker.cpp | /sb-trunk-oldxul/components/mediacore/metadata/manager/src | line 33 | C/C++ Task | | XXXsspitzer hack | globalBindings.xml | /sb-trunk-oldxul/compiled/dist/chrome/songbird/skin/global | line 48 | XML Task | | TODO ! | sbTranscodeAlbumArt.cpp | /sb-trunk-oldxul/components/mediacore/transcode/src | line 401 | C/C++ Task | | TODO what if we can't write to that (e.g. MTP)? | sbTranscodeAlbumArt.cpp | /sb-trunk-oldxul/components/mediacore/transcode/src | line 616 | C/C++ Task | | TODO: Change the use of these functions so we can remove the | servicePane.xml | /sb-trunk-oldxul/compiled/dist/chrome/songbird/content/songbird/bindings | line 1462 | XML Task | | TODO give useful arguments here | nsSystemTrayIconGTK.cpp | /sb-trunk-oldxul/extensions/systray/components/src | line 191 | C/C++ Task | | TODO Insert the comparison for additional properties here | sbDeviceCapsCompatibility.cpp | /sb-trunk-oldxul/components/devices/base/src | line 568 | C/C++ Task | | XXXsspitzer hack | globalBindings.xml | /sb-trunk-oldxul/compiled/dist/chrome/songbird/skin/global | line 66 | XML Task | | XXXsspitzer hack | globalBindings.xml | /sb-trunk-oldxul/compiled/dist/chrome/songbird/skin/global | line 57 | XML Task | | TODO Get additional audio properties | sbDeviceCapsCompatibility.cpp | /sb-trunk-oldxul/components/devices/base/src | line 371 | C/C++ Task | | TODO Do some checks to make sure we successfully copied the file. | sbMediaFileManager.cpp | /sb-trunk-oldxul/components/mediamanager/src | line 1390 | C/C++ Task | | TODO Insert the comparison for additional properties here | sbDeviceCapsCompatibility.cpp | /sb-trunk-oldxul/components/devices/base/src | line 518 | C/C++ Task | | TODO Do some checks to make sure we successfully moved the file. | sbMediaFileManager.cpp | /sb-trunk-oldxul/components/mediamanager/src | line 1400 | C/C++ Task | | TODO Get additional audio properties | sbDeviceCapsCompatibility.cpp | /sb-trunk-oldxul/components/devices/base/src | line 190 | C/C++ Task | | XXX Remove Me !!! | WindowMinMax.h | /sb-trunk-oldxul/components/integration/src/win32 | line 37 | C/C++ Task | | TODO Get additional Video properties | sbDeviceCapsCompatibility.cpp | /sb-trunk-oldxul/components/devices/base/src | line 351 | C/C++ Task | | TODO Check for valid properties | sbLocalDatabaseGUIDArray.cpp | /sb-trunk-oldxul/components/library/localdatabase/src | line 394 | C/C++ Task | | XXX Disable memory sorting in the general case, since it appears to slow things | sbLocalDatabaseGUIDArray.cpp | /sb-trunk-oldxul/components/library/localdatabase/src | line 2105 | C/C++ Task | | TODO: Redundant, but needed for the remote API. | sbTabBrowserTab.xml | /sb-trunk-oldxul/compiled/dist/chrome/songbird/content/songbird/bindings/browser | line 314 | XML Task | | TODO we need to add the notification type to the event eventually | sbSecurityMixin.cpp | /sb-trunk-oldxul/components/remoteapi/src | line 635 | C/C++ Task | | TODO >. | sbIDeviceBase.h | /sb-trunk-oldxul/compiled/components/devicesobsolete/base/public | line 85 | C/C++ Task | | TODO >. | sbIDeviceBase.h | /sb-trunk-oldxul/compiled/components/devicesobsolete/base/public | line 84 | C/C++ Task | | TODO Detail>. | sbIDeviceBase.h | /sb-trunk-oldxul/compiled/components/devicesobsolete/base/public | line 82 | C/C++ Task | | TODO | deviceControl.xml | /sb-trunk-oldxul/compiled/dist/chrome/songbird/content/songbird/bindings/device | line 98 | XML Task | | TODO Brief>. | sbIDeviceBase.h | /sb-trunk-oldxul/compiled/components/devicesobsolete/base/public | line 80 | C/C++ Task | | XXXeps TODO | deviceControl.xml | /sb-trunk-oldxul/compiled/dist/chrome/songbird/content/songbird/bindings/device | line 98 | XML Task | | TODO >. | sbIDeviceBase.h | /sb-trunk-oldxul/compiled/components/devicesobsolete/base/public | line 74 | C/C++ Task | | TODO Detail>. | sbIDeviceBase.h | /sb-trunk-oldxul/compiled/components/devicesobsolete/base/public | line 72 | C/C++ Task | | TODO The following are known GStreamer tags we don't handle: | sbGStreamerMediacoreUtils.cpp | /sb-trunk-oldxul/components/mediacore/gstreamer/src | line 70 | C/C++ Task | | XXX Mook: we have a silly list of blacklisted extensions because we don't | sbGStreamerMediacoreFactory.cpp | /sb-trunk-oldxul/components/mediacore/gstreamer/src | line 197 | C/C++ Task | | TODO Brief>. | sbIDeviceBase.h | /sb-trunk-oldxul/compiled/components/devicesobsolete/base/public | line 70 | C/C++ Task | | XXX We probably should evaluate dispatching events to listeners | sbBaseMediacoreEventTarget.h | /sb-trunk-oldxul/components/mediacore/base/src | line 90 | C/C++ Task | | TODO This function is now a _huge_ mess. We should talk to product about | sbGStreamerMediacoreFactory.cpp | /sb-trunk-oldxul/components/mediacore/gstreamer/src | line 171 | C/C++ Task | | TODO Detail>. | sbIDeviceBase.h | /sb-trunk-oldxul/compiled/components/devicesobsolete/base/public | line 62 | C/C++ Task | | XXX this needs to be fixed to be not gstreamer specific | sbDeviceUtils.cpp | /sb-trunk-oldxul/components/devices/device/src | line 1670 | C/C++ Task | | XXX: for 1.9, change min from 5 to 59, to select the quality Acme | mp4-mjpeg-adpcm.xml | /sb-trunk-oldxul/components/mediacore/gstreamer/encode-profiles | line 33 | XML Task | | XXX Remove Me !!! | WindowResizeHook.h | /sb-trunk-oldxul/components/integration/src/win32 | line 37 | C/C++ Task | | TODO Include the file creation timestamp in the track name | sbDeviceMediaItemDecorator.cpp | /sb-trunk-oldxul/components/devices/device/src | line 95 | C/C++ Task | | TODO Not sure if we still need to do this | sbDeviceLibrary.cpp | /sb-trunk-oldxul/components/devices/base/src | line 589 | C/C++ Task | | XXX consider using StringMap, code to iterate was easier | sbiTunesImporter.h | /sb-trunk-oldxul/components/mediaimport/library/itunes/src | line 141 | C/C++ Task | | XXX This code will probably need to be updated with the device | sbDeviceLibrary.cpp | /sb-trunk-oldxul/components/devices/base/src | line 545 | C/C++ Task | | TODO This code will probably need to be updated with the device | sbDeviceLibrary.cpp | /sb-trunk-oldxul/components/devices/base/src | line 545 | C/C++ Task | | XXX consider using nsDataHashtable | sbiTunesImporter.h | /sb-trunk-oldxul/components/mediaimport/library/itunes/src | line 146 | C/C++ Task | | TODO consider using StringMap, code to iterate was easier | sbiTunesImporter.h | /sb-trunk-oldxul/components/mediaimport/library/itunes/src | line 141 | C/C++ Task | | XXX not implemented! | sbDeviceLibrary.cpp | /sb-trunk-oldxul/components/devices/base/src | line 1335 | C/C++ Task | | XXX For bug 23188 would we need to update listeners. We used to, | sbDeviceLibrary.cpp | /sb-trunk-oldxul/components/devices/base/src | line 971 | C/C++ Task | | TODO consider using nsDataHashtable | sbiTunesImporter.h | /sb-trunk-oldxul/components/mediaimport/library/itunes/src | line 146 | C/C++ Task | | TODO For bug 23188 would we need to update listeners. We used to, | sbDeviceLibrary.cpp | /sb-trunk-oldxul/components/devices/base/src | line 971 | C/C++ Task | | TODO: allow external menu? | sysControls.xml | /sb-trunk-oldxul/compiled/dist/chrome/songbird/content/songbird/bindings | line 397 | XML Task | | XXX Not sure if we still need to do this | sbDeviceLibrary.cpp | /sb-trunk-oldxul/components/devices/base/src | line 589 | C/C++ Task | | XXX % | utils.cpp | /sb-trunk-oldxul/tools/disthelper | line 84 | C/C++ Task | | TODO set might be overkill, vector with a linear search might win | sbiTunesAgentAppWatcher.h | /sb-trunk-oldxul/tools/itunesagent/windows | line 59 | C/C++ Task | | XXX $ with environment variable % | utils.cpp | /sb-trunk-oldxul/tools/disthelper | line 84 | C/C++ Task | | XXX set might be overkill, vector with a linear search might win | sbiTunesAgentAppWatcher.h | /sb-trunk-oldxul/tools/itunesagent/windows | line 59 | C/C++ Task | | XXX $ with environment variable %DISTHELPER_XXX% | utils.cpp | /sb-trunk-oldxul/tools/disthelper | line 73 | C/C++ Task | | XXX slloyd The play queue index is an index into the unfiltered | sbLocalDatabaseTreeView.cpp | /sb-trunk-oldxul/components/library/localdatabase/src | line 2853 | C/C++ Task | | XXX nsTreeBoxObject calls this method with a null to break a cycle so | sbLocalDatabaseTreeView.cpp | /sb-trunk-oldxul/components/library/localdatabase/src | line 2313 | C/C++ Task | | TODO Throw an exception if the mapping is currently in use? | sbIFeathersManager.h | /sb-trunk-oldxul/compiled/components/feathers/public | line 173 | C/C++ Task | | TODO give useful arguments here | nsSystemTrayIconBase.cpp | /sb-trunk-oldxul/extensions/systray/components/src | line 136 | C/C++ Task | | XXX If cache is invalidated, refresh now? | sbLocalDatabaseMediaItem.cpp | /sb-trunk-oldxul/components/library/localdatabase/src | line 152 | C/C++ Task | | TODO consider removing. sbIMetadataChannel appears to be unused. | sbIMetadataHandler.h | /sb-trunk-oldxul/compiled/components/mediacore/metadata/manager/public | line 242 | C/C++ Task | | TODO Brief>. | sbIDeviceBase.h | /sb-trunk-oldxul/compiled/components/devicesobsolete/base/public | line 60 | C/C++ Task | | TODO Revisit the main thread dispatching this may prove a bottleneck for media cores. | sbBaseMediacoreEventTarget.h | /sb-trunk-oldxul/components/mediacore/base/src | line 53 | C/C++ Task | | TODO Detail>. | sbIDeviceBase.h | /sb-trunk-oldxul/compiled/components/devicesobsolete/base/public | line 52 | C/C++ Task | | TODO We probably should evaluate dispatching events to listeners | sbBaseMediacoreEventTarget.h | /sb-trunk-oldxul/components/mediacore/base/src | line 90 | C/C++ Task | | TODO Brief>. | sbIDeviceBase.h | /sb-trunk-oldxul/compiled/components/devicesobsolete/base/public | line 50 | C/C++ Task | | TODO Detail>. | sbIDeviceBase.h | /sb-trunk-oldxul/compiled/components/devicesobsolete/base/public | line 41 | C/C++ Task | | XXX we should look at replacing | sbBaseDevice.h | /sb-trunk-oldxul/components/devices/device/src | line 713 | C/C++ Task | | TODO we should look at replacing | sbBaseDevice.h | /sb-trunk-oldxul/components/devices/device/src | line 713 | C/C++ Task | | TODO is this assertion correct? | versioninfo.cpp | /sb-trunk-oldxul/tools/disthelper/windows | line 243 | C/C++ Task | | XXX Mook: consider wrapping the listener in a proxy | sbBaseMediacoreEventTarget.cpp | /sb-trunk-oldxul/components/mediacore/base/src | line 159 | C/C++ Task | | XXX Mook: if we wrapped listeners, watch for equality! | sbBaseMediacoreEventTarget.cpp | /sb-trunk-oldxul/components/mediacore/base/src | line 196 | C/C++ Task | | XXXMook: need to be smarter and only rebuild things that have changed | propertiesMenuList.xml | /sb-trunk-oldxul/compiled/dist/chrome/songbird/content/songbird/bindings | line 352 | XML Task | | XXX - Due to caching we cannot allow the secondary sort | sbPropertyInfo.cpp | /sb-trunk-oldxul/components/property/src | line 353 | C/C++ Task | | XXX | advanced.dtd | /sb-trunk-oldxul/compiled/dist/chrome/browser-locale/locale/browser/preferences | line 24 | DTD Task | | XXX Mook: I can't figure out when the heck CheckAccess gets called :( | sbXPCScriptableStub.h | /sb-trunk-oldxul/components/remoteapi/src | line 171 | C/C++ Task | | TODO consider using nsDeque | sbMetadataJob.h | /sb-trunk-oldxul/components/mediacore/metadata/manager/src | line 333 | C/C++ Task | | TODO This is redundant, as mFilesrc already | sbGStreamerMediaContainer.h | /sb-trunk-oldxul/components/mediacore/gstreamer/src | line 103 | C/C++ Task | | XXX Mook: if we wrapped listeners, watch for equality! | sbBaseDeviceEventTarget.cpp | /sb-trunk-oldxul/components/devices/base/src | line 242 | C/C++ Task | | TODO if this fails, we could fall back to using the inspector? | sbDeviceTranscoding.cpp | /sb-trunk-oldxul/components/devices/device/src | line 553 | C/C++ Task | | XXX Mook: consider wrapping the listener in a proxy | sbBaseDeviceEventTarget.cpp | /sb-trunk-oldxul/components/devices/base/src | line 206 | C/C++ Task | | XXX Mook: USE_JSSTUB_FOR_ADDPROPERTY is needed to define things on the | sbRemoteLibraryBase.cpp | /sb-trunk-oldxul/components/remoteapi/src | line 1004 | C/C++ Task | | TODO This can cause problems if this fires when the user is dragging | sbWatchFolderService.cpp | /sb-trunk-oldxul/components/watchfolder/src | line 418 | C/C++ Task | | XXX This can cause problems if this fires when the user is dragging | sbWatchFolderService.cpp | /sb-trunk-oldxul/components/watchfolder/src | line 418 | C/C++ Task | | XXX check result? | sbIPDStatus.cpp | /sb-trunk-oldxul/extensions/ipod/components/src | line 501 | C/C++ Task | | XXX should we be calling UpdateStatus? | sbIPDStatus.cpp | /sb-trunk-oldxul/extensions/ipod/components/src | line 424 | C/C++ Task | | XXX This is bad since it assumes the implementation class of the | sbSQLBuilderBase.cpp | /sb-trunk-oldxul/components/sqlbuilder/src | line 211 | C/C++ Task | | XXX is the media item's trackType property. | sbIMediaItemController.h | /sb-trunk-oldxul/compiled/components/library/base/public | line 137 | C/C++ Task | | XXX Remove Me !!! | WindowMinMaxSubclass.h | /sb-trunk-oldxul/components/integration/src/win32 | line 38 | C/C++ Task | | XXX is the media item's trackType property. | sbIMediaItemController.h | /sb-trunk-oldxul/compiled/components/library/base/public | line 138 | C/C++ Task | | TODO Write this documentation | sbIRemoteMediaList.h | /sb-trunk-oldxul/compiled/components/remoteapi/public | line 33 | C/C++ Task | | TODO Implement ability to seek. Right now this is just set to false. | mozillasink.cpp | /sb-trunk-oldxul/components/mediacore/gstreamer/gst-plugins/mozilla | line 47 | C/C++ Task | | XXX Move this off of the main thread | sbFileSystemTree.cpp | /sb-trunk-oldxul/components/filesystemevents/base/src | line 456 | C/C++ Task | | TODO Write this documentation | sbIRemoteCommands.h | /sb-trunk-oldxul/compiled/components/remoteapi/public | line 31 | C/C++ Task | | XXXeps this can be removed since click events now use | playlist.xml | /sb-trunk-oldxul/compiled/dist/chrome/songbird/content/songbird/bindings | line 384 | XML Task | | XXXben This is here to fix a bug where 'this.tree.view' will return | playlist.xml | /sb-trunk-oldxul/compiled/dist/chrome/songbird/content/songbird/bindings | line 272 | XML Task | | XXX Mook: USE_JSSTUB_FOR_ADDPROPERTY is needed to define things on the | sbScriptableFilterItems.cpp | /sb-trunk-oldxul/components/remoteapi/src | line 158 | C/C++ Task | | XXX Mook: this is pretty sucky. We take all the items in the other set | sbScriptableFilterItems.cpp | /sb-trunk-oldxul/components/remoteapi/src | line 444 | C/C++ Task | | TODO Look at top level properties to see if you want them searchable! | sbLocalDatabasePropertyCache.cpp | /sb-trunk-oldxul/components/library/localdatabase/src | line 1184 | C/C++ Task | | TODO Fix this so either we don't need to dispatch or don't hold | sbLocalDatabasePropertyCache.cpp | /sb-trunk-oldxul/components/library/localdatabase/src | line 1260 | C/C++ Task | | XXX Fix this so either we don't need to dispatch or don't hold | sbLocalDatabasePropertyCache.cpp | /sb-trunk-oldxul/components/library/localdatabase/src | line 1260 | C/C++ Task | | XXX Don't add any calls here that could possibly fail! We've already added | DeviceManager.cpp | /sb-trunk-oldxul/components/devicesobsolete/manager/src | line 136 | C/C++ Task | | XXX Mook: we need to adjust to something | sbGStreamerTranscodeDeviceConfigurator.cpp | /sb-trunk-oldxul/components/mediacore/gstreamer/src | line 987 | C/C++ Task | | XXX Mook: | sbGStreamerTranscodeDeviceConfigurator.cpp | /sb-trunk-oldxul/components/mediacore/gstreamer/src | line 616 | C/C++ Task | | TODO match properties | sbGStreamerTranscodeDeviceConfigurator.cpp | /sb-trunk-oldxul/components/mediacore/gstreamer/src | line 616 | C/C++ Task | | XXX Mook: | sbGStreamerTranscodeDeviceConfigurator.cpp | /sb-trunk-oldxul/components/mediacore/gstreamer/src | line 591 | C/C++ Task | | TODO match properties | sbGStreamerTranscodeDeviceConfigurator.cpp | /sb-trunk-oldxul/components/mediacore/gstreamer/src | line 591 | C/C++ Task | | XXX Mook: we only support video for now | sbGStreamerTranscodeDeviceConfigurator.cpp | /sb-trunk-oldxul/components/mediacore/gstreamer/src | line 552 | C/C++ Task | | TODO some ogg-expert please check why there is no Ogg::FLAC in here? | MetadataHandlerTaglib.cpp | /sb-trunk-oldxul/components/mediacore/metadata/handler/taglib/src | line 757 | C/C++ Task | | TODO bug 10932 -- fix WCOP to be like this in TL | MetadataHandlerTaglib.cpp | /sb-trunk-oldxul/components/mediacore/metadata/handler/taglib/src | line 721 | C/C++ Task | | TODO Move all suff here to the WriteXXX methods, for better code | MetadataHandlerTaglib.cpp | /sb-trunk-oldxul/components/mediacore/metadata/handler/taglib/src | line 667 | C/C++ Task | | TODO write other files' metadata. | MetadataHandlerTaglib.cpp | /sb-trunk-oldxul/components/mediacore/metadata/handler/taglib/src | line 666 | C/C++ Task | | TODO bug 10932 -- make WCOP like this in taglib | MetadataHandlerTaglib.cpp | /sb-trunk-oldxul/components/mediacore/metadata/handler/taglib/src | line 2046 | C/C++ Task | | TODO Auto-generated destructor stub | sbiTunesImporterStatus.cpp | /sb-trunk-oldxul/components/mediaimport/library/itunes/src | line 46 | C/C++ Task | | TODO write other files' metadata. | MetadataHandlerTaglib.cpp | /sb-trunk-oldxul/components/mediacore/metadata/handler/taglib/src | line 1146 | C/C++ Task | | TODO support other ISO 8601 formats. | sbStringUtils.cpp | /sb-trunk-oldxul/components/moz/strings/src | line 459 | C/C++ Task | | XXX Mook: temporary hack to make tree build, reopening bug 16158 | MetadataHandlerTaglib.cpp | /sb-trunk-oldxul/components/mediacore/metadata/handler/taglib/src | line 1093 | C/C++ Task | | TODO need to set pWriteCount | MetadataHandlerTaglib.cpp | /sb-trunk-oldxul/components/mediacore/metadata/handler/taglib/src | line 825 | C/C++ Task | | TODO GetFileSize will fail, since it wont use this URL | sbMetadataJob.cpp | /sb-trunk-oldxul/components/mediacore/metadata/manager/src | line 491 | C/C++ Task | | XXX Mook: We should be looking at whether the | MetadataHandlerTaglib.cpp | /sb-trunk-oldxul/components/mediacore/metadata/handler/taglib/src | line 2837 | C/C++ Task | | TODO needed with current taglib? | MetadataHandlerTaglib.cpp | /sb-trunk-oldxul/components/mediacore/metadata/handler/taglib/src | line 2516 | C/C++ Task | | XXX Mook this is not always true for ogx, but we need something for now | MetadataHandlerTaglib.cpp | /sb-trunk-oldxul/components/mediacore/metadata/handler/taglib/src | line 2269 | C/C++ Task | | XXX Mook not always true for mp4, but good enough for now | MetadataHandlerTaglib.cpp | /sb-trunk-oldxul/components/mediacore/metadata/handler/taglib/src | line 2260 | C/C++ Task | | TODO MP4 did not yet recive the update to PropertyMap for non-default | MetadataHandlerTaglib.cpp | /sb-trunk-oldxul/components/mediacore/metadata/handler/taglib/src | line 3493 | C/C++ Task | | TODO Make "GetSeekable()" work. Right now, nothing is seekable. | sbIMetadataChannel.h | /sb-trunk-oldxul/compiled/components/mediacore/metadata/manager/public | line 73 | C/C++ Task | | TODO ASF did not yet recive the update to PropertyMap for non-default | MetadataHandlerTaglib.cpp | /sb-trunk-oldxul/components/mediacore/metadata/handler/taglib/src | line 3370 | C/C++ Task | | TODO Make "SetPos()" work past the end of the buffer. Right now, nothing is seekable. | sbIMetadataChannel.h | /sb-trunk-oldxul/compiled/components/mediacore/metadata/manager/public | line 53 | C/C++ Task | | TODO discuss APE mapping in general. | MetadataHandlerTaglib.cpp | /sb-trunk-oldxul/components/mediacore/metadata/handler/taglib/src | line 3334 | C/C++ Task | | TODO in far future once the PropertyMap supports binary stuff: | MetadataHandlerTaglib.cpp | /sb-trunk-oldxul/components/mediacore/metadata/handler/taglib/src | line 3160 | C/C++ Task | | TODO Bad times! Shouldn't do anything involving channels off of the main thread. | sbMetadataManager.cpp | /sb-trunk-oldxul/components/mediacore/metadata/manager/src | line 153 | C/C++ Task | | TODO tweak me | sbMainThreadMetadataProcessor.cpp | /sb-trunk-oldxul/components/mediacore/metadata/manager/src | line 47 | C/C++ Task | | TODO VALIDATE VALUES | sbLibraryConstraints.cpp | /sb-trunk-oldxul/components/library/base/src | line 258 | C/C++ Task | | TODO VALIDATE VALUES | sbLibraryConstraints.cpp | /sb-trunk-oldxul/components/library/base/src | line 288 | C/C++ Task | | TODO something better than looking at file extensions?! | MetadataHandlerTaglib.cpp | /sb-trunk-oldxul/components/mediacore/metadata/handler/taglib/src | line 664 | C/C++ Task | | TODO get rid of this | MetadataHandlerTaglib.cpp | /sb-trunk-oldxul/components/mediacore/metadata/handler/taglib/src | line 643 | C/C++ Task | | XXX Remove Me !!! | WindowRegion.h | /sb-trunk-oldxul/components/integration/src/win32 | line 37 | C/C++ Task | | TODO Make this a full-blown component so JS can use it. | sbProcess.h | /sb-trunk-oldxul/components/moz/process/src | line 72 | C/C++ Task | | TODO Add support for providing a done callback as well as a monitor | sbProcess.h | /sb-trunk-oldxul/components/moz/process/src | line 73 | C/C++ Task | | XXX changes. | sbMediacoreSequencer.cpp | /sb-trunk-oldxul/components/mediacore/manager/src | line 2251 | C/C++ Task | | TODO Clean up, localize, or remove from the product | sbFileMetadataService.cpp | /sb-trunk-oldxul/components/mediacore/metadata/manager/src | line 726 | C/C++ Task | | TODO Not necessary, but we could shut down the scanners here | sbFileMetadataService.cpp | /sb-trunk-oldxul/components/mediacore/metadata/manager/src | line 664 | C/C++ Task | | TODO consider giving preference to the most recent job | sbFileMetadataService.cpp | /sb-trunk-oldxul/components/mediacore/metadata/manager/src | line 440 | C/C++ Task | | TODO Not necessary, but could only start processors when needed. | sbFileMetadataService.cpp | /sb-trunk-oldxul/components/mediacore/metadata/manager/src | line 395 | C/C++ Task | | TODO Add getter_AddRefs equivalent | sbPropertyBagUtils.h | /sb-trunk-oldxul/components/moz/xpcom/src | line 258 | C/C++ Task | | TODO Determine what info this will actually be passed! | sbITranscodeVideoJob.h | /sb-trunk-oldxul/compiled/components/mediacore/transcode/public | line 136 | C/C++ Task | | XXXeps Should use sb-sys-outer-frame and sb-sys-wizard-titlebar but don't | wizard.xml | /sb-trunk-oldxul/compiled/dist/chrome/songbird/content/songbird/bindings | line 63 | XML Task | | XXXeps have time to do the wizard specific work this late into Fugazi. | wizard.xml | /sb-trunk-oldxul/compiled/dist/chrome/songbird/content/songbird/bindings | line 64 | XML Task | | XXXeps wizards have a header instead of a titlebar and the wizard header | wizard.xml | /sb-trunk-oldxul/compiled/dist/chrome/songbird/content/songbird/bindings | line 65 | XML Task | | XXX rearrange entities to match physical layout when l10n isn't an issue | sanitize.dtd | /sb-trunk-oldxul/compiled/dist/chrome/browser-locale/locale/browser | line 7 | DTD Task | | XXXeps has a few differences that should be accounted for. | wizard.xml | /sb-trunk-oldxul/compiled/dist/chrome/songbird/content/songbird/bindings | line 66 | XML Task | | TODO Should we make a copy? | sbAlbumArtScanner.cpp | /sb-trunk-oldxul/components/albumart/src | line 133 | C/C++ Task | | TODO cleanup | sbMetadataCrashTracker.h | /sb-trunk-oldxul/components/mediacore/metadata/manager/src | line 35 | C/C++ Task | | TODO discuss XiphComment mapping in general. | MetadataHandlerTaglib.cpp | /sb-trunk-oldxul/components/mediacore/metadata/handler/taglib/src | line 3539 | C/C++ Task |