Revert of Extracting placeholder information from Webkit to Blimp

Revert of Extracting placeholder information from Webkit to Blimp (patchset #15 id:280001 of https://codereview.chromium.org/2370393002/ )

Reason for revert:
I'm speculatively reverting this to see if it causes the failure in blink_platform_unittests noted in https://crbug.com/667094 and as seen in

https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Linux%20Trusty/builds/20123

. It's not at all obvious to my why this would cause that failure, but it's not obvious to me what other changes would've, either, unless maybe it was pdr's change (which I will try if this doesn't fix it).

Original issue's description:
> Extracting placeholder information from Webkit to Blimp
>
> For text input, the new Blimp IME requires few additional information
> associated with the text field such as label and placeholder attributes.
> For a mock UI, visit go/blimp-type-3
>
> In this CL, the Blimp tab makes a call to RenderFrameHost to access
> the information about the currently focused text input field and
> supplies a callback to act upon obtaining the information. The
> RenderFrame gets the information, populates it into a FormFieldData
> struct and passes it to the browser which then invokes the supplied
> callback by the embedder.
>
> BUG=651902
> CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
>
> Committed: https://crrev.com/aacf8d1d11beaff516bd79f88b8571eae0f5d031
> Cr-Commit-Position: refs/heads/master@{#433364}

TBR=creis@chromium.org,ekaramad@chromium.org,dtrainor@chromium.org,nyquist@chromium.org,shuchen@chromium.org,tkent@chromium.org,kenrb@chromium.org,shaktisahu@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=651902
Committed: https://crrev.com/fa9428f0ee39451a7c8c4f1386d20949bede4320
Cr-Commit-Position: refs/heads/master@{#433460}

[Dirk Pranke, 2016-11-20 19:55:52]

The CQ bit was checked by dpranke@chromium.org

[Dirk Pranke, 2016-11-20 19:55:53]

Created Revert of Extracting placeholder information from Webkit to Blimp

[commit-bot: I haz the power, 2016-11-20 19:56:01]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-11-20 21:19:02]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-11-20 21:19:03]

Try jobs failed on following builders:
  android_n5x_swarming_rel on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_n5x_...)

[Dirk Pranke, 2016-11-20 21:48:53]

The CQ bit was checked by dpranke@chromium.org

[commit-bot: I haz the power, 2016-11-20 21:49:03]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-11-20 22:27:29]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1479678533725830, "parent_rev":
["46f3cf5a5d6e48fd61b3f5aafb9214eb7244f682", null], "commit_rev":
["bcc489f0cb0f3642f070b4eafba25e1179446cd3", null]}

[commit-bot: I haz the power, 2016-11-20 22:27:59]

Description was changed from

==========
Revert of Extracting placeholder information from Webkit to Blimp (patchset #15
id:280001 of https://codereview.chromium.org/2370393002/ )

Reason for revert:
I'm speculatively reverting this to see if it causes the failure in
blink_platform_unittests noted in https://crbug.com/667094 and as seen in

https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Linux%20Trusty...

. It's not at all obvious to my why this would cause that failure, but it's not
obvious to me what other changes would've, either, unless maybe it was pdr's
change (which I will try if this doesn't fix it).

Original issue's description:
> Extracting placeholder information from Webkit to Blimp
> 
> For text input, the new Blimp IME requires few additional information
> associated with the text field such as label and placeholder attributes.
> For a mock UI, visit go/blimp-type-3
> 
> In this CL, the Blimp tab makes a call to RenderFrameHost to access
> the information about the currently focused text input field and
> supplies a callback to act upon obtaining the information. The
> RenderFrame gets the information, populates it into a FormFieldData
> struct and passes it to the browser which then invokes the supplied
> callback by the embedder.
> 
> BUG=651902
> CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
> 
> Committed: https://crrev.com/aacf8d1d11beaff516bd79f88b8571eae0f5d031
> Cr-Commit-Position: refs/heads/master@{#433364}

TBR=creis@chromium.org,ekaramad@chromium.org,dtrainor@chromium.org,nyquist@ch...
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=651902
==========

to

==========
Revert of Extracting placeholder information from Webkit to Blimp (patchset #15
id:280001 of https://codereview.chromium.org/2370393002/ )

Reason for revert:
I'm speculatively reverting this to see if it causes the failure in
blink_platform_unittests noted in https://crbug.com/667094 and as seen in

https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Linux%20Trusty...

. It's not at all obvious to my why this would cause that failure, but it's not
obvious to me what other changes would've, either, unless maybe it was pdr's
change (which I will try if this doesn't fix it).

Original issue's description:
> Extracting placeholder information from Webkit to Blimp
> 
> For text input, the new Blimp IME requires few additional information
> associated with the text field such as label and placeholder attributes.
> For a mock UI, visit go/blimp-type-3
> 
> In this CL, the Blimp tab makes a call to RenderFrameHost to access
> the information about the currently focused text input field and
> supplies a callback to act upon obtaining the information. The
> RenderFrame gets the information, populates it into a FormFieldData
> struct and passes it to the browser which then invokes the supplied
> callback by the embedder.
> 
> BUG=651902
> CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
> 
> Committed: https://crrev.com/aacf8d1d11beaff516bd79f88b8571eae0f5d031
> Cr-Commit-Position: refs/heads/master@{#433364}

TBR=creis@chromium.org,ekaramad@chromium.org,dtrainor@chromium.org,nyquist@ch...
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=651902
==========

[commit-bot: I haz the power, 2016-11-20 22:28:00]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-11-20 22:30:34]

Description was changed from

==========
Revert of Extracting placeholder information from Webkit to Blimp (patchset #15
id:280001 of https://codereview.chromium.org/2370393002/ )

Reason for revert:
I'm speculatively reverting this to see if it causes the failure in
blink_platform_unittests noted in https://crbug.com/667094 and as seen in

https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Linux%20Trusty...

. It's not at all obvious to my why this would cause that failure, but it's not
obvious to me what other changes would've, either, unless maybe it was pdr's
change (which I will try if this doesn't fix it).

Original issue's description:
> Extracting placeholder information from Webkit to Blimp
> 
> For text input, the new Blimp IME requires few additional information
> associated with the text field such as label and placeholder attributes.
> For a mock UI, visit go/blimp-type-3
> 
> In this CL, the Blimp tab makes a call to RenderFrameHost to access
> the information about the currently focused text input field and
> supplies a callback to act upon obtaining the information. The
> RenderFrame gets the information, populates it into a FormFieldData
> struct and passes it to the browser which then invokes the supplied
> callback by the embedder.
> 
> BUG=651902
> CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
> 
> Committed: https://crrev.com/aacf8d1d11beaff516bd79f88b8571eae0f5d031
> Cr-Commit-Position: refs/heads/master@{#433364}

TBR=creis@chromium.org,ekaramad@chromium.org,dtrainor@chromium.org,nyquist@ch...
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=651902
==========

to

==========
Revert of Extracting placeholder information from Webkit to Blimp (patchset #15
id:280001 of https://codereview.chromium.org/2370393002/ )

Reason for revert:
I'm speculatively reverting this to see if it causes the failure in
blink_platform_unittests noted in https://crbug.com/667094 and as seen in

https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Linux%20Trusty...

. It's not at all obvious to my why this would cause that failure, but it's not
obvious to me what other changes would've, either, unless maybe it was pdr's
change (which I will try if this doesn't fix it).

Original issue's description:
> Extracting placeholder information from Webkit to Blimp
>
> For text input, the new Blimp IME requires few additional information
> associated with the text field such as label and placeholder attributes.
> For a mock UI, visit go/blimp-type-3
>
> In this CL, the Blimp tab makes a call to RenderFrameHost to access
> the information about the currently focused text input field and
> supplies a callback to act upon obtaining the information. The
> RenderFrame gets the information, populates it into a FormFieldData
> struct and passes it to the browser which then invokes the supplied
> callback by the embedder.
>
> BUG=651902
> CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
>
> Committed: https://crrev.com/aacf8d1d11beaff516bd79f88b8571eae0f5d031
> Cr-Commit-Position: refs/heads/master@{#433364}

TBR=creis@chromium.org,ekaramad@chromium.org,dtrainor@chromium.org,nyquist@ch...
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=651902

Committed: https://crrev.com/fa9428f0ee39451a7c8c4f1386d20949bede4320
Cr-Commit-Position: refs/heads/master@{#433460}
==========

[commit-bot: I haz the power, 2016-11-20 22:30:35]

Patchset 1 (id:??) landed as
https://crrev.com/fa9428f0ee39451a7c8c4f1386d20949bede4320
Cr-Commit-Position: refs/heads/master@{#433460}

[Dirk Pranke, 2016-11-21 17:44:32]

A revert of this CL (patchset #1 id:1) has been created in
https://codereview.chromium.org/2520013002/ by dpranke@chromium.org.

The reason for reverting is: The revert did not fix the problem, so I'm
relanding this now.

See also crbug.com/667147..