On Wed, Jun 4, 2014 at 4:25 PM, Georg Baum <georg.b...@post.rwth-aachen.de> wrote: > Scott Kostyshak wrote: > >> Thanks for taking a look, Georg. From what I understand, the only >> missing piece to the patch is the XML parser. > > Yes. > >> But I added a line >> lyxerr << "str is: " << str << std::endl; >> right before the "Now use some XML parser" comment, and it is only >> triggered when pasting from Firefox, not from Chromium. > > This indicates either a bug (maybe GuiClipboard::on_dataChanged() is not > called although it should), or I misunderstood how > GuiClipboard::on_dataChanged() works. > However, I think you got the general idea.
I investigated further. The reason is that cache_.hasText() is true so hasTextContents is true so has_text_contents_not_only_image_link_ is true, so the if is not triggered. Does that help? Scott