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

Unified Diff: ui/views/accessibility/native_view_accessibility_unittest.cc

Issue 2715543003: Views a11y: Implement AXPlatformNode::FromNativeViewAccessible on all platforms. (Closed)
Patch Set: Make GetForView protected. Created 3 years, 10 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: ui/views/accessibility/native_view_accessibility_unittest.cc
diff --git a/ui/views/accessibility/native_view_accessibility_unittest.cc b/ui/views/accessibility/native_view_accessibility_unittest.cc
index eecbff06989d23b22813e8b76bfd5eff8c9dd24e..c0c6af372728a50a9076e24513a3792e7d2e0bca 100644
--- a/ui/views/accessibility/native_view_accessibility_unittest.cc
+++ b/ui/views/accessibility/native_view_accessibility_unittest.cc
@@ -45,11 +45,11 @@ class NativeViewAccessibilityTest : public ViewsTestBase {
button_ = new TestButton();
button_->SetSize(gfx::Size(20, 20));
- button_accessibility_ = NativeViewAccessibility::Create(button_);
+ button_accessibility_ = NativeViewAccessibility::CreateForView(button_);
label_ = new Label();
button_->AddChildView(label_);
- label_accessibility_ = NativeViewAccessibility::Create(label_);
+ label_accessibility_ = NativeViewAccessibility::CreateForView(label_);
widget_->GetContentsView()->AddChildView(button_);
widget_->Show();
@@ -58,19 +58,15 @@ class NativeViewAccessibilityTest : public ViewsTestBase {
void TearDown() override {
if (!widget_->IsClosed())
widget_->Close();
- button_accessibility_->Destroy();
- button_accessibility_ = NULL;
- label_accessibility_->Destroy();
- label_accessibility_ = NULL;
ViewsTestBase::TearDown();
}
protected:
views::Widget* widget_;
TestButton* button_;
- NativeViewAccessibility* button_accessibility_;
+ std::unique_ptr<NativeViewAccessibility> button_accessibility_;
Label* label_;
- NativeViewAccessibility* label_accessibility_;
+ std::unique_ptr<NativeViewAccessibility> label_accessibility_;
};
TEST_F(NativeViewAccessibilityTest, RoleShouldMatch) {

Powered by Google App Engine
This is Rietveld 408576698