Closed Bug 1382326 Opened 7 years ago Closed 4 years ago

Remote or cache native look and feel metrics queries in content processes

Categories

(Core :: Widget: Win32, defect, P1)

Unspecified
Windows
defect

Tracking

()

RESOLVED DUPLICATE of bug 1400317

People

(Reporter: Alex_Gaynor, Assigned: jimm)

References

(Blocks 2 open bugs)

Details

(Whiteboard: sb+)

Attachments

(1 file)

Doing this goes through a win32k syscall. Here's what a stacktrace looks like:


2 - USER32!NtUserGetCaretBlinkTime
    USER32!NtUserGetCaretBlinkTime
    xul!nsLookAndFeel::GetIntImpl+0xb5
    xul!mozilla::LookAndFeel::GetInt+0xc
    xul!nsCaret::ResetBlinking+0x40
    xul!nsCaret::SetVisible+0x14
    xul!mozilla::PresShell::SetCaretEnabled+0x1f
    xul!nsTextInputSelectionImpl::SetCaretEnabled+0x8c
    xul!mozilla::EditorBase::InitializeSelection+0x14a
    xul!mozilla::EditorBase::OnFocus+0xe
    xul!mozilla::EditorEventListener::Focus+0x25a
    xul!mozilla::EditorEventListener::HandleEvent+0x95
Group: core-security
Component: Security: Process Sandboxing → Widget: Win32
Whiteboard: sb+
accidentally marked this sec, please remove.
Flags: needinfo?(dveditz)
Group: core-security
Flags: needinfo?(dveditz)
Depends on: 1383659
Another stack that passes through nsLookAndFeel:

    win32u!NtUserSystemParametersInfo
    USER32!RealSystemParametersInfoW+0x82
    UxTheme!ClassicSystemParametersInfoW+0x29
    UxTheme!_InternalSystemParametersInfo+0x29
    UxTheme!ThemeSystemParametersInfoW+0xcb
    USER32!SystemParametersInfoW+0xa7
    UxTheme!IsHighContrastMode+0x2a
    UxTheme!IsThemeActive+0x5a
    UxTheme!IsAppThemed+0x16
    xul!nsLookAndFeel::GetIntImpl+0x40d
    xul!nsCSSRuleProcessor::InitSystemMetrics+0x2db
    xul!nsCSSRuleProcessor::HasSystemMetric+0xe
    xul!GetSystemMetric+0x52
Priority: -- → P3
Summary: Stop trying to get the caret blink time in the content process → Remote or cache native look and feel metrics queries in content processes
Assignee: nobody → jmathies
Priority: P3 → P1
Depends on: 1400317
Blocks: 1400317
No longer depends on: 1400317
Blocks: 1156934
Blocks: 1474200
Attached patch wipSplinter Review

This will be tackled as part of bug 1400317.

Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → DUPLICATE
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: