1024966 - SelectionCarets::UpdateSelectionCarets should check whether the frame is null or not

Status: RESOLVED FIXED

(Core :: DOM: Selection, defect)

Description

[StevenLee[:slee]]

When the element is hide, nsIContent, [1], will be null. If we do not skip it, it generates too many assertion fail and cause test case fails, [2]. 

[1] http://dxr.mozilla.org/mozilla-central/source/layout/base/SelectionCarets.cpp#472
[2] https://tbpl.mozilla.org/php/getParsedLog.php?id=41663846&tree=Try&full=1

Comment 1

[Morris Tseng [:mtseng] [:Morris] (Inactive)]

Attachment: bug1024966

Check mResultContent before using it!

Comment 2

[Morris Tseng [:mtseng] [:Morris] (Inactive)]

try run: https://tbpl.mozilla.org/?tree=Try&rev=d6e8a0a5c1d7

Comment 3

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/c9b503e42352

Comment 4

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/c9b503e42352