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

Unified Diff: chrome/browser/ui/ash/accessibility/ax_tree_source_aura_unittest.cc

Issue 1003283002: Enable chrome.automation.getDesktop on all aura platforms. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 9 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: chrome/browser/ui/ash/accessibility/ax_tree_source_aura_unittest.cc
diff --git a/chrome/browser/ui/ash/accessibility/ax_tree_source_ash_unittest.cc b/chrome/browser/ui/ash/accessibility/ax_tree_source_aura_unittest.cc
similarity index 68%
rename from chrome/browser/ui/ash/accessibility/ax_tree_source_ash_unittest.cc
rename to chrome/browser/ui/ash/accessibility/ax_tree_source_aura_unittest.cc
index 2b42f5c4dc5b0cfb17ebb646e85da9ea37e5c054..a0b1c3568b8b8a68123fb460d9c2684cac902cd8 100644
--- a/chrome/browser/ui/ash/accessibility/ax_tree_source_ash_unittest.cc
+++ b/chrome/browser/ui/ash/accessibility/ax_tree_source_aura_unittest.cc
@@ -6,7 +6,7 @@
#include "ash/test/ash_test_base.h"
#include "base/strings/utf_string_conversions.h"
-#include "chrome/browser/ui/ash/accessibility/ax_tree_source_ash.h"
+#include "chrome/browser/ui/aura/accessibility/ax_tree_source_aura.h"
#include "testing/gtest/include/gtest/gtest.h"
#include "ui/accessibility/ax_enums.h"
#include "ui/accessibility/ax_node.h"
@@ -39,10 +39,10 @@ size_t GetSize(AXAuraObjWrapper* tree) {
return count;
}
-class AXTreeSourceAshTest : public ash::test::AshTestBase {
+class AXTreeSourceAuraTest : public ash::test::AshTestBase {
public:
- AXTreeSourceAshTest() {}
- ~AXTreeSourceAshTest() override {}
+ AXTreeSourceAuraTest() {}
+ ~AXTreeSourceAuraTest() override {}
void SetUp() override {
AshTestBase::SetUp();
@@ -66,11 +66,11 @@ class AXTreeSourceAshTest : public ash::test::AshTestBase {
Textfield* textfield_;
private:
- DISALLOW_COPY_AND_ASSIGN(AXTreeSourceAshTest);
+ DISALLOW_COPY_AND_ASSIGN(AXTreeSourceAuraTest);
};
-TEST_F(AXTreeSourceAshTest, Accessors) {
- AXTreeSourceAsh ax_tree;
+TEST_F(AXTreeSourceAuraTest, Accessors) {
+ AXTreeSourceAura ax_tree;
ASSERT_TRUE(ax_tree.GetRoot());
// ID's should start at 1 and there should be a root.
@@ -102,42 +102,8 @@ TEST_F(AXTreeSourceAshTest, Accessors) {
ASSERT_EQ(ax_tree.GetRoot(), test_root);
}
-TEST_F(AXTreeSourceAshTest, Serialization) {
dmazzoni 2015/03/26 18:44:58 Are we losing this test too?
David Tseng 2015/03/26 23:20:47 Fixed this one.
- AXTreeSourceAsh ax_tree;
- ui::AXTreeSerializer<AXAuraObjWrapper*> ax_serializer(&ax_tree);
- ui::AXTreeUpdate out_update;
-
- // This is the initial serialization.
- ax_serializer.SerializeChanges(ax_tree.GetRoot(), &out_update);
-
- // We should get an update per node.
- ASSERT_EQ(GetSize(ax_tree.GetRoot()), out_update.nodes.size());
-
- // Try removing some child views and re-adding.
- content_->RemoveAllChildViews(false /* delete_children */);
- content_->AddChildView(textfield_);
-
- // Grab the textfield since serialization only walks up the tree (not down
- // from root).
- AXAuraObjWrapper* textfield_wrapper =
- AXAuraObjCache::GetInstance()->GetOrCreate(textfield_);
-
- // Now, re-serialize.
- ui::AXTreeUpdate out_update2;
- ax_serializer.SerializeChanges(textfield_wrapper, &out_update2);
-
- // We should have far fewer updates this time around.
- ASSERT_EQ(2U, out_update2.nodes.size());
- ASSERT_EQ(ui::AX_ROLE_CLIENT,
- out_update2.nodes[0].role);
-
- ASSERT_EQ(textfield_wrapper->GetID(), out_update2.nodes[1].id);
- ASSERT_EQ(ui::AX_ROLE_TEXT_FIELD,
- out_update2.nodes[1].role);
-}
-
-TEST_F(AXTreeSourceAshTest, DoDefault) {
- AXTreeSourceAsh ax_tree;
+TEST_F(AXTreeSourceAuraTest, DoDefault) {
+ AXTreeSourceAura ax_tree;
// Grab a wrapper to |DoDefault| (click).
AXAuraObjWrapper* textfield_wrapper =
@@ -149,8 +115,8 @@ TEST_F(AXTreeSourceAshTest, DoDefault) {
ASSERT_TRUE(textfield_->HasFocus());
}
-TEST_F(AXTreeSourceAshTest, Focus) {
- AXTreeSourceAsh ax_tree;
+TEST_F(AXTreeSourceAuraTest, Focus) {
+ AXTreeSourceAura ax_tree;
// Grab a wrapper to focus.
AXAuraObjWrapper* textfield_wrapper =

Powered by Google App Engine
This is Rietveld 408576698