Restore image level when setting up image button in AppMenu

Update setupImageButton() in AppMenuAdapter to restore drawable's level after button.setImageDrawable() is called.

BUG=386526
NOTRY=true

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=280882

[Ian Wen, 2014-06-30 21:52:14]

Changes in AppMenuPropertiesDelegate have been reverted. Now this patch only
deals with issues when updating level of menu icons.

[aurimas (slooooooooow), 2014-06-30 21:54:30]

https://codereview.chromium.org/343183006/diff/20001/chrome/android/java/src/...
File
chrome/android/java/src/org/chromium/chrome/browser/appmenu/AppMenuAdapter.java
(right):

https://codereview.chromium.org/343183006/diff/20001/chrome/android/java/src/...
chrome/android/java/src/org/chromium/chrome/browser/appmenu/AppMenuAdapter.java:213:
// Thus we need to remember initial level for reload button.
remove "reload button" as it is true for any image that could have a level set.

https://codereview.chromium.org/343183006/diff/20001/chrome/android/java/src/...
File
chrome/android/java/src/org/chromium/chrome/browser/appmenu/AppMenuHandler.java
(right):

https://codereview.chromium.org/343183006/diff/20001/chrome/android/java/src/...
chrome/android/java/src/org/chromium/chrome/browser/appmenu/AppMenuHandler.java:160:
mObservers.get(i).onMenuVisibilityChanged(isVisible);
Can we just register as a menu visibility observer instead of adding
onMenuDismissed call?

[aurimas (slooooooooow), 2014-07-01 16:14:04]

lgtm after you update CL title and description

[Ian Wen, 2014-07-01 16:38:44]

On 2014/07/01 16:14:04, aurimas wrote:
> lgtm after you update CL title and description

Yep. I have updated the title and description. :)

[aurimas (slooooooooow), 2014-07-01 16:41:19]

https://chromiumcodereview.appspot.com/343183006/diff/40001/chrome/android/ja...
File
chrome/android/java/src/org/chromium/chrome/browser/appmenu/AppMenuAdapter.java
(right):

https://chromiumcodereview.appspot.com/343183006/diff/40001/chrome/android/ja...
chrome/android/java/src/org/chromium/chrome/browser/appmenu/AppMenuAdapter.java:213:
// restes drawable to default level.
s/restes/resets

[Ian Wen, 2014-07-01 17:51:15]

The CQ bit was checked by ianwen@chromium.org

[commit-bot: I haz the power, 2014-07-01 17:52:59]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/ianwen@chromium.org/343183006/60001

[commit-bot: I haz the power, 2014-07-01 19:50:26]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  android_dbg_triggered_tests on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/android_dbg_triggered...)
  win_chromium_x64_rel on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/win_chromium_x64_rel/...)

[commit-bot: I haz the power, 2014-07-01 19:54:07]

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

[commit-bot: I haz the power, 2014-07-01 19:54:08]

Try jobs failed on following builders:
  win_chromium_x64_rel on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/win_chromium_x64_rel/...)

[Ian Wen, 2014-07-01 20:29:00]

The CQ bit was checked by ianwen@chromium.org

[commit-bot: I haz the power, 2014-07-01 20:30:22]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/ianwen@chromium.org/343183006/60001

[commit-bot: I haz the power, 2014-07-01 21:12:08]

Change committed as 280882