Closed Bug 1393662 Opened 7 years ago Closed 6 years ago

Making IsResistFingerprintingEnabled() checks in nsRFPService::ReduceTimePrecisionAs* inline and changing the name of nsRFPService::ReduceTimePrecisionAs* into MaybeReduceTimePrecisionAs*

Categories

(Core :: DOM: Security, enhancement, P2)

Product:
Core
Component:
DOM: Security
Type:
enhancement

Tracking

()

Status:
RESOLVED WONTFIX

People

(Reporter: timhuang, Assigned: tjr)

References

(Blocks 1 open bug)

Details

(Whiteboard: [fingerprinting][tor][domsecurity-backlog][fp:m4]) 

Per Bug 1382545 comment 6, we should make IsResistFingerprintingEnabled() in nsRFPService::ReduceTimePrecisionAs*() inline, which can reduce the overhead of calling this function when fingerprinting resistance is disabled.

And changing the name 'ReduceTimePrecisionAs*' into 'MaybeReduceTimePrecisionAs*', which serves better for describing the functionality of this method.
See Also: → 1382545
Assignee: nobody → ettseng
Whiteboard: [fingerprinting][tor][domsecurity-backlog] → [fingerprinting][tor][domsecurity-backlog][fp:m4]
Assignee: ethantseng → tom
Priority: P3 → P2
Now that we've made these functions much more complicated, making them inline isn't really beneficial anymore.  Also, no one else complained about the name (and time reduction is on my default now).
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.