- most of the code is behind an always-on macro from 2003 - confused comments from 2006 - the function would need to be touched for bug 1387143 anyway Patch coming.

Comment on attachment 8960344 [details] Bug 1447121 - Clean up ShouldDisplaySelection, convert it to use Selection and nsRange. https://reviewboard.mozilla.org/r/229096/#review235424 ::: layout/generic/nsImageFrame.cpp (Diff revision 1) > #include "nsDisplayList.h" > #include "nsIContent.h" > #include "nsIDocument.h" > #include "FrameLayerBuilder.h" > -#include "nsISelectionController.h" > -#include "nsISelection.h" You need to include mozilla/dom/Selection.h instead, right? ::: layout/generic/nsImageFrame.cpp (Diff revision 1) > #include "imgIContainer.h" > #include "imgLoader.h" > #include "imgRequestProxy.h" > > #include "nsCSSFrameConstructor.h" > -#include "nsIDOMRange.h" And mozilla/dom/Range.h r=me with those fixed.

Pushed by bzbarsky@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/86d2f7875bcc Clean up ShouldDisplaySelection, convert it to use Selection and nsRange. r=bz