Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Issue 4875867805188096: issue #660 - HTTP cache api v2 fixes (Closed)

Created:
July 4, 2014, 8:54 a.m. by saroyanm
Modified:
July 5, 2014, 6:35 a.m.
Reviewers:
Wladimir Palant
Visibility:
Public.

Description

This issue is related to current ticket: https://issues.adblockplus.org/ticket/660#comment:13

Patch Set 1 #

Total comments: 4

Patch Set 2 : #

Unified diffs Side-by-side diffs Delta from patch set Stats (+8 lines, -3 lines) Patch
M chrome/content/ui/sidebar.js View 1 3 chunks +8 lines, -3 lines 0 comments Download

Messages

Total messages: 4
saroyanm
Wladimir can you please have a look.
July 4, 2014, 9 a.m. (2014-07-04 09:00:17 UTC) #1
Wladimir Palant
http://codereview.adblockplus.org/4875867805188096/diff/5629499534213120/chrome/content/ui/sidebar.js File chrome/content/ui/sidebar.js (left): http://codereview.adblockplus.org/4875867805188096/diff/5629499534213120/chrome/content/ui/sidebar.js#oldcode324 chrome/content/ui/sidebar.js:324: onCacheEntryCheck: function (entry, appCache) Please don't remove the parameters ...
July 4, 2014, 6:09 p.m. (2014-07-04 18:09:00 UTC) #2
saroyanm
New patch uploaded. http://codereview.adblockplus.org/4875867805188096/diff/5629499534213120/chrome/content/ui/sidebar.js File chrome/content/ui/sidebar.js (left): http://codereview.adblockplus.org/4875867805188096/diff/5629499534213120/chrome/content/ui/sidebar.js#oldcode324 chrome/content/ui/sidebar.js:324: onCacheEntryCheck: function (entry, appCache) On 2014/07/04 ...
July 4, 2014, 7:32 p.m. (2014-07-04 19:32:12 UTC) #3
Wladimir Palant
July 5, 2014, 6:17 a.m. (2014-07-05 06:17:51 UTC) #4
LGTM

Powered by Google App Engine
This is Rietveld