Chromium Code Reviews
Help | Chromium Project | Gerrit Changes | Sign in
(2)

Issue 2759283003: Remove Physical Web optin recording code (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
2 months ago by cco3
Modified:
2 months ago
CC:
chromium-reviews, agrieve+watch_chromium.org
Target Ref:
refs/heads/master
Project:
chromium
Visibility:
Public.

Description

Remove Physical Web optin recording code We used to locally record the number of times we showed an optin notification in order to keep from doing this too many times. Now that we don't show notifications, this code is no longer needed. BUG=680747 Review-Url: https://codereview.chromium.org/2759283003 Cr-Commit-Position: refs/heads/master@{#459148} Committed: https://chromium.googlesource.com/chromium/src/+/c9aec66ca15c3630dcaeff94a0ffdc3068c11baa

Patch Set 1 #

Patch Set 2 : Rebase #

Messages

Total messages: 17 (9 generated)
cco3
Another cleanup CL, Matt
2 months ago (2017-03-20 21:43:06 UTC) #2
mattreynolds
lgtm
2 months ago (2017-03-20 22:26:16 UTC) #4
cco3
Hi Tommy, more cleanup code.
2 months ago (2017-03-20 23:14:01 UTC) #6
nyquist (nychthemeron ping)
lgtm
2 months ago (2017-03-22 19:03:59 UTC) #7
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2759283003/1
2 months ago (2017-03-23 16:42:40 UTC) #9
commit-bot: I haz the power
Try jobs failed on following builders: ios-device on master.tryserver.chromium.mac (JOB_FAILED, http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device/builds/176184) mac_chromium_compile_dbg_ng on master.tryserver.chromium.mac (JOB_FAILED, ...
2 months ago (2017-03-23 16:45:33 UTC) #11
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2759283003/20001
2 months ago (2017-03-23 17:25:42 UTC) #14
commit-bot: I haz the power
2 months ago (2017-03-23 18:17:16 UTC) #17
Message was sent while issue was closed.
Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/c9aec66ca15c3630dcaeff94a0ff...
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld 650457f06