Opened 5 years ago

Closed 5 years ago

Last modified 12 months ago

#11 closed change (fixed)

[libadblockplus integration cleanup] Don't lower-case the header value

Reported by: fhd Assignee: fhd
Priority: P3 Milestone: Adblock-Plus-for-Android-1.3
Module: Adblock-Plus-for-Android Keywords:
Cc: Blocked By:
Blocking: #7 Platform:
Ready: Confidential: no
Tester: Verified working: no
Review URL(s):

Description (last modified by fhd)

Background

See #7.

What to change

Don't lower-case the header value (http://codereview.adblockplus.org/11172036/diff/1/jni/AndroidWebRequest.cpp#newcode255)

Change History (13)

comment:1 Changed 5 years ago by philll

  • Keywords libadblockplus integration cleanup removed

comment:2 Changed 5 years ago by trev

  • Component changed from Unknown to ABP-Android

comment:3 Changed 5 years ago by philll

  • Owner set to rjeschke
  • Status changed from new to assigned

comment:4 Changed 5 years ago by fhd

  • Reporter changed from philll to fhd

comment:5 Changed 5 years ago by fhd

  • Priority changed from Unknown to P3

comment:6 Changed 5 years ago by fhd

  • Description modified (diff)
  • Priority changed from P3 to Unknown
  • Summary changed from [libadblockplus integration cleanup] Don't lower case the header value to [libadblockplus integration cleanup] Don't lower-case the header value

comment:7 Changed 5 years ago by fhd

  • Owner rjeschke deleted

comment:8 Changed 5 years ago by fhd

  • Status changed from assigned to accepted

comment:9 Changed 5 years ago by fhd

  • Priority changed from Unknown to P3

comment:10 Changed 5 years ago by fhd

  • Owner set to fhd
  • Status changed from accepted to assigned

comment:11 Changed 5 years ago by fhd

  • Resolution set to fixed
  • Status changed from assigned to closed

comment:12 Changed 5 years ago by trev

  • Milestone set to Adblock-Plus-for-Android-next

comment:13 Changed 12 months ago by abpbot

Some commits referencing this issue have landed:

Note: See TracTickets for help on using tickets.