Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1003)

Side by Side Diff: third_party/WebKit/Source/web/WebSharedWorkerImpl.cpp

Issue 1984593002: Remove OwnPtr::release() calls in web/ & wtf/. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Merge with trunk. Created 4 years, 7 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
OLDNEW
1 /* 1 /*
2 * Copyright (C) 2009 Google Inc. All rights reserved. 2 * Copyright (C) 2009 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 328 matching lines...) Expand 10 before | Expand all | Expand 10 after
339 contentSecurityPolicy ? contentSecurityPolicy->headers().get() : nullptr , 339 contentSecurityPolicy ? contentSecurityPolicy->headers().get() : nullptr ,
340 starterOrigin, 340 starterOrigin,
341 workerClients, 341 workerClients,
342 m_mainScriptLoader->responseAddressSpace(), 342 m_mainScriptLoader->responseAddressSpace(),
343 m_mainScriptLoader->originTrialTokens()); 343 m_mainScriptLoader->originTrialTokens());
344 m_loaderProxy = WorkerLoaderProxy::create(this); 344 m_loaderProxy = WorkerLoaderProxy::create(this);
345 m_workerThread = SharedWorkerThread::create(m_name, m_loaderProxy, *this); 345 m_workerThread = SharedWorkerThread::create(m_name, m_loaderProxy, *this);
346 InspectorInstrumentation::scriptImported(m_loadingDocument.get(), m_mainScri ptLoader->identifier(), m_mainScriptLoader->script()); 346 InspectorInstrumentation::scriptImported(m_loadingDocument.get(), m_mainScri ptLoader->identifier(), m_mainScriptLoader->script());
347 m_mainScriptLoader.clear(); 347 m_mainScriptLoader.clear();
348 348
349 workerThread()->start(startupData.release()); 349 workerThread()->start(std::move(startupData));
350 m_workerInspectorProxy->workerThreadCreated(toDocument(m_loadingDocument.get ()), workerThread(), m_url); 350 m_workerInspectorProxy->workerThreadCreated(toDocument(m_loadingDocument.get ()), workerThread(), m_url);
351 m_client->workerScriptLoaded(); 351 m_client->workerScriptLoaded();
352 } 352 }
353 353
354 void WebSharedWorkerImpl::terminateWorkerContext() 354 void WebSharedWorkerImpl::terminateWorkerContext()
355 { 355 {
356 terminateWorkerThread(); 356 terminateWorkerThread();
357 } 357 }
358 358
359 void WebSharedWorkerImpl::pauseWorkerContextOnStart() 359 void WebSharedWorkerImpl::pauseWorkerContextOnStart()
(...skipping 31 matching lines...) Expand 10 before | Expand all | Expand 10 after
391 if (devtoolsAgent) 391 if (devtoolsAgent)
392 devtoolsAgent->dispatchOnInspectorBackend(sessionId, callId, method, mes sage); 392 devtoolsAgent->dispatchOnInspectorBackend(sessionId, callId, method, mes sage);
393 } 393 }
394 394
395 WebSharedWorker* WebSharedWorker::create(WebSharedWorkerClient* client) 395 WebSharedWorker* WebSharedWorker::create(WebSharedWorkerClient* client)
396 { 396 {
397 return new WebSharedWorkerImpl(client); 397 return new WebSharedWorkerImpl(client);
398 } 398 }
399 399
400 } // namespace blink 400 } // namespace blink
OLDNEW
« no previous file with comments | « third_party/WebKit/Source/web/WebEmbeddedWorkerImpl.cpp ('k') | third_party/WebKit/Source/web/WebViewImpl.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698