mozilla
/
gecko-dev
зеркало из https://github.com/mozilla/gecko-dev.git
1
0
Форкнуть 0
Код Задачи Пакеты Проекты Релизы Вики Активность

Bug 1547657: Null check frame in HyperTextAccessible::TextBounds. r=MarcoZ 

Differential Revision: https://phabricator.services.mozilla.com/D78631
This commit is contained in:
James Teh 2020-06-08 05:11:23 +00:00
Родитель 58a574a9c2
Коммит b1c5fca60f
3 изменённых файлов: 42 добавлений и 1 удалений
Показать статистику Скачать Patch файл Скачать Diff файл Показать все файлы Свернуть все файлы

6
accessible/generic/HyperTextAccessible.cpp
Просмотреть файл

@ -1182,7 +1182,11 @@ nsIntRect HyperTextAccessible::TextBounds(int32_t aStartOffset,
if (CharacterCount() == 0) {
nsPresContext* presContext = mDoc->PresContext();
// Empty content, use our own bound to at least get x,y coordinates
return GetFrame()->GetScreenRectInAppUnits().ToNearestPixels(
nsIFrame* frame = GetFrame();
if (!frame) {
return nsIntRect();
}
return frame->GetScreenRectInAppUnits().ToNearestPixels(
presContext->AppUnitsPerDevPixel());
}

1
accessible/tests/mochitest/text/a11y.ini
Просмотреть файл

@ -13,5 +13,6 @@ support-files = doc.html
[test_passwords.html]
[test_selection.html]
[test_settext_input_event.html]
[test_textBounds.html]
[test_wordboundary.html]
[test_words.html]

36
accessible/tests/mochitest/text/test_textBounds.html Normal file
Просмотреть файл

@ -0,0 +1,36 @@
<!DOCTYPE html>
<html>
<head>
<title>TextBounds tests</title>
<link rel="stylesheet" type="text/css"
href="chrome://mochikit/content/tests/SimpleTest/test.css" />
<script src="chrome://mochikit/content/tests/SimpleTest/SimpleTest.js"></script>
<script type="application/javascript"
src="../common.js"></script>
<script type="application/javascript"
src="../layout.js"></script>
<script type="application/javascript">
function doTest() {
// Returned rect should be all 0 if no frame; e.g. display: contents.
testTextBounds(
"displayContents", 0, 0, [0, 0, 0, 0], COORDTYPE_SCREEN_RELATIVE
);
SimpleTest.finish();
}
SimpleTest.waitForExplicitFinish();
addA11yLoadEvent(doTest);
</script>
</head>
<body>
<p id="display"></p>
<div id="content" style="display: none"></div>
<pre id="test">
</pre>
<section id="displayContents" style="display: contents;"></section>
</body>
</html>