ServiceWorker: Remove macros that were used to introduce SWRegistration

ServiceWorker: Remove macros that were used to introduce SWRegistration

These macros are no longer necessary after https://codereview.chromium.org/466723002/


BUG=396400
TEST=content_unittests --gtest_filter=ServiceWorker*

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=290224

[nhiroki, 2014-08-18 06:26:29]

Ptal, thanks!

[falken, 2014-08-18 06:41:39]

lgtm, yay!

https://codereview.chromium.org/479123002/diff/1/content/child/service_worker...
File content/child/service_worker/service_worker_dispatcher.cc (right):

https://codereview.chromium.org/479123002/diff/1/content/child/service_worker...
content/child/service_worker/service_worker_dispatcher.cc:253: // We should
adopt and destroy an unused handle ref.
not from this patch, but this comment says "should" which makes it unclear
whether it's a TODO or explaining the code, especially since it's above a TODO.
I'd just say "The handle ref is unused, so adopt and destroy it."

[nhiroki, 2014-08-18 06:45:34]

https://codereview.chromium.org/479123002/diff/1/content/child/service_worker...
File content/child/service_worker/service_worker_dispatcher.cc (right):

https://codereview.chromium.org/479123002/diff/1/content/child/service_worker...
content/child/service_worker/service_worker_dispatcher.cc:253: // We should
adopt and destroy an unused handle ref.
On 2014/08/18 06:41:39, falken wrote:
> not from this patch, but this comment says "should" which makes it unclear
> whether it's a TODO or explaining the code, especially since it's above a
TODO.
> I'd just say "The handle ref is unused, so adopt and destroy it."

Done. Thanks!

[nhiroki, 2014-08-18 06:45:38]

The CQ bit was checked by nhiroki@chromium.org

[commit-bot: I haz the power, 2014-08-18 06:46:50]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/nhiroki@chromium.org/479123002/20001

[commit-bot: I haz the power, 2014-08-18 08:05:06]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  linux_chromium_gn_rel on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[commit-bot: I haz the power, 2014-08-18 08:14:27]

Committed patchset #2 (20001) as 290224