Remove unused variable in MediaDrmCredentialManager.java

Remove unused variable in MediaDrmCredentialManager.java

R=tedchoc@chromium.org

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

[Hyungwook Lee, 2014-03-01 05:47:47]

Please review my patches.

[Ted C, 2014-03-03 21:34:15]

https://codereview.chromium.org/185223002/diff/1/content/public/android/java/...
File
content/public/android/java/src/org/chromium/content/browser/MediaDrmCredentialManager.java
(right):

https://codereview.chromium.org/185223002/diff/1/content/public/android/java/...
content/public/android/java/src/org/chromium/content/browser/MediaDrmCredentialManager.java:15:
private long mNativeMediaDrmCredentialManager;
Actually, it doesn't look like this variable is used at all.  I think you should
remove this line instead.

[Hyungwook Lee, 2014-03-04 02:16:10]

On 2014/03/03 21:34:15, Ted C wrote:
>
https://codereview.chromium.org/185223002/diff/1/content/public/android/java/...
> File
>
content/public/android/java/src/org/chromium/content/browser/MediaDrmCredentialManager.java
> (right):
> 
>
https://codereview.chromium.org/185223002/diff/1/content/public/android/java/...
>
content/public/android/java/src/org/chromium/content/browser/MediaDrmCredentialManager.java:15:
> private long mNativeMediaDrmCredentialManager;
> Actually, it doesn't look like this variable is used at all.  I think you
should
> remove this line instead.

I lost my changes in my working branches.
I will close this code review url and re-open another one.

[Hyungwook Lee, 2014-03-04 02:25:00]

On 2014/03/04 02:16:10, Hyungwook Lee wrote:
> On 2014/03/03 21:34:15, Ted C wrote:
> >
>
https://codereview.chromium.org/185223002/diff/1/content/public/android/java/...
> > File
> >
>
content/public/android/java/src/org/chromium/content/browser/MediaDrmCredentialManager.java
> > (right):
> > 
> >
>
https://codereview.chromium.org/185223002/diff/1/content/public/android/java/...
> >
>
content/public/android/java/src/org/chromium/content/browser/MediaDrmCredentialManager.java:15:
> > private long mNativeMediaDrmCredentialManager;
> > Actually, it doesn't look like this variable is used at all.  I think you
> should
> > remove this line instead.
> 
> I lost my changes in my working branches.
> I will close this code review url and re-open another one.

Sorry, I found my changes.
I've deleted mNativeMediaDrmCredentialManager in MediaDrmCredentialManager.java.

[Ted C, 2014-03-05 21:39:15]

lgtm

[Ted C, 2014-03-05 21:39:37]

The CQ bit was checked by tedchoc@chromium.org

[commit-bot: I haz the power, 2014-03-05 21:41:17]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/withlhw@gmail.com/185223002/20001

[commit-bot: I haz the power, 2014-03-05 23:10:03]

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

[commit-bot: I haz the power, 2014-03-05 23:10:04]

Retried try job too often on win_rel for step(s) interactive_ui_tests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win_rel&nu...

[Ted C, 2014-03-06 00:51:50]

The CQ bit was checked by tedchoc@chromium.org

[commit-bot: I haz the power, 2014-03-06 00:51:55]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/withlhw@gmail.com/185223002/20001

[commit-bot: I haz the power, 2014-03-06 02:52:45]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/withlhw@gmail.com/185223002/20001

[commit-bot: I haz the power, 2014-03-06 03:17:19]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/withlhw@gmail.com/185223002/20001

[commit-bot: I haz the power, 2014-03-06 10:56:36]

Change committed as 255308