Source/modules/serviceworkers/ServiceWorker.h - Issue 330173003: Make ServiceWorker an ActiveDOMObject

Side by Side Diff: Source/modules/serviceworkers/ServiceWorker.h

Issue 330173003: Make ServiceWorker an ActiveDOMObject (Closed) Base URL: svn://svn.chromium.org/blink/trunk

Patch Set: fix hasPendingActivity Created 6 years, 6 months ago

Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.

Jump to:

View unified diff | Download patch | Annotate | Revision Log

« LayoutTests/http/tests/serviceworker/service-worker-gc.html ('K') | « LayoutTests/virtual/serviceworker/http/tests/serviceworker/service-worker-gc-expected.txt ('k') | Source/modules/serviceworkers/ServiceWorker.cpp » ('j') | Source/modules/serviceworkers/ServiceWorker.cpp » ('J')
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')

OLD	NEW
1 /*	1 /*

2 * Copyright (C) 2013 Google Inc. All rights reserved.	2 * Copyright (C) 2013 Google Inc. All rights reserved.

3 *	3 *

4 * Redistribution and use in source and binary forms, with or without	4 * Redistribution and use in source and binary forms, with or without

5 * modification, are permitted provided that the following conditions are	5 * modification, are permitted provided that the following conditions are

6 * met:	6 * met:

7 *	7 *

8 * * Redistributions of source code must retain the above copyright	8 * * Redistributions of source code must retain the above copyright

9 * notice, this list of conditions and the following disclaimer.	9 * notice, this list of conditions and the following disclaimer.

10 * * Redistributions in binary form must reproduce the above	10 * * Redistributions in binary form must reproduce the above

(...skipping 56 matching lines...) Expand 10 before \| Expand all \| Expand 10 after Loading...
67 const AtomicString& state() const;	67 const AtomicString& state() const;

68 DEFINE_ATTRIBUTE_EVENT_LISTENER(statechange);	68 DEFINE_ATTRIBUTE_EVENT_LISTENER(statechange);

69	69

70 // WebServiceWorkerProxy overrides.	70 // WebServiceWorkerProxy overrides.

71 virtual bool isReady() OVERRIDE;	71 virtual bool isReady() OVERRIDE;

72 virtual void dispatchStateChangeEvent() OVERRIDE;	72 virtual void dispatchStateChangeEvent() OVERRIDE;

73	73

74 // AbstractWorker overrides.	74 // AbstractWorker overrides.

75 virtual const AtomicString& interfaceName() const OVERRIDE;	75 virtual const AtomicString& interfaceName() const OVERRIDE;

76	76

	77 // ActiveDOMObject overrides.
	dominicc (has gone to gerrit) 2014/06/12 22:23:38 Let's make these private? Let's make these private? falken 2014/06/13 19:22:59 Done. Show quoted text On 2014/06/12 22:23:38, dominicc wrote: > Let's make these private? Done.
	78 virtual bool hasPendingActivity() const OVERRIDE;

	79 virtual void stop() OVERRIDE;

	80

77 private:	81 private:

78 class ThenFunction;	82 class ThenFunction;

79	83

80 static PassRefPtr<ServiceWorker> create(ExecutionContext*, PassOwnPtr<blink: :WebServiceWorker>);	84 static PassRefPtr<ServiceWorker> create(ExecutionContext*, PassOwnPtr<blink: :WebServiceWorker>);

81 ServiceWorker(ExecutionContext*, PassOwnPtr<blink::WebServiceWorker>);	85 ServiceWorker(ExecutionContext*, PassOwnPtr<blink::WebServiceWorker>);

82 void onPromiseResolved();	86 void onPromiseResolved();

83 void waitOnPromise(ScriptPromise);	87 void waitOnPromise(ScriptPromise);

84	88

85 OwnPtr<blink::WebServiceWorker> m_outerWorker;	89 OwnPtr<blink::WebServiceWorker> m_outerWorker;

86 bool m_isPromisePending;	90 bool m_isPromisePending;

	91

	92 // Whether ActiveDOMObject::stop has been called.
	dominicc (has gone to gerrit) 2014/06/12 22:23:38 We can probably omit this comment. We can probably omit this comment. falken 2014/06/13 19:22:59 Done. Show quoted text On 2014/06/12 22:23:38, dominicc wrote: > We can probably omit this comment. Done.
	93 bool m_contextStopped;

87 };	94 };

88	95

89 } // namespace WebCore	96 } // namespace WebCore

90	97

91 #endif // ServiceWorker_h	98 #endif // ServiceWorker_h

OLD	NEW