aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* Merge "Don't copy the whole SdkController into the SDK."Xavier Ducrohet2012-02-221-1/+3
|\
| * Don't copy the whole SdkController into the SDK.Xavier Ducrohet2012-02-221-1/+3
| | | | | | | | | | | | We don't want the multi-touch part in there yet. Change-Id: I4b7d214dd42b124cac592981cf71aedd408966ec
* | Merge "Add lint check for "old" proguard configurations with generic Android ↵Tor Norbye2012-02-229-15/+254
|\ \ | | | | | | | | | config"
| * | Add lint check for "old" proguard configurations with generic Android configTor Norbye2012-02-229-15/+254
| | | | | | | | | | | | Change-Id: I79cbdf874cffe78eb62f9af2e22e6cc780853e6a
* | | Merge "ADT: Suggest solution to ADT version check error."Raphael2012-02-2214-105/+380
|\ \ \ | |/ / |/| |
| * | ADT: Suggest solution to ADT version check error.Raphael2012-02-2214-105/+380
| |/ | | | | | | | | | | | | | | | | | | | | | | | | This amends the ADT version check to help the user either: - open the SDK Manager - open the P2 Updater - open the Android Preference On Windows the launch the *external* SDK Manager since eventually we know that ADT will lock something that would prevent the update from working in the first place. Change-Id: Ib20e4e1411b36e3cd794cccbc02518db0a40ced9
* | Merge "Split ProGuard file into two halves"Tor Norbye2012-02-2218-51/+269
|\ \ | |/ |/|
| * Split ProGuard file into two halvesTor Norbye2012-02-2118-51/+269
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This changeset splits the proguard.cfg into two halves: (1) All the general Android settings go into $ANDROID_SDK/proguard/proguard-android.txt. This defines shrinking rules like keep custom views, etc. The crucial point is that this information is maintained and updated by Tools updates, so whenever new APIs are added to Android, or whenever bugs are found in the configuration such as flags needed to work with Dalvik, we can make the updates - we don't have old snapshots living on in projects. (2) Any project specific settings go to proguard-project.txt in the project. (3) The proguard.config property in project.properties now refers to a *path* of configuration files, which are all passed to ProGuard in the given order. The code which processes this setting will substitute android.sdk.home and user.home variables, so the path does not have to be hardcoded to point to the project-android.txt file. The default project templates have been updated to include a commented out configuration setting up proguard as described above. The default proguard file name was changed from proguard.cfg to proguard-project.txt such that it can be directly opened in Eclipse and to make it clear it's an editable text file. Lint was updated to find the Proguard file via the proguard.config property as well as via the old and new default names for projects not enabled with ProGuard. A subsequent CL will add a lint check which identifies projects containing the old setup (full local configuration) and offer to replace it with the new setup. Change-Id: I44b4c97a160114c2382f02f843c95486a0dc9d6b
* | Merge "ddms: Add Network Stats View"Siva Velusamy2012-02-225-0/+59
|\ \
| * | ddms: Add Network Stats ViewSiva Velusamy2012-02-215-0/+59
| | | | | | | | | | | | Change-Id: I89bf6b84a2ef2ed9a876eec41fae9a974ba93137
* | | Merge "gltrace: Add a GLCall detail provider."Siva Velusamy2012-02-2212-29/+181
|\ \ \ | |/ /
| * | gltrace: Add a GLCall detail provider.Siva Velusamy2012-02-2112-29/+181
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The following changes are performed in this CL: - The details view is moved into the views.detail package. - The details view supports detail providers for GL State Variables. This patch adds a new interface for providers that contribute details for GL Calls. This interface is used to provide support for displaying the framebuffer that might be part of a glDraw call. Change-Id: I2e327f02c6326759c23c2952acd361ff48f8f904
* | | Add lint rule to catch incorrect setColor callsTor Norbye2012-02-2110-17/+199
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Android has various setColor methods (such as setTextColor) which take an integer, where it expects RGB values in the bytes. Since this is an integer, and since color resources are integers, sometimes code incorrectly passes the color resource id rather than a resolved color to the setter: paint.setColor(R.color.red) Obviously, the color should be "resolved" first via getResource().getColor(). This changeset adds a lint detector which catches these kinds of bugs. Change-Id: I2970e5220d24ad3b844a9f19fbe99b932d9f7fb4
* | | Merge "Suppress support and location tracking for translation detector"Tor Norbye2012-02-216-82/+224
|\ \ \
| * | | Suppress support and location tracking for translation detectorTor Norbye2012-02-216-82/+224
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This changeset adds support for suppressing lint errors on translations. For a missing translation, place the suppress attribute on the default language key. For strings defined in other languages but not the default locale, place the attribute on the extra translation. This required adding better location tracking (which is also beneficial in that the translation warnings will show up as editor underlines in Eclipse etc). Instead of having errors generated for each locale ("the following strings are missing from locale X") it now generates a unique error for each string, listing which locales it's missing from. This also solves a different issue where the list of missing strings was truncated when large; there's no more truncation now. Change-Id: If63e92a6a750a5314af2f11441347cf9fc0b0a15
* | | Merge "Make API detector in XML files consider folder version"Tor Norbye2012-02-214-20/+55
|\ \ \ | |/ / |/| |
| * | Make API detector in XML files consider folder versionTor Norbye2012-02-214-20/+55
| | | | | | | | | | | | | | | | | | | | | | | | | | | Take the folder version into account when checking the API level in XML files. For example, even if minSdkVersion=5, it's okay to have a <GridLayout> element in a layout if that layout is in a layout-v14 (or higher) folder. Change-Id: Idbd3647c1145e4b3d03f90626339ef7e6b10c827
* | | Merge "gltrace: Show per frame summary statistics."Siva Velusamy2012-02-216-138/+419
|\ \ \
| * | | gltrace: Show per frame summary statistics.Siva Velusamy2012-02-216-138/+419
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This CL changes the framebuffer view to be a frame summary view. The Frame Summary View displays summary information regarding the currently displayed frame. For each frame, it displays: - the contents of the framebuffer at the end of the frame. - summary statistics regarding the GL Calls present in the frame. Change-Id: I293f5b0de40aac315dee257fbc3eaa3d5ff0919c
* | | | Merge "gltrace: Fix state update race condition."Siva Velusamy2012-02-211-30/+34
|\ \ \ \ | |/ / /
| * | | gltrace: Fix state update race condition.Siva Velusamy2012-02-211-30/+34
| | | | | | | | | | | | | | | | | | | | | | | | | | | | The GL State is updated in reaction to user selections, and should be performed only in a synchronized block. Change-Id: Ieeaba4fd5a088b5b9e85a21fb6738c214d6f8c01
* | | | Merge "Lint rule to catch edit-usage of TextViews (should be EditText)"Tor Norbye2012-02-219-6/+485
|\ \ \ \
| * | | | Lint rule to catch edit-usage of TextViews (should be EditText)Tor Norbye2012-02-219-6/+485
| | |/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | TextView provides a number of attributes related to text editing. However, these are generally intended for the EditText subclass of EditText. This lint check looks for <TextView> elements in layouts where one or more of the edit-related attributes (such as inputType) are defined. It also updates the deprecation detector to check various deprecated attributes (it only checked for deprecated elements before this.) Change-Id: I1b94902d906cd0bee6a6564368125a68b3c9bb2a
* | | | Merge "Use lint error mechanism for config issues like no classes found"Tor Norbye2012-02-216-13/+66
|\ \ \ \
| * | | | Use lint error mechanism for config issues like no classes foundTor Norbye2012-02-216-13/+66
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If Lint cannot find the .class files for a project, it cannot run any of the class-file based checks (such as the NewApi check). This changeset adds a new category and issue id, "Lint Error", for these types of issues. In HTML reports, these errors are listed at the top. The issue explanation states that these errors don't represent bugs in the user's code, but that lint was not able to check certain things for the reasons given. In the case of no .class files found, it asks whether the project needs to be built first. It also uses these lint errors to emit errors in processing lint.xml configuration files. (Note that if you don't want to see these types of errors, you can suppress it via --disable LintError.) Change-Id: Ifc2f55566f3a0cd20189d43e4205201bc21ee280
* | | | Merge "Fix lint library project configuration handling"Tor Norbye2012-02-215-15/+17
|\ \ \ \
| * | | | Fix lint library project configuration handlingTor Norbye2012-02-215-15/+17
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | A lint.xml file in a library project should be able to suppress lint errors in that project. This changeset fixes a couple of bugs which prevented this from working: In Eclipse, the configuration was cached, which meant a single lint run through multiple projects would just use the configuration of the first encountered project. And from the command line driver, the code to handle --check handling would override configuration severity in one scenario. Change-Id: Ib08e2b14582ab53e5ab89244738d1bd77839610e
* | | | Merge "SDK Stats: report OS vs JVM architecture."Raphael2012-02-211-32/+180
|\ \ \ \ | |_|/ / |/| | |
| * | | SDK Stats: report OS vs JVM architecture.Raphael2012-02-171-32/+180
| | | | | | | | | | | | | | | | Change-Id: I1fef83d30fc9e7fa91c136fa9c459bf18a1e1d9c
* | | | ADT editors Hyperlinks potentially not using the right GLE.Raphael2012-02-215-7/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This fixes a wrong GLE reference in Hyperlinks.getCongigutation(). Also suppresses a few annoying warnings (mostly obsolete imports.) Change-Id: Ibff38ecf3048d96c2eb3090b2dfd2f4066fe8bc6
* | | | ADT fix save on layout configuration change.Raphael2012-02-2112-79/+80
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When a layout is opened and its configuration is changed, the new delegate layout wasn't properly saving the layout before changing files. This is due to another instance of the delegate code calling a method on the delegate whereas it should have called a method with the same exact name on the editor. To reduce this confusion, this CL adds a 'delegate' prefix to methods which are common in the CommonXmlEditor and the delegates. At the expense of being a bit more verbose it makes it clearer which method is being invoked. Change-Id: I34e620a9d25272fe75dab54279161b04a961a598
* | | | ADT fix GLE configuration change.Raphael2012-02-212-3/+4
| |/ / |/| | | | | | | | | | | | | | SDK Bug: 6036506 Change-Id: I92f86940bf158ca8da8597a7f387d1f2595c0131
* | | Merge "Lint rule checking that activities and services are registered"Tor Norbye2012-02-2119-2/+501
|\ \ \ | |_|/ |/| |
| * | Lint rule checking that activities and services are registeredTor Norbye2012-02-1619-2/+501
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This changeset adds a lint rule checking that any named, non-abstract classes which extend Activity, Service or ContentProvider have a corresponding registration in the manifest file. It also ensures that the registration is under the right kind of tag (<activity>, <service>, <provider> or <receiver>). Change-Id: Ifbb587ae7b3eb6529d5099b33245d0478d48c41f
* | | Merge "gltrace: Show wall time and thread time."Siva Velusamy2012-02-176-15/+83
|\ \ \
| * | | gltrace: Show wall time and thread time.Siva Velusamy2012-02-176-15/+83
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Extend the protobuf to include wall time and thread time. Both durations are displayed in the function view. Change-Id: I407255464469195f7e6cf1a4b526637cb7c02823
* | | | Merge "Fix lombok jar location name"Tor Norbye2012-02-171-1/+1
|\ \ \ \ | |/ / / |/| | |
| * | | Fix lombok jar location nameTor Norbye2012-02-171-1/+1
| | | | | | | | | | | | | | | | Change-Id: I4881ffe726e72549b5a782ac648ae8dcbec19435
* | | | Fix build: generate addon.xml in v3, not new v4 xsdRaphael2012-02-171-0/+0
| |_|/ |/| | | | | | | | Change-Id: I7a416dfacfa871755ce86660feb73b699ff97dc1
* | | Merge "Remove warning that minsdkVersion is lower than the compile target."Xavier Ducrohet2012-02-163-21/+14
|\ \ \ | |/ / |/| |
| * | Remove warning that minsdkVersion is lower than the compile target.Xavier Ducrohet2012-02-163-21/+14
| | | | | | | | | | | | | | | | | | | | | This is not needed anymore now that Lint can check if this is actually a problem or not. Change-Id: I9f43f46033e07c44b78728e8582364b355ff1ccf
* | | Add @android resource version checking to the lint API detectorTor Norbye2012-02-168-35/+159
| |/ |/| | | | | | | | | | | | | This changeset makes the API lint detector look at XML attribute values and XML value text nodes and check any references to @android resources to ensure that they are available in all supported versions. Change-Id: Iab0d23423c30381e06b32f54aa902a31cc1f9a1c
* | Add suppress-scope handling for more XML detectorsTor Norbye2012-02-1621-85/+458
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This changeset adds is-suppressed checking to most of the remaining XML detectors. These are detectors which do post processing on data in order to compute the warnings, at which point they no longer have access to the original DOM node context. This changeset adds various checks to handle this, such as storing the DOM node in the location handle client data and checking explicitly before using the handles. It adds various unit tests for the suppress scenarios too. There are now two remaining detectors which don't properly handle suppression: the TranslateDetector, and the DuplicateIdDetector (in cross layout scenarios). These will be addressed in a separate CL. Change-Id: I53e792dd2f2802c7dce16beb73127f20d9b2dba8
* | Build fix: fix import packageTor Norbye2012-02-162-2/+4
| | | | | | | | Change-Id: Id8f467c5a088101e6b50c33fa0703520e31bdede
* | Merge "Add onClick check for lint"Tor Norbye2012-02-169-28/+521
|\ \
| * | Add onClick check for lintTor Norbye2012-02-169-28/+521
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This lint check makes sure that if a layout references android:onClick="foo" then one of the Java classes provide a public void foo(View) method. It also looks for simple typos, and warns if the access modifiers are not right. (It also moves some ASM-related utility code out of specific detectors and into the ClassContext class) Change-Id: Ifb0820221175a0760b83992f54e99f761b4d7097
* | | Merge "hierarchyviewer: Add DumpDisplayList action."Siva Velusamy2012-02-161-0/+3
|\ \ \ | |_|/ |/| |
| * | hierarchyviewer: Add DumpDisplayList action.Siva Velusamy2012-02-161-0/+3
| | | | | | | | | | | | Change-Id: Ic9df226e765db323881cbd53f7f3220029e21fae
* | | Merge "Delayed builds from the builder."Xavier Ducrohet2012-02-162-2/+40
|\ \ \ | |_|/ |/| |
| * | Delayed builds from the builder.Xavier Ducrohet2012-01-202-2/+40
| | | | | | | | | | | | Change-Id: I70adc2f7ad75aa631e4d90139ab7b08a6aa0b5a0