Removed WebFrameClient.cpp from Source/web

third_party/WebKit/Source/web/WebEmbeddedWorkerImpl.cpp

 /*
  * Copyright (C) 2013 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 303 matching lines @@
   // to have same origin as the worker so the loading checks work correctly.
   CString content("");
   RefPtr<SharedBuffer> buffer(
       SharedBuffer::Create(content.data(), content.length()));
   loading_shadow_page_ = true;
   main_frame_->GetFrame()->Loader().Load(
       FrameLoadRequest(0, ResourceRequest(worker_start_data_.script_url),
                        SubstituteData(buffer, "text/html", "UTF-8", KURL())));
 }
 
+void WebEmbeddedWorkerImpl::FrameDetached(WebLocalFrame* frame,
+                                          DetachType type) {
+  if (type == DetachType::kRemove && frame->Parent())
+    frame->Parent()->RemoveChild(frame);

[kinuko.yasuda, 2017/05/11 04:07:17]

I think we can remove this (and possibly below) and just have DCHECK, but that
can be done separately. 

[sashab, 2017/05/15 02:18:02]

Added TODOs for both :) Will do in follow-up patch.

[dcheng, 2017/05/15 03:58:47]

Given that we fixed up the other places (in WebViewPlugin), let's just go ahead
and do it here too?

[sashab, 2017/05/15 06:23:48]

Done :)

+
+  if (frame->FrameWidget())
+    frame->FrameWidget()->Close();
+
+  frame->Close();
+}
+
 void WebEmbeddedWorkerImpl::DidFinishDocumentLoad() {
   DCHECK(!main_script_loader_);
   DCHECK(main_frame_);
   DCHECK(worker_context_client_);
   DCHECK(loading_shadow_page_);
   DCHECK(!asked_to_terminate_);
   loading_shadow_page_ = false;
   main_frame_->DataSource()->SetServiceWorkerNetworkProvider(
       worker_context_client_->CreateServiceWorkerNetworkProvider());
   main_script_loader_ = WorkerScriptLoader::Create();
@@ skipping 139 matching lines @@
   // populate the task runners with null document not to confuse the frame
   // scheduler (which will end up using the thread's default task runner).
   worker_thread_->Start(std::move(startup_data),
                         ParentFrameTaskRunners::Create(nullptr));
 
   worker_inspector_proxy_->WorkerThreadCreated(document, worker_thread_.get(),
                                                script_url);
 }
 
 }  // namespace blink