Reland of Extracting placeholder information from Webkit to Blimp

Reland of Extracting placeholder information from Webkit to Blimp (patchset #1 id:1 of https://codereview.chromium.org/2513333002/ )

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

See also crbug.com/667147.

Original issue's description:
> 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}

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
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=651902
Committed: https://crrev.com/32d4b3a47daccad1a7ea2612753292d3552f6412
Cr-Commit-Position: refs/heads/master@{#433582}

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

The CQ bit was checked by dpranke@chromium.org

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

Created Reland of Extracting placeholder information from Webkit to Blimp

[commit-bot: I haz the power, 2016-11-21 17:44:55]

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-21 17:45:42]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1479750272454180, "parent_rev":
"93001d320a6629087364928ee3e059f3d514988b", "commit_rev":
"2aada4549d2c77f8a3066356915a30a1270a997f"}

[commit-bot: I haz the power, 2016-11-21 17:46:06]

Description was changed from

==========
Reland of Extracting placeholder information from Webkit to Blimp (patchset #1
id:1 of https://codereview.chromium.org/2513333002/ )

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

See also crbug.com/667147.

Original issue's description:
> 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}

TBR=creis@chromium.org,ekaramad@chromium.org,dtrainor@chromium.org,nyquist@ch...
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=651902
==========

to

==========
Reland of Extracting placeholder information from Webkit to Blimp (patchset #1
id:1 of https://codereview.chromium.org/2513333002/ )

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

See also crbug.com/667147.

Original issue's description:
> 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}

TBR=creis@chromium.org,ekaramad@chromium.org,dtrainor@chromium.org,nyquist@ch...
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=651902
==========

[commit-bot: I haz the power, 2016-11-21 17:46:07]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-11-21 17:47:50]

Description was changed from

==========
Reland of Extracting placeholder information from Webkit to Blimp (patchset #1
id:1 of https://codereview.chromium.org/2513333002/ )

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

See also crbug.com/667147.

Original issue's description:
> 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}

TBR=creis@chromium.org,ekaramad@chromium.org,dtrainor@chromium.org,nyquist@ch...
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=651902
==========

to

==========
Reland of Extracting placeholder information from Webkit to Blimp (patchset #1
id:1 of https://codereview.chromium.org/2513333002/ )

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

See also crbug.com/667147.

Original issue's description:
> 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}

TBR=creis@chromium.org,ekaramad@chromium.org,dtrainor@chromium.org,nyquist@ch...
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=651902

Committed: https://crrev.com/32d4b3a47daccad1a7ea2612753292d3552f6412
Cr-Commit-Position: refs/heads/master@{#433582}
==========

[commit-bot: I haz the power, 2016-11-21 17:47:52]

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

[tkent, 2016-11-22 00:29:20]

lgtm