Closed
Bug 1225911
Opened 8 years ago
Closed 8 years ago
Typo in tabbrowser.xml, incorrect usage of boolean-and when binary-and was intended
Categories
(Core Graveyard :: Security: UI, defect)
Core Graveyard
Security: UI
Tracking
(firefox46 fixed)
RESOLVED
FIXED
mozilla46
Tracking | Status | |
---|---|---|
firefox46 | --- | fixed |
People
(Reporter: jaws, Assigned: jaws)
References
Details
Attachments
(1 file, 1 obsolete file)
1.53 KB,
patch
|
jaws
:
review+
|
Details | Diff | Splinter Review |
See http://hg.mozilla.org/mozilla-central/annotate/1d6155d7e6c9/browser/base/content/tabbrowser.xml#l769
Flags: needinfo?(felipc)
Assignee | ||
Updated•8 years ago
|
Flags: needinfo?(dao)
Assignee | ||
Comment 1•8 years ago
|
||
This is the obvious fix, but Felipe or Dao should test that the behavior with the fix is correct, and what bug might have gone unnoticed before. Probably good to have a test for this. (not assigning to myself as someone may want to write a test for this and that will be 99% of the work for this bug)
Updated•8 years ago
|
Flags: needinfo?(dao)
Assignee | ||
Updated•8 years ago
|
Attachment #8689079 -
Flags: review?(dao)
Updated•8 years ago
|
Attachment #8689079 -
Flags: review?(dao) → review+
Assignee | ||
Updated•8 years ago
|
Comment 2•8 years ago
|
||
Hi, this failed to apply: applying 1225911.patch patching file browser/base/content/tabbrowser.xml Hunk #1 FAILED at 760 1 out of 1 hunks FAILED -- saving rejects to file browser/base/content/tabbrowser.xml.rej patch failed, unable to continue (try -v) patch failed, rejects left in working directory errors during apply, please fix and qrefresh 1225911.patch
Flags: needinfo?(jaws)
Keywords: checkin-needed
Assignee | ||
Comment 3•8 years ago
|
||
Attachment #8689079 -
Attachment is obsolete: true
Flags: needinfo?(jaws)
Attachment #8707478 -
Flags: review+
Assignee | ||
Updated•8 years ago
|
Keywords: checkin-needed
Comment 5•8 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/8a2d17f7f3d1
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
status-firefox46:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla46
Updated•7 years ago
|
Product: Core → Core Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•