Title: [138742] trunk/Source/WebKit/chromium
- Revision
- 138742
- Author
- pilg...@chromium.org
- Date
- 2013-01-03 14:26:02 -0800 (Thu, 03 Jan 2013)
Log Message
[Chromium] Remove all references to sharedWorkerRepository()
https://bugs.webkit.org/show_bug.cgi?id=104704
Reviewed by Adam Barth.
Now that Chromium calls setSharedWorkerRepository upon
initializing WebKit (
https://codereview.chromium.org/10990121/ and
https://codereview.chromium.org/11576028/ ) and no longer
tries to override sharedWorkerRepository, it is safe to remove
all references to this function.
* public/platform/WebKitPlatformSupport.h:
(WebKit):
(WebKit::WebKitPlatformSupport::idbFactory):
* src/SharedWorkerRepository.cpp:
(WebKit::sharedWorkerRepository):
(WebCore::SharedWorkerRepository::isAvailable):
Modified Paths
Diff
Modified: trunk/Source/WebKit/chromium/ChangeLog (138741 => 138742)
--- trunk/Source/WebKit/chromium/ChangeLog 2013-01-03 22:23:40 UTC (rev 138741)
+++ trunk/Source/WebKit/chromium/ChangeLog 2013-01-03 22:26:02 UTC (rev 138742)
@@ -1,3 +1,24 @@
+2013-01-03 Mark Pilgrim <pilg...@chromium.org>
+
+ [Chromium] Remove all references to sharedWorkerRepository()
+ https://bugs.webkit.org/show_bug.cgi?id=104704
+
+ Reviewed by Adam Barth.
+
+ Now that Chromium calls setSharedWorkerRepository upon
+ initializing WebKit (
+ https://codereview.chromium.org/10990121/ and
+ https://codereview.chromium.org/11576028/ ) and no longer
+ tries to override sharedWorkerRepository, it is safe to remove
+ all references to this function.
+
+ * public/platform/WebKitPlatformSupport.h:
+ (WebKit):
+ (WebKit::WebKitPlatformSupport::idbFactory):
+ * src/SharedWorkerRepository.cpp:
+ (WebKit::sharedWorkerRepository):
+ (WebCore::SharedWorkerRepository::isAvailable):
+
2013-01-03 Tommy Widenflycht <tom...@google.com>
MediaStream API: Add a missing state to RTCPeerConnection
Modified: trunk/Source/WebKit/chromium/public/platform/WebKitPlatformSupport.h (138741 => 138742)
--- trunk/Source/WebKit/chromium/public/platform/WebKitPlatformSupport.h 2013-01-03 22:23:40 UTC (rev 138741)
+++ trunk/Source/WebKit/chromium/public/platform/WebKitPlatformSupport.h 2013-01-03 22:26:02 UTC (rev 138742)
@@ -36,7 +36,6 @@
namespace WebKit {
class WebIDBFactory; // FIXME: Does this belong in platform?
-class WebSharedWorkerRepository; // FIXME: Does this belong in platform?
// FIXME: Eventually all these API will need to move to WebKit::Platform.
class WebKitPlatformSupport : public Platform {
@@ -45,11 +44,6 @@
virtual WebIDBFactory* idbFactory() { return 0; }
-
- // Shared Workers ------------------------------------------------------
-
- virtual WebSharedWorkerRepository* sharedWorkerRepository() { return 0; }
-
protected:
~WebKitPlatformSupport() { }
};
Modified: trunk/Source/WebKit/chromium/src/SharedWorkerRepository.cpp (138741 => 138742)
--- trunk/Source/WebKit/chromium/src/SharedWorkerRepository.cpp 2013-01-03 22:23:40 UTC (rev 138741)
+++ trunk/Source/WebKit/chromium/src/SharedWorkerRepository.cpp 2013-01-03 22:26:02 UTC (rev 138742)
@@ -69,15 +69,8 @@
static WebSharedWorkerRepository* sharedWorkerRepository()
{
- WebSharedWorkerRepository* repository;
-
- repository = s_sharedWorkerRepository;
- if (!repository) {
- repository = webKitPlatformSupport()->sharedWorkerRepository();
- setSharedWorkerRepository(repository);
- }
-
- return repository;
+ // Will only be non-zero if the embedder has set the shared worker repository upon initialization. Nothing in WebKit sets this.
+ return s_sharedWorkerRepository;
}
}
@@ -218,8 +211,6 @@
bool SharedWorkerRepository::isAvailable()
{
- // Allow the WebKitPlatformSupport to determine if SharedWorkers
- // are available.
return WebKit::sharedWorkerRepository();
}
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
http://lists.webkit.org/mailman/listinfo/webkit-changes
