Opened on 07/31/2018 at 08:45:55 PM

Closed on 07/31/2018 at 08:47:53 PM

#6824 closed defect (duplicate)

Remove /u flag for regular expressions

Reported by: hfiguiere Assignee:
Priority: Unknown Milestone:
Module: Core Keywords:
Cc: kzar, sebastian, mjethani Blocked By:
Blocking: Platform: Chrome
Ready: no Confidential: no
Tester: Ross Verified working: no
Review URL(s):

Description

Environment

Chrome 49 doesn't support /u in regular expression.

So we should remove it.

How to reproduce

See issue #6821

Observed behaviour

  • Errors pop on Chrome 49

Expected behaviour

No error.

What to change

  • Both lib/snippets.js and lib/common.js use /u in redundant way: remove it
  • test/browser/elemHideEmulation.js has a test for the /u flags. Ensure it is safe to use at runtime.

Attachments (0)

Change History (1)

comment:1 Changed on 07/31/2018 at 08:47:53 PM by hfiguiere

  • Blocking 6821 removed
  • Resolution set to duplicate
  • Status changed from new to closed

Manish filed #6823 just before this. Marking as dupe.

Add Comment

Modify Ticket

Change Properties
Action
as closed .
The resolution will be deleted. Next status will be 'reopened'.
to The owner will be changed from (none).
Attachments ↑ Description ↑
 
Note: See TracTickets for help on using tickets.