New issue
Advanced search Search tips
Note: Color blocks (like █ or █) mean that a user may not be available. Tooltip shows the reason.

Issue 920360 link

Starred by 1 user
Status: Fixed
Owner:
█ zqzh...@chromium.org
Android MediaRouter only (left Chro...
Closed: Jan 17
Cc:
mfo...@chromium.org
mlamouri@chromium.org
btolsch@chromium.org
taku...@chromium.org
powerb@chromium.org
tguilbert@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 1
Type: Bug



Sign in to add a comment

java.lang.NullPointerException: Attempt to read from field 'java.lang.String com.google.android.gms.cast.ApplicationMetadata.a' on a null object reference

Project Member Reported by zqzh...@chromium.org, Jan 9 
Stack trace:

java.lang.NullPointerException: Attempt to read from field 'java.lang.String com.google.android.gms.cast.ApplicationMetadata.a' on a null object reference
at com.google.android.gms.cast.ApplicationMetadata.getApplicationId	(ApplicationMetadata.java:13 )
at org.chromium.chrome.browser.media.router.caf.CafMessageHandler.buildSessionMessage	(CafMessageHandler.java:696 )
at org.chromium.chrome.browser.media.router.caf.CafMessageHandler.notifySessionConnectedToClient	(CafMessageHandler.java:155 )
at org.chromium.chrome.browser.media.router.caf.CafMessageHandler.handleClientConnectMessage	(CafMessageHandler.java:193 )
at org.chromium.chrome.browser.media.router.caf.CafMessageHandler.handleMessageFromClient	(CafMessageHandler.java:170 )
at org.chromium.chrome.browser.media.router.caf.CafMediaRouteProvider.sendStringMessage	(CafMediaRouteProvider.java:110 )
at org.chromium.chrome.browser.media.router.ChromeMediaRouter.sendStringMessage	(ChromeMediaRouter.java:377 )
at android.os.MessageQueue.nativePollOnce	(MessageQueue.java )
at android.os.MessageQueue.next	(MessageQueue.java:326 )
at android.os.Looper.loop	(Looper.java:160 )
at android.app.ActivityThread.main	(ActivityThread.java:6718 )
at java.lang.reflect.Method.invoke	(Method.java )
at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run	(RuntimeInit.java:493 )
at com.android.internal.os.ZygoteInit.main	(ZygoteInit.java:858 )
Project Member

Comment 1 by bugdroid1@chromium.org, Jan 9 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/70ee1088650b14ffe68d9093cc7a1525b1261360

commit 70ee1088650b14ffe68d9093cc7a1525b1261360
Author: Zhiqiang Zhang <zqzhang@google.com>
Date: Wed Jan 09 22:13:07 2019

Fix an NPE when getting app ID from ApplicationMetadata

In some cases, the ApplicationMetadata returned by CastSession might
return null. In this CL, if this happens, the app ID will be read from
the RouteCreationInfo in the session controller.

Bug:  920360 
Change-Id: I2fb9c10efa052a57b65c5314b1526feba83eac7c
Reviewed-on: https://chromium-review.googlesource.com/c/1403915
Reviewed-by: Thomas Guilbert <tguilbert@chromium.org>
Commit-Queue: Zhiqiang Zhang <zqzhang@chromium.org>
Cr-Commit-Position: refs/heads/master@{#621321}
[modify] https://crrev.com/70ee1088650b14ffe68d9093cc7a1525b1261360/chrome/android/java/src/org/chromium/chrome/browser/media/router/caf/CafMessageHandler.java

Comment 2 by zqzh...@chromium.org, Jan 17 (5 days ago)

Status: Fixed (was: Assigned)
►

Sign in to add a comment