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

Unified Diff: blimp/client/core/contents/blimp_contents_manager.cc

Issue 2624903006: Remove all blimp client code. (Closed)
Patch Set: Update buildbot configuration Created 3 years, 11 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 side-by-side diff with in-line comments
Download patch
Index: blimp/client/core/contents/blimp_contents_manager.cc
diff --git a/blimp/client/core/contents/blimp_contents_manager.cc b/blimp/client/core/contents/blimp_contents_manager.cc
deleted file mode 100644
index ff1c57f1664512492cfd83d97a2928f0aa49fdb2..0000000000000000000000000000000000000000
--- a/blimp/client/core/contents/blimp_contents_manager.cc
+++ /dev/null
@@ -1,143 +0,0 @@
-// Copyright 2016 The Chromium Authors. All rights reserved.
-// Use of this source code is governed by a BSD-style license that can be
-// found in the LICENSE file.
-
-#include "blimp/client/core/contents/blimp_contents_manager.h"
-
-#include "base/bind.h"
-#include "base/memory/ptr_util.h"
-#include "base/threading/thread_task_runner_handle.h"
-#include "blimp/client/core/compositor/blimp_compositor_dependencies.h"
-#include "blimp/client/core/contents/blimp_contents_impl.h"
-#include "blimp/client/core/contents/navigation_feature.h"
-#include "blimp/client/core/contents/tab_control_feature.h"
-#include "blimp/client/core/render_widget/render_widget_feature.h"
-#include "blimp/client/public/contents/blimp_contents_observer.h"
-
-namespace blimp {
-namespace client {
-
-class BlimpContentsManager::BlimpContentsDeletionObserver
- : public BlimpContentsObserver {
- public:
- BlimpContentsDeletionObserver(BlimpContentsManager* blimp_contents_manager,
- BlimpContentsImpl* blimp_contents);
-
- void OnContentsDestroyed() override;
-
- private:
- // The BlimpContentsManager containing this BlimpContentsDeletionObserver
- BlimpContentsManager* blimp_contents_manager_;
-
- DISALLOW_COPY_AND_ASSIGN(BlimpContentsDeletionObserver);
-};
-
-BlimpContentsManager::BlimpContentsDeletionObserver::
- BlimpContentsDeletionObserver(BlimpContentsManager* blimp_contents_manager,
- BlimpContentsImpl* blimp_contents)
- : BlimpContentsObserver(blimp_contents),
- blimp_contents_manager_(blimp_contents_manager) {}
-
-void BlimpContentsManager::BlimpContentsDeletionObserver::
- OnContentsDestroyed() {
- BlimpContents* contents = blimp_contents();
- blimp_contents_manager_->OnContentsDestroyed(contents);
-}
-
-BlimpContentsManager::BlimpContentsManager(
- BlimpCompositorDependencies* blimp_compositor_dependencies,
- ImeFeature* ime_feature,
- NavigationFeature* nav_feature,
- RenderWidgetFeature* render_widget_feature,
- TabControlFeature* tab_control_feature)
- : blimp_compositor_dependencies_(blimp_compositor_dependencies),
- ime_feature_(ime_feature),
- navigation_feature_(nav_feature),
- render_widget_feature_(render_widget_feature),
- tab_control_feature_(tab_control_feature),
- weak_ptr_factory_(this) {}
-
-BlimpContentsManager::~BlimpContentsManager() {}
-
-std::unique_ptr<BlimpContentsImpl> BlimpContentsManager::CreateBlimpContents(
- gfx::NativeWindow window) {
- if (tab_exists_) return nullptr;
- tab_exists_ = true;
-
- int id = CreateBlimpContentsId();
-
- std::unique_ptr<BlimpContentsImpl> new_contents =
- base::MakeUnique<BlimpContentsImpl>(
- id, window, blimp_compositor_dependencies_, ime_feature_,
- navigation_feature_, render_widget_feature_, tab_control_feature_);
-
- // Create an observer entry for the contents.
- std::unique_ptr<BlimpContentsDeletionObserver> observer =
- base::MakeUnique<BlimpContentsDeletionObserver>(this, new_contents.get());
- observer_map_.insert(
- std::pair<int, std::unique_ptr<BlimpContentsDeletionObserver>>(
- id, std::move(observer)));
-
- // Notifies the engine that we've created a new BlimpContents.
- tab_control_feature_->CreateTab(id);
-
- return new_contents;
-}
-
-BlimpContentsImpl* BlimpContentsManager::GetBlimpContents(int id) {
- if (observer_map_.find(id) == observer_map_.end()) return nullptr;
-
- BlimpContentsDeletionObserver* observer = observer_map_.at(id).get();
- // If the BlimpContents that the observer tracks is empty, it means
- // OnContentsDestroyed was called on this observer, but the task to erase
- // the observer from the map hasn't been run.
- if (observer->blimp_contents())
- return static_cast<BlimpContentsImpl*>(observer->blimp_contents());
-
- return nullptr;
-}
-
-std::vector<BlimpContentsImpl*>
-BlimpContentsManager::GetAllActiveBlimpContents() {
- std::vector<BlimpContentsImpl*> all_blimp_contents;
- for (const auto& item : observer_map_) {
- BlimpContentsImpl* blimp_contents =
- static_cast<BlimpContentsImpl*>(item.second.get()->blimp_contents());
- if (!blimp_contents) {
- continue;
- }
- all_blimp_contents.push_back(blimp_contents);
- }
- return all_blimp_contents;
-}
-
-int BlimpContentsManager::CreateBlimpContentsId() {
- return next_blimp_contents_id_++;
-}
-
-void BlimpContentsManager::EraseObserverFromMap(int id) {
- observer_map_.erase(id);
-}
-
-void BlimpContentsManager::OnContentsDestroyed(BlimpContents* contents) {
- DCHECK(tab_exists_);
- tab_exists_ = false;
-
- int id = static_cast<BlimpContentsImpl*>(contents)->id();
-
- // Notify the engine that we've destroyed the BlimpContents.
- tab_control_feature_->CloseTab(id);
-
- // Destroy the observer entry from the observer_map_.
- DCHECK(base::ThreadTaskRunnerHandle::Get());
- base::ThreadTaskRunnerHandle::Get()->PostTask(
- FROM_HERE, base::Bind(&BlimpContentsManager::EraseObserverFromMap,
- this->GetWeakPtr(), id));
-}
-
-base::WeakPtr<BlimpContentsManager> BlimpContentsManager::GetWeakPtr() {
- return weak_ptr_factory_.GetWeakPtr();
-}
-
-} // namespace client
-} // namespace blimp
« no previous file with comments | « blimp/client/core/contents/blimp_contents_manager.h ('k') | blimp/client/core/contents/blimp_contents_manager_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698