зеркало из https://github.com/mozilla/gecko-dev.git
Merge mozilla-central to autoland a=merge
This commit is contained in:
Коммит
9df7619fc3
|
@ -7009,7 +7009,8 @@ void nsPIDOMWindowInner::BroadcastReport(Report* aReport) {
|
|||
}
|
||||
|
||||
void nsPIDOMWindowInner::NotifyReportingObservers() {
|
||||
for (ReportingObserver* observer : mReportingObservers) {
|
||||
nsTArray<RefPtr<ReportingObserver>> reportingObservers(mReportingObservers);
|
||||
for (ReportingObserver* observer : reportingObservers) {
|
||||
observer->MaybeNotify();
|
||||
}
|
||||
}
|
||||
|
|
|
@ -1617,7 +1617,6 @@ class StaticAnalysisMonitor(object):
|
|||
self._warnings_database = WarningsDatabase()
|
||||
|
||||
def on_warning(warning):
|
||||
filename = warning['filename']
|
||||
self._warnings_database.insert(warning)
|
||||
|
||||
self._warnings_collector = WarningsCollector(on_warning, objdir=objdir)
|
||||
|
@ -2466,7 +2465,6 @@ class StaticAnalysis(MachCommandBase):
|
|||
regex_header = re.compile(
|
||||
r'(.+):(\d+):(\d+): (warning|error): ([^\[\]\n]+)(?: \[([\.\w-]+)\])?$', re.MULTILINE)
|
||||
|
||||
something = regex_header.finditer(clang_output)
|
||||
headers = sorted(
|
||||
regex_header.finditer(clang_output),
|
||||
key=lambda h: h.start()
|
||||
|
|
|
@ -7,7 +7,13 @@
|
|||
<body>
|
||||
<div id='fs'></div>
|
||||
<script>
|
||||
var observer1;
|
||||
var observer2;
|
||||
|
||||
var check_report_format = (reports, observer) => {
|
||||
// Test that observer2 is notified, even if it is disconnected.
|
||||
observer1.disconnect();
|
||||
observer2.disconnect();
|
||||
let report = reports[0];
|
||||
assert_equals(report.type, "feature-policy-violation");
|
||||
assert_equals(report.url, document.location.href);
|
||||
|
@ -18,9 +24,19 @@ var check_report_format = (reports, observer) => {
|
|||
assert_equals(report.body.disposition, "enforce");
|
||||
};
|
||||
|
||||
var check_second_observer = (reports, observer) => {
|
||||
let report = reports[0];
|
||||
assert_equals(report.type, "feature-policy-violation");
|
||||
assert_equals(report.body.featureId, "fullscreen");
|
||||
};
|
||||
|
||||
async_test(t => {
|
||||
new ReportingObserver(t.step_func_done(check_report_format),
|
||||
{types: ['feature-policy-violation']}).observe();
|
||||
observer1 = new ReportingObserver(t.step_func(check_report_format),
|
||||
{types: ['feature-policy-violation']});
|
||||
observer1.observe();
|
||||
observer2 = new ReportingObserver(t.step_func_done(check_second_observer),
|
||||
{types: ['feature-policy-violation']});
|
||||
observer2.observe();
|
||||
document.getElementById('fs').requestFullscreen().then(t.unreached_func(
|
||||
"Fullscreen should not be allowed in this document.")).catch(()=>{});
|
||||
}, "Fullscreen Report Format");
|
||||
|
|
Загрузка…
Ссылка в новой задаче