Removed unregisterTerminationObserver from DatabaseContext::contextDestroyed

Method DatabaseContext::contextDestroyed always calls from ExecutionContext destructor. In the case when the context is WorkerGlobalScope - from WorkerGlobalScope destructor. This unregister call does nothing in this situation.
And it looks dangerous when we try to use the methods of the object in destructor. E.g. if we make the method ExecutionContext::isWorkerGlobalScope virtual the program will crash in DatabaseContext::contextDestroyed.

R=tkent@chromium.org,vsevik@chromium.org

Review URL: https://codereview.chromium.org/551373003

git-svn-id: svn://svn.chromium.org/blink/trunk@181711 bbb929c8-8fbe-4397-9dbb-9b2b20218538

third_party/WebKit/Source/core/workers/WorkerGlobalScope.cpp

@@ -203,13 +203,6 @@ void WorkerGlobalScope::registerTerminationObserver(TerminationObserver* observe
     m_terminationObserver = observer;
 }
 
-void WorkerGlobalScope::unregisterTerminationObserver(TerminationObserver* observer)
-{
-    ASSERT(observer);
-    ASSERT(m_terminationObserver == observer);
-    m_terminationObserver = 0;
-}
-
 void WorkerGlobalScope::wasRequestedToTerminate()
 {
     if (m_terminationObserver)

third_party/WebKit/Source/core/workers/WorkerGlobalScope.h

@@ -94,7 +94,6 @@ public:
         virtual void wasRequestedToTerminate() = 0;
     };
     void registerTerminationObserver(TerminationObserver*);
-    void unregisterTerminationObserver(TerminationObserver*);
     void wasRequestedToTerminate();
 
     void dispose();

third_party/WebKit/Source/modules/webdatabase/DatabaseContext.cpp

@@ -133,8 +133,6 @@ void DatabaseContext::contextDestroyed()
 {
     RefPtrWillBeRawPtr<DatabaseContext> protector(this);
     stopDatabases();
-    if (executionContext()->isWorkerGlobalScope())
-        toWorkerGlobalScope(executionContext())->unregisterTerminationObserver(this);
     DatabaseManager::manager().unregisterDatabaseContext(this);
     ActiveDOMObject::contextDestroyed();
 }