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

Unified Diff: components/ui_devtools/views/ui_devtools_dom_agent.h

Issue 2899783002: Move DevTools out of ash and turn it to a component. (Closed)
Patch Set: NON_EXPORTED_BASE base class Created 3 years, 6 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: components/ui_devtools/views/ui_devtools_dom_agent.h
diff --git a/ash/devtools/ash_devtools_dom_agent.h b/components/ui_devtools/views/ui_devtools_dom_agent.h
similarity index 83%
rename from ash/devtools/ash_devtools_dom_agent.h
rename to components/ui_devtools/views/ui_devtools_dom_agent.h
index 8a171fea93b71589b06cfba33769106936e97637..b5ae551f4842ee25102f96df966d9563f28a2392 100644
--- a/ash/devtools/ash_devtools_dom_agent.h
+++ b/components/ui_devtools/views/ui_devtools_dom_agent.h
@@ -2,14 +2,12 @@
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
-#ifndef ASH_DEVTOOLS_ASH_DEVTOOLS_DOM_AGENT_H_
-#define ASH_DEVTOOLS_ASH_DEVTOOLS_DOM_AGENT_H_
+#ifndef COMPONENTS_UI_DEVTOOLS_VIEWS_UI_DEVTOOLS_DOM_AGENT_H_
+#define COMPONENTS_UI_DEVTOOLS_VIEWS_UI_DEVTOOLS_DOM_AGENT_H_
-#include "ash/ash_export.h"
-#include "ash/devtools/ui_element_delegate.h"
-#include "base/macros.h"
#include "components/ui_devtools/DOM.h"
#include "components/ui_devtools/devtools_base_agent.h"
+#include "components/ui_devtools/views/ui_element_delegate.h"
#include "ui/aura/env_observer.h"
#include "ui/views/view.h"
#include "ui/views/widget/widget.h"
@@ -18,25 +16,25 @@ namespace aura {
class Window;
}
-namespace ash {
+namespace ui {
namespace devtools {
class UIElement;
-class ASH_EXPORT AshDevToolsDOMAgentObserver {
+class UIDevToolsDOMAgentObserver {
public:
// TODO(thanhph): Use UIElement* as input argument instead.
virtual void OnNodeBoundsChanged(int node_id) = 0;
};
-class ASH_EXPORT AshDevToolsDOMAgent
+class UIDevToolsDOMAgent
: public NON_EXPORTED_BASE(ui::devtools::UiDevToolsBaseAgent<
sadrul 2017/06/14 03:25:57 You should not need NON_EXPORTED_BASE.
thanhph 2017/06/14 15:26:37 Done.
ui::devtools::protocol::DOM::Metainfo>),
public UIElementDelegate,
public aura::EnvObserver {
public:
- AshDevToolsDOMAgent();
- ~AshDevToolsDOMAgent() override;
+ UIDevToolsDOMAgent();
+ ~UIDevToolsDOMAgent() override;
// DOM::Backend:
ui::devtools::protocol::Response disable() override;
@@ -55,8 +53,8 @@ class ASH_EXPORT AshDevToolsDOMAgent
void OnUIElementBoundsChanged(UIElement* ui_element) override;
bool IsHighlightingWindow(aura::Window* window) override;
- void AddObserver(AshDevToolsDOMAgentObserver* observer);
- void RemoveObserver(AshDevToolsDOMAgentObserver* observer);
+ void AddObserver(UIDevToolsDOMAgentObserver* observer);
+ void RemoveObserver(UIDevToolsDOMAgentObserver* observer);
UIElement* GetElementFromNodeId(int node_id);
UIElement* window_element_root() const { return window_element_root_.get(); };
const std::vector<aura::Window*>& root_windows() const {
@@ -98,12 +96,12 @@ class ASH_EXPORT AshDevToolsDOMAgent
std::unordered_map<int, UIElement*> node_id_to_ui_element_;
std::unique_ptr<views::Widget> widget_for_highlighting_;
std::vector<aura::Window*> root_windows_;
- base::ObserverList<AshDevToolsDOMAgentObserver> observers_;
+ base::ObserverList<UIDevToolsDOMAgentObserver> observers_;
- DISALLOW_COPY_AND_ASSIGN(AshDevToolsDOMAgent);
+ DISALLOW_COPY_AND_ASSIGN(UIDevToolsDOMAgent);
};
} // namespace devtools
-} // namespace ash
+} // namespace ui
-#endif // ASH_DEVTOOLS_ASH_DEVTOOLS_DOM_AGENT_H_
+#endif // COMPONENTS_UI_DEVTOOLS_VIEWS_UI_DEVTOOLS_DOM_AGENT_H_

Powered by Google App Engine
This is Rietveld 408576698