Closed Bug 1458504 Opened 6 years ago Closed 6 years ago

Move Tokenizer into it's own file so it can be shared by the CSP Parser and the Feature Policy Parser

Categories

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

Product:
Core
Component:
DOM: Security
Type:
enhancement

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla62
Tracking Flags:
Tracking Status
firefox62 --- fixed

People

(Reporter: ckerschb, Assigned: ckerschb)

References

Details

(Whiteboard: [domsecurity-active])

Attachments

(1 file)

bug_1458504_factor_out_tokenizer.patch
13.49 KB, patch
jkt
: review+
Details | Diff | Splinter Review 
      No description provided.
Blocks: 1390801
Priority: -- → P3
Whiteboard: [domsecurity-backlog1]
Assignee: nobody → ckerschb
Status: NEW → ASSIGNED
Priority: P3 → P2
Whiteboard: [domsecurity-backlog1] → [domsecurity-active]
Jonathan, just moving the tokenizer into it's own file so it can be included by the CSP Parser as well as the Feature Policy Parser.
Attachment #8972632 - Flags: review?(jkt)
Comment on attachment 8972632 [details] [diff] [review]
bug_1458504_factor_out_tokenizer.patch

LGTM sorry for the delay.
Attachment #8972632 - Flags: review?(jkt) → review+
Pushed by ryanvm@gmail.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/4e532ee4c189
Move Tokenizer into it's own file so it can be shared by the CSP Parser and the Feature Policy Parser. r=jkt
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/4e532ee4c189
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
status-firefox62: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla62
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: