Revert of Instrumenting opening password database to find jank

Revert of Instrumenting opening password database to find jank (patchset #1 id:1 of https://codereview.chromium.org/807263003/)

Reason for revert:
Undoing profiler instrumentations since the jank is fixed.

Original issue's description:
> Instrumenting opening password database to find jank.
>
> This database opening happens in UI thread. Checking what jankiness this causes.
>
> Background: one of jankiness investigations (crbug.com/426272) came to a conclusion that the jank is actually caused by this issue. Instrumenting to get the numbers.
>
> BUG=138903
>
> Committed: https://crrev.com/42466460ec1a5c0f2953d13e57d0d290cb38ca21
> Cr-Commit-Position: refs/heads/master@{#308702}

TBR=isherman@chromium.org
BUG=138903
Committed: https://crrev.com/d6300bd0250c5b55a4b8021e930a216ee959b5cd
Cr-Commit-Position: refs/heads/master@{#313168}

[vadimt, 2015-01-26 21:54:47]

Created Revert of Instrumenting opening password database to find jank

[commit-bot: I haz the power, 2015-01-26 21:55:53]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/878763002/1

[commit-bot: I haz the power, 2015-01-26 21:56:34]

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

[commit-bot: I haz the power, 2015-01-26 21:56:35]

Failed to apply patch for
components/password_manager/core/browser/login_database.cc:
While running git apply --index -3 -p1;
  error: patch failed:
components/password_manager/core/browser/login_database.cc:157
  Falling back to three-way merge...
  Applied patch to 'components/password_manager/core/browser/login_database.cc'
with conflicts.
  U components/password_manager/core/browser/login_database.cc

Patch:       components/password_manager/core/browser/login_database.cc
Index: components/password_manager/core/browser/login_database.cc
diff --git a/components/password_manager/core/browser/login_database.cc
b/components/password_manager/core/browser/login_database.cc
index
a9d329f3d63f36b1bbc97022500f1b0882adf7c1..ef739c3b74169c539761387af47218c2930a1421
100644
--- a/components/password_manager/core/browser/login_database.cc
+++ b/components/password_manager/core/browser/login_database.cc
@@ -12,7 +12,6 @@
 #include "base/logging.h"
 #include "base/metrics/histogram.h"
 #include "base/pickle.h"
-#include "base/profiler/scoped_tracker.h"
 #include "base/strings/string_util.h"
 #include "base/strings/stringprintf.h"
 #include "base/time/time.h"
@@ -157,15 +156,9 @@
   db_.set_exclusive_locking();
   db_.set_restrict_to_user();
 
-  {
-    // TODO(vadimt): Remove ScopedTracker below once crbug.com/138903 is fixed.
-    tracked_objects::ScopedTracker tracking_profile(
-        FROM_HERE_WITH_EXPLICIT_FUNCTION("138903 LoginDatabase::Init db
init"));
-
-    if (!db_.Open(db_path)) {
-      LOG(WARNING) << "Unable to open the password store database.";
-      return false;
-    }
+  if (!db_.Open(db_path)) {
+    LOG(WARNING) << "Unable to open the password store database.";
+    return false;
   }
 
   sql::Transaction transaction(&db_);

[commit-bot: I haz the power, 2015-01-26 21:56:55]

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

[vadimt, 2015-01-26 22:39:33]

The CQ bit was checked by vadimt@chromium.org

[commit-bot: I haz the power, 2015-01-26 22:40:20]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/878763002/40001

[commit-bot: I haz the power, 2015-01-26 23:36:17]

Committed patchset #2 (id:40001)

[commit-bot: I haz the power, 2015-01-26 23:38:32]

Patchset 2 (id:??) landed as
https://crrev.com/d6300bd0250c5b55a4b8021e930a216ee959b5cd
Cr-Commit-Position: refs/heads/master@{#313168}

[Ilya Sherman, 2015-01-26 23:47:56]

LGTM.  Thanks for coming back around to this cleanup task :)

[vadimt, 2015-01-26 23:48:56]

My pleasure :)

On Mon, Jan 26, 2015 at 3:47 PM, <isherman@chromium.org> wrote:

> LGTM.  Thanks for coming back around to this cleanup task :)
>
> https://codereview.chromium.org/878763002/
>

To unsubscribe from this group and stop receiving emails from it, send an email
to chromium-reviews+unsubscribe@chromium.org.