ActionBarActivityで finalになったonMenuItemSelectedのソースを追ってみた

ActionBarActivity#onOptionsItemSelectedのソースを追ってみた。

android.support.v7.app.ActionBarActivity

view plainprint?

1. ActionBarActivityDelegate mImpl;  
2.   
3. @Override  
4. public final boolean onMenuItemSelected(int featureId, android.view.MenuItem item) {  
5.     if (mImpl.onMenuItemSelected(featureId, item)) {  
6.         return true;  
7.     }  
8.   
9.     final ActionBar ab = getSupportActionBar();  
10.     if (item.getItemId() == android.R.id.home && ab != null &&  
11.             (ab.getDisplayOptions() & ActionBar.DISPLAY_HOME_AS_UP) != 0) {  
12.         return onSupportNavigateUp();  
13.     }  
14.     return false;  
15. }  

ActionBarActivityDelegate mImpl; @Override public final boolean onMenuItemSelected(int featureId, android.view.MenuItem item) { if (mImpl.onMenuItemSelected(featureId, item)) { return true; } final ActionBar ab = getSupportActionBar(); if (item.getItemId() == android.R.id.home && ab != null && (ab.getDisplayOptions() & ActionBar.DISPLAY_HOME_AS_UP) != 0) { return onSupportNavigateUp(); } return false; } 5行目で、ActionBarActivityDelegateのonMenuItemSelectedを呼び出している。
ActionBarActivityDelegateを実装は幾つかのOSレベル毎に分かれているので、 その１つActionBarActivityDelegateBaseを見てみる。

android.support.v7.app.ActionBarActivityDelegateBase

view plainprint?

1. @Override  
2. public boolean onMenuItemSelected(int featureId, MenuItem item) {  
3.     if (featureId == Window.FEATURE_OPTIONS_PANEL) {  
4.         item = MenuWrapperFactory.createMenuItemWrapper(item);  
5.     }  
6.     return mActivity.superOnMenuItemSelected(featureId, item);  
7. }  

@Override public boolean onMenuItemSelected(int featureId, MenuItem item) { if (featureId == Window.FEATURE_OPTIONS_PANEL) { item = MenuWrapperFactory.createMenuItemWrapper(item); } return mActivity.superOnMenuItemSelected(featureId, item); } ActionBarActivityのsuperOnMenuItemSelectedを呼んでいる。

android.support.v7.app.ActionBarActivity

view plainprint?

1. boolean superOnMenuItemSelected(int featureId, MenuItem menuItem) {  
2.     return super.onMenuItemSelected(featureId, menuItem);  
3. }  

boolean superOnMenuItemSelected(int featureId, MenuItem menuItem) { return super.onMenuItemSelected(featureId, menuItem); } ActionBarActivityの親クラスFragmentActivityのonMenuItemSelectedを呼んでいる。

android.support.v4.app.FragmentActivity

view plainprint?

1. /** 
2.  * Dispatch context and options menu to fragments. 
3.  */  
4. @Override  
5. public boolean onMenuItemSelected(int featureId, MenuItem item) {  
6.     if (super.onMenuItemSelected(featureId, item)) {  
7.         return true;  
8.     }  
9.       
10.     switch (featureId) {  
11.         case Window.FEATURE_OPTIONS_PANEL:  
12.             return mFragments.dispatchOptionsItemSelected(item);  
13.               
14.         case Window.FEATURE_CONTEXT_MENU:  
15.             return mFragments.dispatchContextItemSelected(item);  
16.   
17.         default:  
18.             return false;  
19.     }  
20. }  

/** * Dispatch context and options menu to fragments. */ @Override public boolean onMenuItemSelected(int featureId, MenuItem item) { if (super.onMenuItemSelected(featureId, item)) { return true; } switch (featureId) { case Window.FEATURE_OPTIONS_PANEL: return mFragments.dispatchOptionsItemSelected(item); case Window.FEATURE_CONTEXT_MENU: return mFragments.dispatchContextItemSelected(item); default: return false; } } 6行目の親クラスActivityActivityのonMenuItemSelectedを呼んでいる。
メニューが処理されなかったら、Fragmentのメニューを呼び出している。

• Window.FEATURE_OPTIONS_PANEL - ActionBarや昔のメニューパネル
• Window.FEATURE_CONTEXT_MENU - ロングタップで表示されるコンテキストメニュー

android.app.Activity

view plainprint?

1. public boolean onMenuItemSelected(int featureId, MenuItem item) {  
2.     CharSequence titleCondensed = item.getTitleCondensed();  
3.   
4.     switch (featureId) {  
5.         case Window.FEATURE_OPTIONS_PANEL:  
6.             // Put event logging here so it gets called even if subclass  
7.             // doesn't call through to superclass's implmeentation of each  
8.             // of these methods below  
9.             if(titleCondensed != null) {  
10.                 EventLog.writeEvent(50000, 0, titleCondensed.toString());  
11.             }  
12.             if (onOptionsItemSelected(item)) {  
13.                 return true;  
14.             }  
15.             if (mFragments.dispatchOptionsItemSelected(item)) {  
16.                 return true;  
17.             }  
18.             if (item.getItemId() == android.R.id.home && mActionBar != null &&  
19.                     (mActionBar.getDisplayOptions() & ActionBar.DISPLAY_HOME_AS_UP) != 0) {  
20.                 if (mParent == null) {  
21.                     return onNavigateUp();  
22.                 } else {  
23.                     return mParent.onNavigateUpFromChild(this);  
24.                 }  
25.             }  
26.             return false;  
27.               
28.         case Window.FEATURE_CONTEXT_MENU:  
29.             if(titleCondensed != null) {  
30.                 EventLog.writeEvent(50000, 1, titleCondensed.toString());  
31.             }  
32.             if (onContextItemSelected(item)) {  
33.                 return true;  
34.             }  
35.             return mFragments.dispatchContextItemSelected(item);  
36.   
37.         default:  
38.             return false;  
39.     }  
40. }  

public boolean onMenuItemSelected(int featureId, MenuItem item) { CharSequence titleCondensed = item.getTitleCondensed(); switch (featureId) { case Window.FEATURE_OPTIONS_PANEL: // Put event logging here so it gets called even if subclass // doesn't call through to superclass's implmeentation of each // of these methods below if(titleCondensed != null) { EventLog.writeEvent(50000, 0, titleCondensed.toString()); } if (onOptionsItemSelected(item)) { return true; } if (mFragments.dispatchOptionsItemSelected(item)) { return true; } if (item.getItemId() == android.R.id.home && mActionBar != null && (mActionBar.getDisplayOptions() & ActionBar.DISPLAY_HOME_AS_UP) != 0) { if (mParent == null) { return onNavigateUp(); } else { return mParent.onNavigateUpFromChild(this); } } return false; case Window.FEATURE_CONTEXT_MENU: if(titleCondensed != null) { EventLog.writeEvent(50000, 1, titleCondensed.toString()); } if (onContextItemSelected(item)) { return true; } return mFragments.dispatchContextItemSelected(item); default: return false; } } メニューは、12行目でonOptionsItemSelectedを呼び出し。 コンテキストメニューは、32行目でonContextItemSelectedを呼び出し。


リファレンスのOnMenuItemSelectedの記述

Default implementation of onMenuItemSelected(int, MenuItem) for activities. This calls through to the new onOptionsItemSelected(MenuItem) method for the FEATURE_OPTIONS_PANEL panel, so that subclasses of Activity don't need to deal with feature codes.

この通り、onOptionsItemSelectedをオーバーライドしてメニューの処理を実装を行なうのが正しいようだ。

コンテキストメニューはonContextItemSelectedをオーバーライドして処理を実装を行なうのが正しいようだ。


Honycombから、Activity#onMenuItemSelectedにはFragmentやActionBarのメニュー処理が行なわれてるためオーバーライドは注意が必要。

ActionBarActivityのOnMenuItemSelectedはオーバーライドできない。

Supoport Libraryに追加されたandroid-support-v7-appcompatのActionBarActivityのOnMenuItemSelectedはオーバーライドできない。

OnMenuItemSelectedはfinalメソッドとなっている。
代わりに、
onOptionsItemSelectedをオーバーライドして使用します。

リファレンスのOnMenuItemSelectedの記述は以下の通りです。

Default implementation of onMenuItemSelected(int, MenuItem) for activities. This calls through to the new onOptionsItemSelected(MenuItem) method for the FEATURE_OPTIONS_PANEL panel, so that subclasses of Activity don't need to deal with feature codes.

 そもそも、OnMenuItemSelectedを実装しているのが間違っていた？

Android SDK 22 では、Build Toolsが必要になったようです。

今朝リリースされた Android SDK 22について注意

このバージョンより Android SDK Build-toolsが必要になりました。 

インストールしていないと、
ant debug/release 実行に、以下のようなメッセージが出て、ビルドできません。 {android-sdk}/tools/ant/build.xml:479: SDK does not have any Build Tools installed.

 Eclipseでも、
Android Tools > Export Signed/Unsigned Application Package... を実行すると同じようなエラーダイアログが表示されました。

 

SDK 22からToolsに Android SDK Build-tools が増えてるのでインストールしておくように!

JavaScript Memo

Closure Tools — Google Developers https://developers.google.com/closure/utilities/

うるさいLint先生にXMLの警告を無視させる方法

xmlns:tools="http://schemas.android.com/tools"
を定義して、
tools:ignore="ContentDescription,PxUsage"
無効にしたいチェックを記述します。

view plainprint?

1. <relativelayout xmlns:android="http://schemas.android.com/apk/res/android" xmlns:tools="http://schemas.android.com/tools" tools:ignore="ContentDescription,PxUsage" android:layout_width="match_parent" android:layout_height="match_parent" android:background="#ffffff">  
2.   
3.     <imageview android:layout_width="match_parent" android:layout_height="1px" android:background="@android:drawable/divider_horizontal_dark">  
4.   
5.     <imagebutton android:id="@+id/prev" style="@android:style/MediaButton.Previous" android:visibility="gone">  
6.   
7. </imagebutton></imageview></relativelayout>  

ImageView / ImageButton の contentDescriptionとか記述めんどくさいし。
セパレータに使用している ImageViewの heightは ピクセル(px)で定義したいし。