Re-land: Change actions.txt to actions.xml

Re-land: Change the user action file format from .txt to .xml.

In this way, more information can be added (currently added 'description' and 'owner' for each action) 

A few functions are moved from tools/metrics/histograms to tools/metrics/common to be shared by tools/metrics and tools/histograms.

CL 149503005 broke in http://crrev.com/255357. Add a patch to fix histograms presubmit. 

BUG=340735
NOTRY=true

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

[Alexei Svitkine (slow), 2014-03-06 20:46:10]

Changes LGTM.

Please re-use the same description as the previous CL but add a note about the
extra bugfixes in this version.

Does this also handle the executable bit issue that was mentioned in the revert?

[yiyaoliu, 2014-03-06 20:57:56]

On 2014/03/06 20:46:10, Alexei Svitkine wrote:
> Changes LGTM.
> 
> Please re-use the same description as the previous CL but add a note about the
> extra bugfixes in this version.
> 
> Does this also handle the executable bit issue that was mentioned in the
revert?

I think so, at least the executable bit for both print_style.py are not set now.

[yiyaoliu, 2014-03-06 21:13:25]

Hi Jói, 

I made a mistake in CL 149503005, which is reverted and I'm adding a new patch
to fix it. src/PRESUBMIT.py is not changed from your last review. 
Could you take a look and LGTM it? 

Thanks, 
Yiyao

[Jói, 2014-03-06 21:51:52]

LGTM

[yiyaoliu, 2014-03-07 19:06:48]

The CQ bit was checked by yiyaoliu@google.com

[commit-bot: I haz the power, 2014-03-07 19:07:22]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/yiyaoliu@chromium.org/180743027/20001

[commit-bot: I haz the power, 2014-03-07 19:07:36]

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

[commit-bot: I haz the power, 2014-03-07 19:07:37]

Failed to apply patch for PRESUBMIT.py:
While running patch -p1 --forward --force --no-backup-if-mismatch;
  patching file PRESUBMIT.py
  Hunk #1 FAILED at 1014.
  1 out of 1 hunk FAILED -- saving rejects to file PRESUBMIT.py.rej

Patch:       PRESUBMIT.py
Index: PRESUBMIT.py
diff --git a/PRESUBMIT.py b/PRESUBMIT.py
index
1e6526f26ea9c9dca0f46179f948e5a14216afed..c7b9fde382ad0d393c43de587156eae447764df1
100644
--- a/PRESUBMIT.py
+++ b/PRESUBMIT.py
@@ -1014,28 +1014,32 @@ def _CheckCygwinShell(input_api, output_api):
 
 def _CheckUserActionUpdate(input_api, output_api):
   """Checks if any new user action has been added."""
-  if any('chromeactions.txt' == input_api.os_path.basename(f) for f in
+  if any('actions.xml' == input_api.os_path.basename(f) for f in
          input_api.LocalPaths()):
-    # If chromeactions.txt is already included in the changelist, the PRESUBMIT
-    # for chromeactions.txt will do a more complete presubmit check.
+    # If actions.xml is already included in the changelist, the PRESUBMIT
+    # for actions.xml will do a more complete presubmit check.
     return []
 
-  with open('tools/metrics/actions/chromeactions.txt') as f:
-    current_actions = f.read()
-
   file_filter = lambda f: f.LocalPath().endswith(('.cc', '.mm'))
   action_re = r'[^a-zA-Z]UserMetricsAction\("([^"]*)'
+  current_actions = None
   for f in input_api.AffectedFiles(file_filter=file_filter):
     for line_num, line in f.ChangedContents():
       match = input_api.re.search(action_re, line)
       if match:
+        # Loads contents in tools/metrics/actions/actions.xml to memory. It's
+        # loaded only once.
+        if not current_actions:
+          with open('tools/metrics/actions/actions.xml') as actions_f:
+            current_actions = actions_f.read()
+        # Search for the matched user action name in |current_actions|.
         for action_name in match.groups():
-          name_pattern = r'\t%s\n' % action_name
-          if name_pattern not in current_actions:
+          action = 'name="{0}"'.format(action_name)
+          if action not in current_actions:
             return [output_api.PresubmitPromptWarning(
               'File %s line %d: %s is missing in '
-              'tools/metrics/actions/chromeactions.txt. Please run '
-              'tools/metrics/actions/extract_actions.py --hash to update.'
+              'tools/metrics/actions/actions.xml. Please run '
+              'tools/metrics/actions/extract_actions.py to update.'
               % (f.LocalPath(), line_num, action_name))]
   return []

[yiyaoliu, 2014-03-07 19:18:41]

The CQ bit was checked by yiyaoliu@google.com

[commit-bot: I haz the power, 2014-03-07 19:20:21]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/yiyaoliu@chromium.org/180743027/60001

[commit-bot: I haz the power, 2014-03-07 19:21:55]

Change committed as 255680