diff options
| author | zea@chromium.org <zea@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-05-21 01:27:38 +0000 |
|---|---|---|
| committer | zea@chromium.org <zea@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-05-21 01:27:38 +0000 |
| commit | 501ff56703d315d36e6d576f29a3e9466b356596 (patch) | |
| tree | 851cb0a55199d7d815960bddf6390e0c6c6db798 | |
| parent | 55f45227d8e7d70c1f8490c271f6aadda5499d34 (diff) | |
| download | chromium_src-501ff56703d315d36e6d576f29a3e9466b356596.zip chromium_src-501ff56703d315d36e6d576f29a3e9466b356596.tar.gz chromium_src-501ff56703d315d36e6d576f29a3e9466b356596.tar.bz2 | |
Revert 271773 "Fix RlzLibTest by setting OmniboxLog.is_popup_ope..."
Breaks official mac builder. See bug.
> Fix RlzLibTest by setting OmniboxLog.is_popup_open to be true in function SimulateOmniboxUsage.
>
> BUG=369249
>
> Review URL: https://codereview.chromium.org/282293003
TBR=yiyaoliu@chromium.org
Review URL: https://codereview.chromium.org/299703005
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@271790 0039d316-1c4b-4281-b951-d872f2087c98
| -rw-r--r-- | chrome/browser/rlz/rlz_unittest.cc | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/chrome/browser/rlz/rlz_unittest.cc b/chrome/browser/rlz/rlz_unittest.cc index 81686da..2b09959 100644 --- a/chrome/browser/rlz/rlz_unittest.cc +++ b/chrome/browser/rlz/rlz_unittest.cc @@ -230,12 +230,9 @@ void RlzLibTest::SetRegistryBrandValue(const wchar_t* name, #endif void RlzLibTest::SimulateOmniboxUsage() { - // Set is_popup_open for the OmniboxLog to be true. - OmniboxLog dummy = OmniboxLog(NULL, false, NULL, true, NULL, NULL, NULL, NULL, - NULL, NULL, NULL, NULL); tracker_.Observe(chrome::NOTIFICATION_OMNIBOX_OPENED_URL, content::NotificationService::AllSources(), - content::Details<OmniboxLog>(dummy)); + content::Details<OmniboxLog>(NULL)); } void RlzLibTest::SimulateHomepageUsage() { |