[Contextual Search] Add support for crushed sprites and animate the search provider icon

build/android/lint/suppressions.xml

 <?xml version="1.0" encoding="utf-8"?>
 <lint>
   <!--
 STOP! It looks like you want to suppress some lint errors:
 - Have you tried identifing the offending patch?
   Ask the author for a fix and/or revert the patch.
 - It is preferred to add suppressions in the code instead of
   sweeping it under the rug here. See:
 
     http://developer.android.com/tools/debugging/improving-w-lint.html
@@ skipping 35 matching lines @@
     <ignore path="components/web_contents_delegate_android/android/java/res/drawable-xxhdpi"/>
     <ignore path="components/web_contents_delegate_android/android/java/res/drawable-xxxhdpi"/>
     <ignore path="content/public/android/java/res/drawable-xxhdpi"/>
     <ignore path="content/public/android/java/res/drawable-xxxhdpi"/>
     <ignore path="chrome/android/java/res/drawable-xxhdpi"/>
     <ignore path="chrome/android/java/res/drawable-xxxhdpi"/>
     <ignore path="ui/android/java/res/drawable-xxhdpi"/>
     <ignore path="ui/android/java/res/drawable-xxxhdpi"/>
     <ignore regexp=".*: data_reduction_illustration.png, reader_mode_bar_background.9.png, tabs_moved_htc.png, tabs_moved_nexus.png, tabs_moved_samsung.png$"/>
   </issue>
+  <issue id="IconDipSize">
+    <ignore regexp=".*google_icon_sprite.png.*"/>

[newt (away), 2015/10/01 17:19:46]

Just confirming: this regex won't cause other IconDipSize warnings to be
ignored? And same with the InconsistentArrays regex below.

[Theresa, 2015/10/01 17:57:34]

Yes, it's a different line for each warning for both of these warnings.

For IconDipSize, I tested by changing account_manager_on_picture.png and saw
this warning with the lint suppression in place:
chrome/android/java/res/drawable-mdpi/account_management_no_picture.png The
image `account_management_no_picture.png` varies significantly in its
density-independent (dip) size across the various density versions:
drawable-hdpi/account_management_no_picture.png: 40x40 dp (60x60 px),
drawable-xhdpi/account_management_no_picture.png: 40x40 dp (80x80 px),
drawable-xxhdpi/account_management_no_picture.png: 40x40 dp (120x120 px),
drawable-xxxhdpi/account_management_no_picture.png: 40x40 dp (160x160 px),
drawable-mdpi/account_management_no_picture.png: 39x1000 dp (39x1000 px):
IconDipSize [warning]


For InconsistentArrays, if a second, differently named file has an array with
the same name as one of the arrays in
contextual_search_crushed_sprite_frames.xml and that array has a different # of
items, the error will still be suppressed. If two differently named files have
inconsistent arrays then the error will still be surfaced. The
InconsistentArrays warnings look like this:

chrome/android/java/res/values-hdpi/contextual_search_crushed_sprite_frames.xml:291
Array `contextual_search_sprite_frame_17` has an inconsistent number of items (0
in `values-ldpi/contextual_search_crushed_sprite_frames.xml`, 12 in
`values-hdpi/contextual_search_crushed_sprite_frames.xml`): InconsistentArrays
[warning]
  <integer-array name="contextual_search_sprite_frame_17">
  ^

I could make the regexp more specific, but I personally don't think it's
necessary.

[newt (away), 2015/10/01 18:18:04]

Sounds good to me. Thanks for confirming.

+  </issue>
   <issue id="IconLocation">
     <!-- It is OK for content_shell_apk to have missing assets. -->
     <ignore path="content/shell/android/java/res/"/>
     <!-- Suppression for chrome/test/chromedriver/test/webview_shell/java/res/drawable/icon.png -->
     <ignore path="res/drawable/icon.png"/>
     <!-- TODO(lambroslambrou) remove this once crbug.com/502030 is fixed. -->
     <ignore path="remoting/android/java/res"/>
   </issue>
+  <issue id="InconsistentArrays">
+    <ignore regexp=".*contextual_search_crushed_sprite_frames.xml.*"/>
+  </issue>
   <issue id="InconsistentLayout" severity="ignore"/>
   <issue id="InflateParams" severity="ignore"/>
   <issue id="MissingApplicationIcon" severity="ignore"/>
   <issue id="MissingRegistered" severity="ignore"/>
   <issue id="MissingVersion">
     <ignore path="AndroidManifest.xml"/>
   </issue>
   <issue id="InlinedApi" severity="ignore"/>
   <issue id="NewApi">
     <ignore regexp="Attribute `paddingStart` referenced here can result in a crash on some specific devices older than API 17"/>
@@ skipping 27 matching lines @@
     <ignore path="content/shell/android/shell_apk/res/values/strings.xml" />
   </issue>
   <issue id="SignatureOrSystemPermissions" severity="ignore"/>
   <issue id="UnusedAttribute" severity="ignore"/>
   <issue id="ViewConstructor" severity="ignore"/>
   <issue id="WrongCall" severity="ignore"/>
   <issue id="UselessParent">
     <ignore path="chrome/android/java/res/layout/data_reduction_promo_screen.xml" />
   </issue>
 </lint>