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

Issue 633271 link

Starred by 1 user
Status: Fixed
Owner:
█ weiran@google.com
Email to this user bounced
Closed: Aug 2016
Cc:
fgor...@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 1
Type: Bug



Sign in to add a comment

Android manifest warning

Project Member Reported by dewittj@chromium.org, Aug 1 2016 
Some build warnings regarding the path patterns we just added:

AndroidManifest.xml:185 android:pathPattern attribute should start with '/', but it is : .*\\.mhtml: AppIndexingError [warning]
                <data android:pathPattern=".*\\.mhtml"/>
                      ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
AndroidManifest.xml:186 android:pathPattern attribute should start with '/', but it is : .*\\.mht: AppIndexingError [warning]
                <data android:pathPattern=".*\\.mht"/>
                      ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
AndroidManifest.xml:187 android:pathPattern attribute should start with '/', but it is : .*\\.eml: AppIndexingError [warning]
                <data android:pathPattern=".*\\.eml"/> 
                      ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
AndroidManifest.xml:199 android:pathPattern attribute should start with '/', but it is : .*\\.mhtml: AppIndexingError [warning]
                <data android:pathPattern=".*\\.mhtml"/>
                      ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
AndroidManifest.xml:200 android:pathPattern attribute should start with '/', but it is : .*\\.mht: AppIndexingError [warning]
                <data android:pathPattern=".*\\.mht"/>
                      ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
AndroidManifest.xml:201 android:pathPattern attribute should start with '/', but it is : .*\\.eml: AppIndexingError [warning]
                <data android:pathPattern=".*\\.eml"/> 
                      ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
Project Member

Comment 1 by bugdroid1@chromium.org, Aug 1 2016 

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

commit 6538c29330b8ffc57b737829cff17d48d5e54a5d
Author: weiran <weiran@google.com>
Date: Mon Aug 01 21:13:17 2016

Fix pathpattern warning and delete eml support

EML is a generic MIME file format, that can contain arbitrary email messages, including attachments etc. We want to stay focus on mhtml only.

BUG= 633271 

Review-Url: https://codereview.chromium.org/2197343002
Cr-Commit-Position: refs/heads/master@{#409056}

[modify] https://crrev.com/6538c29330b8ffc57b737829cff17d48d5e54a5d/chrome/android/java/AndroidManifest.xml

Comment 2 by weiran@google.com, Aug 1 2016

Status: Fixed (was: Untriaged)
►

Sign in to add a comment