Remove unnecessary kEditable state from PDF text field AXNode

Bug: 1030242
Change-Id: I84a837b95eb48f05c144bf47d557bbcf303fa78c
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2135101
Commit-Queue: Dominic Mazzoni <dmazzoni@chromium.org>
Reviewed-by: Dominic Mazzoni <dmazzoni@chromium.org>
Cr-Commit-Position: refs/heads/master@{#756415}

components/pdf/renderer/pdf_accessibility_tree.cc

@@ -977,8 +977,6 @@ ui::AXNodeData* PdfAccessibilityTree::CreateTextFieldNode(
     text_field_node->AddState(ax::mojom::State::kRequired);
   if (text_field.is_password)
     text_field_node->AddState(ax::mojom::State::kProtected);
-  if (!text_field.is_read_only)
-    text_field_node->AddState(ax::mojom::State::kEditable);
   text_field_node->relative_bounds.bounds =
       PpFloatRectToGfxRectF(text_field.bounds);
   return text_field_node;

components/pdf/renderer/pdf_accessibility_tree_browsertest.cc

@@ -630,7 +630,6 @@ TEST_F(PdfAccessibilityTreeTest, TestTextFieldNodeCreation) {
                         ax::mojom::StringAttribute::kValue));
   EXPECT_FALSE(text_field_node->data().HasState(ax::mojom::State::kRequired));
   EXPECT_FALSE(text_field_node->data().HasState(ax::mojom::State::kProtected));
-  EXPECT_TRUE(text_field_node->data().HasState(ax::mojom::State::kEditable));
   EXPECT_NE(ax::mojom::Restriction::kReadOnly,
             text_field_node->data().GetRestriction());
   EXPECT_EQ(gfx::RectF(1.0f, 1.0f, 5.0f, 6.0f),
@@ -646,7 +645,6 @@ TEST_F(PdfAccessibilityTreeTest, TestTextFieldNodeCreation) {
                           ax::mojom::StringAttribute::kValue));
   EXPECT_TRUE(text_field_node->data().HasState(ax::mojom::State::kRequired));
   EXPECT_TRUE(text_field_node->data().HasState(ax::mojom::State::kProtected));
-  EXPECT_FALSE(text_field_node->data().HasState(ax::mojom::State::kEditable));
   EXPECT_EQ(ax::mojom::Restriction::kReadOnly,
             text_field_node->data().GetRestriction());
   EXPECT_EQ(gfx::RectF(1.0f, 10.0f, 5.0f, 6.0f),