Closed
Bug 725010
Opened 12 years ago
Closed 12 years ago
Web API hooks not added for pages without /apps/ in path
Categories
(Firefox OS Graveyard :: General, defect)
Firefox OS Graveyard
General
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: cjones, Assigned: cjones)
Details
(Whiteboard: [inbound])
Attachments
(1 file, 1 obsolete file)
1.17 KB,
patch
|
vingtetun
:
review+
|
Details | Diff | Splinter Review |
This prevents the hooks from being installed for |foo.html|, for example. This check doesn't improve security, so I'm ok with removing it.
Assignee | ||
Comment 1•12 years ago
|
||
Assignee: nobody → jones.chris.g
Attachment #595087 -
Flags: review?(21)
Comment 2•12 years ago
|
||
Comment on attachment 595087 [details] [diff] [review] Remove check for /apps/ in path for shim web APIs Wrong patch
Updated•12 years ago
|
Attachment #595087 -
Flags: review?(21)
Assignee | ||
Comment 3•12 years ago
|
||
Thanks. I botched the qimport and hg didn't complain. Grr.
Attachment #595087 -
Attachment is obsolete: true
Attachment #595093 -
Flags: review?(21)
Attachment #595093 -
Flags: review?(21) → review+
Assignee | ||
Comment 4•12 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/fb59e7eae2a9
Whiteboard: [inbound]
Comment 5•12 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/fb59e7eae2a9
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•