Autofill - Add question mark asset

Autofill - Add question mark asset

The name help_outline is what the icon library calls it.

BUG=464640
TBR=dfalcantara@chromium.org

Committed: https://chromium.googlesource.com/chromium/src/+/6ceeba6c6cce2ab18cba7618f6fa2f6a8676d76b

[commit-bot: I haz the power, 2015-03-09 17:37:04]

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

[Evan Stade, 2015-03-09 17:37:18]

Committed patchset #1 (id:1) manually as
6ceeba6c6cce2ab18cba7618f6fa2f6a8676d76b (presubmit successful).

[aurimas (slooooooooow), 2015-03-09 17:42:14]

aurimas@chromium.org changed reviewers:
+ aurimas@chromium.org

[aurimas (slooooooooow), 2015-03-09 17:42:14]

Where is this used?

[Evan Stade, 2015-03-09 17:48:57]

On 2015/03/09 17:42:14, aurimas wrote:
> Where is this used?

will be used here: https://codereview.chromium.org/990543002/

[aurimas (slooooooooow), 2015-03-09 17:52:27]

On 2015/03/09 at 17:48:57, estade wrote:
> On 2015/03/09 17:42:14, aurimas wrote:
> > Where is this used?
> 
> will be used here: https://codereview.chromium.org/990543002/

Why did you not land these assets in that CL then? :/

[Evan Stade, 2015-03-09 18:02:56]

On 2015/03/09 17:52:27, aurimas wrote:
> On 2015/03/09 at 17:48:57, estade wrote:
> > On 2015/03/09 17:42:14, aurimas wrote:
> > > Where is this used?
> > 
> > will be used here: https://codereview.chromium.org/990543002/
> 
> Why did you not land these assets in that CL then? :/

I'm in the habit of landing assets manually (it is possible the CQ can handle
binary files now and this is an antiquated practice)

[aurimas (slooooooooow), 2015-03-09 18:08:04]

On 2015/03/09 at 18:02:56, estade wrote:
> On 2015/03/09 17:52:27, aurimas wrote:
> > On 2015/03/09 at 17:48:57, estade wrote:
> > > On 2015/03/09 17:42:14, aurimas wrote:
> > > > Where is this used?
> > > 
> > > will be used here: https://codereview.chromium.org/990543002/
> > 
> > Why did you not land these assets in that CL then? :/
> 
> I'm in the habit of landing assets manually (it is possible the CQ can handle
binary files now and this is an antiquated practice)

Even with this habit, you should land the assets manually together with the
other CL, now everyone with Android checkout will get lint warnings about unused
asset :(

[Evan Stade, 2015-03-09 18:11:42]

On 2015/03/09 18:08:04, aurimas wrote:
> On 2015/03/09 at 18:02:56, estade wrote:
> > On 2015/03/09 17:52:27, aurimas wrote:
> > > On 2015/03/09 at 17:48:57, estade wrote:
> > > > On 2015/03/09 17:42:14, aurimas wrote:
> > > > > Where is this used?
> > > > 
> > > > will be used here: https://codereview.chromium.org/990543002/
> > > 
> > > Why did you not land these assets in that CL then? :/
> > 
> > I'm in the habit of landing assets manually (it is possible the CQ can
handle
> binary files now and this is an antiquated practice)
> 
> Even with this habit, you should land the assets manually together with the
> other CL,

I get yelled at when I land code without using the CQ.

> now everyone with Android checkout will get lint warnings about unused
> asset :(

I did not realize that. In the future I'll try to land patches like these more
simultaneously, or land the assets after the other change and simultaneous to
the resource ID switch in code.

[aurimas (slooooooooow), 2015-03-09 18:16:36]

On 2015/03/09 at 18:11:42, estade wrote:
> On 2015/03/09 18:08:04, aurimas wrote:
> > On 2015/03/09 at 18:02:56, estade wrote:
> > > On 2015/03/09 17:52:27, aurimas wrote:
> > > > On 2015/03/09 at 17:48:57, estade wrote:
> > > > > On 2015/03/09 17:42:14, aurimas wrote:
> > > > > > Where is this used?
> > > > > 
> > > > > will be used here: https://codereview.chromium.org/990543002/
> > > > 
> > > > Why did you not land these assets in that CL then? :/
> > > 
> > > I'm in the habit of landing assets manually (it is possible the CQ can
handle
> > binary files now and this is an antiquated practice)
> > 
> > Even with this habit, you should land the assets manually together with the
> > other CL,
> 
> I get yelled at when I land code without using the CQ.
> 
> > now everyone with Android checkout will get lint warnings about unused
> > asset :(
> 
> I did not realize that. In the future I'll try to land patches like these more
simultaneously, or land the assets after the other change and simultaneous to
the resource ID switch in code.

I am pretty sure that CQ let's you land images now.

[gone, 2015-03-09 18:33:05]

+1 for all of aurimas' comments.  I've been landing assets in the same CLs that
use them for months now.